02:08:18 selsta:could i test it,could i,couldi?plz. 02:28:29 .merges 02:28:29 -xmr-pr- 8299 8323 8352 8359 8379 8415 8419 8427 8428 8442 8444 8450 8451 8454 8465 8466 8483 05:36:30 IITC it can happen if a peer is very slow. monerod can then decide the peer isn't going to reply, and ask again (but this time, what's needed may be different). So if the peer really was slow, rather than didn't send, and sends later, the preivously requested block doesn't match the new request. 08:03:37 moneromooo: I don't think that's the reason for the error. The block hashes I'm getting do not exist in the main chain, e.g. b188d86adc0648692aa1ef28a40234de7cb6dc3a4667e352391387d1aca1adf9 10:20:00 Hello, Software Engineer here, always thought about contributing open-source projects and now I have some free time... can I talk to someone, I would love to join and contribute / help, I would just need some quick directions,. 10:29:13 Hi. 10:30:00 I usually tell people to use the software, find something that's a bit rough somewhere and improve it. It means it's usually a small/easy thing and you have a tangible benefit. 10:30:15 Otherwise, you can check the issues list on github to have ideas. 10:30:51 When posting a patch, please try to have "small" ones, within reason. That is, no random renaming/reindent, several patches if you awnt to do several things, etc. 10:31:17 If you have particular questions, just ask, and wait for someone who can answer to be around. 10:33:22 Okay I will do it this way then, thanks for the quick answer! 10:34:54 If unsure about whether we'll ok a patch, ask first. Rwriting for "I prefer it this way" is typically unwanted. 10:35:25 (and there are parts of the code which a lot of people would prefer another way :P) 10:35:59 Oh, one easy first patch: 10:36:30 a couple days ago, someone had trouble with something, and they ended up writing a sweep_all (IIRC) command with the wrong syntax (swapped parameters). 10:36:58 There was no error. Having an error there when the parameter order is wrong would be nice. 10:37:13 Seems an easy first issue if you like. 10:42:18 It sounds the best for starting, I will look into that. 10:43:29 Sounds good. In case you don't know, there is a testnet for monero (parallel network for testing). Always use that for testing, not the live network. monero tools take the --testnet command line parameter to use the test network. 10:55:39 How testnet deamon says it found block when starting mining to a wallet address even if there is not block chain database provided 10:58:24 Someone needs to learn to ask good questions... 13:23:50 moneromooo: should this predicate be negated? https://github.com/monero-project/monero/commit/79e09fbf8c5f04a7f308762a5bc7a0927a4cd909#r80623901 13:29:05 I don't think so. 13:29:35 View wallets are no good reason to import outputs I thin. 13:29:41 have no 13:30:57 And AFACIT negating the predicate would break the existing flow. 13:53:40 .merges 13:53:40 -xmr-pr- 8299 8323 8352 8359 8379 8415 8419 8427 8428 8442 8444 8450 8465 16:50:10 https://github.com/monero-project/monero/pull/8381 needs a last approval before we can merge it 17:58:25 I read and approved it. 19:31:42 8381 isn't going to the branch? 19:33:22 there is one but I'm not sure if I want to include it in v0.18.1.0 19:35:21 sech1: should I include 8381 in v0.18.1.0 or not? I don't want any mining related bugs shortly before the HF :D 20:29:59 luigi1111w: can you also merge the remaining things + gui? I'd prefer if master and release don't get out of sync 20:41:02 moneromooo: do you remember https://github.com/monero-project/monero/pull/976 ? 20:41:54 currently on macOS beta I have to do the opposite, set e.g. max-concurrency 8 otherwise I get a crash 20:42:36 http://paste.debian.net/hidden/2ad85f51/ 20:43:57 Yes, I kinda do. 20:44:11 Vaguely... 20:44:24 I never knew the reason, so... 21:15:28 are libcncrypto.a and libwallet-crypto.a alternatives for different platforms? is one better than the other? libwallet-crypto.a is not built on some platforms 21:19:04 AFAIK libwallet-crypto.a is x86_64 asm for some functions, which are used in the wallet. 21:19:23 The idea being that you want to avoid more experimental crypto code for consensus functions. 21:28:53 selsta it's better to include 8381. That function is only used by p2pool and 8381 fixes a few issues there (including check for duplicate key images) 21:36:04 .merge+ 8455 8381 21:36:04 Added 23:57:53 luigi1111w: if you could do cli + gui merges today then we can tag tomorrow 23:58:38 fork is in 4.8 days