01:35:43 <moneromooo> .merges?
01:35:51 <moneromooo> .merges
01:35:52 -xmr-pr- 6812 6813
01:36:32 <moneromooo> Can someone review 6815 please ? It would be nice to get in 0.17, it's a fix for pre-rct txes failing to create.
01:37:03 <moneromooo> .merge+ 6745
01:37:03 <xmr-pr> Added
01:46:46 <dsc_> moneromooo: rubber stamped
01:46:53 <moneromooo> Thank you
07:28:24 <fluffypony> http://highaltitudehacks.com/2020/09/05/arm64-reversing-and-exploitation-part-1-arm-instruction-set-heap-overflow/
07:28:33 <fluffypony> interesting read on ARM
12:01:49 <TheCharlatan> I ran a check on the contents of the merge commits after I read about sia silently introducing controversial changes.
12:02:57 <TheCharlatan> If found the following merge commits containing extra changes: 1f6e6001ed37b48788ef071f3297b7e9d67586a5 e69849e97911a6543759aabf9feccd865445337f 77224bbf4842f3c7fb53ad7685f45cf768f5ec38 7223eebbe71833e895fca95267cb243834dedf83
12:03:47 <TheCharlatan> All seems in order with them, they contain some submodule and vendored dependency changes.
12:04:19 <moneromooo> git show 1f6e6001ed37b48788ef071f3297b7e9d67586a5 doesn't show any diff. How do you get a diff ?
12:06:50 <TheCharlatan> ^ does for me.
12:07:10 <TheCharlatan> and github https://github.com/monero-project/monero/commits/1f6e6001ed37b48788ef071f3297b7e9d67586a5
12:08:38 <moneromooo> What git version ?
12:46:35 <TheCharlatan> 2.17.1
13:05:08 <moneromooo> Do you get different output than https://paste.debian.net/hidden/256af446/ ?
13:05:28 <moneromooo> Or do you use particular git options ?
13:06:36 <moneromooo> Should we bump the max out connections a bit, now that traffic has been cut a fair bit ?
13:10:20 <selsta> I have latest git version (2.28.0) and get the same output, no diff
13:10:50 <selsta> bump to 12-16?
13:17:24 <selsta> .merges
13:17:24 -xmr-pr- 6745 6812 6813
13:17:38 <selsta> .merge+ 6815
13:17:39 <xmr-pr> Added
13:22:23 <TheCharlatan> ah, my script was still showing commits with two parents. Looks like they are all clean then :)
13:55:29 <pyu> !
14:27:49 <selsta> TheCharlatan: what do you mean with descriptor .yml files in /gitian?
14:40:42 <NHnCH3> latest master "release-static"  building failed?
14:40:46 <vv01f> can you pint me to the "official URI spec"? I just found the function in the client/API and the fandom wiki… which seems not sufficient.
14:41:48 <moneromooo> NHnCH3: IIRC I used the "fandom wiki" to write the wallet source.
14:41:59 <moneromooo> er, vv01f ^
14:43:04 <moneromooo> What in particular seems not sufficient ?
14:43:15 <vv01f> k, just wanted the taler guys to add monero for RFC 8905
14:43:56 <vv01f> so I will point him to the source then with the note that the wiki was the definition.
14:48:06 <moneromooo> NHnCH3: you can check github, there's a bot building all PRs. Not sure if t's building static though. It should say in the logs.
14:50:13 <TheCharlatan> selsta these here: https://github.com/monero-project/monero/blob/master/contrib/gitian/gitian-linux.yml#L2
14:53:05 <selsta> TheCharlatan: thx updated
14:58:15 <vv01f> moneromooo: is there any direct contact address for them (Christian Grothoff) to reach out to you? (in your github account I cant find any)
15:03:08 <moneromooo> No.
15:03:20 <moneromooo> They're welcome to ask here though.
15:04:57 <vv01f> k, will mention that one.
15:08:50 <vv01f> where to publish a payto-URI specification (payto://monero/ according to the RFC) for registering the respective payment target type in the GANA registry?
15:09:31 <moneromooo> Discussing this here or in a github issue seems best.
15:09:33 <vv01f> so that monero community can give its ok beforehand and publish it as a spec.
15:09:50 <moneromooo> Probably github since it'll likely be long lived.
15:10:04 <vv01f> would you think it's worth the ffort to include monero early on in that?
15:11:24 <moneromooo> I don't know, not knowing what this is about, etc.
15:11:37 <moneromooo> Opening a github issue would be a good way to ascertain it.
15:12:18 <vv01f> the GNU Taler devs want to spec the handling of general payment URI officially and with that backing also encourage institutions to adopt
15:12:47 <vv01f> their way is specs first in general, same for PEP etc.
15:14:05 <vv01f> thanks for now, will have a look on github later on.
15:19:26 <NHnCH3> thanks moo
16:02:47 <vv01f> moneromooo: added https://github.com/monero-project/monero/issues/6817 and looking forward to comments
16:03:00 <moneromooo> ty
16:37:18 <luigi1111w> .merges
16:37:18 -xmr-pr- 6745 6812 6813 6815
22:32:11 <selsta> 6819 and then branch?
22:33:14 <selsta> .merge+ 6793 6819
22:33:14 <xmr-pr> Added
22:37:31 * moneromooo has no objection
22:37:38 <selsta> 6816 too?
22:37:40 <selsta> does not matter I guess
22:42:00 <selsta> .merge+ 6816
22:42:01 <xmr-pr> Added
22:42:43 <selsta> .merges
22:42:43 -xmr-pr- 6745 6793 6816 6819
22:43:27 <selsta> 6745 needs a rebase now
22:44:27 <moneromooo> Oh, sorry, slipped my mind. Rebasing now.
22:45:45 * moneromooo 's alerts triggered
22:45:57 <moneromooo> Latest commit on master is not signed
22:46:19 <moneromooo> luigi1111w: did you push 9bba1a24eadbc7d23a5a58d83024df224f9cafcb ?
22:46:44 <moneromooo> Or was it worked by someone after you did...
22:49:04 <moneromooo> Looks like my exact code. Thankfully a small patch.
22:50:18 <moneromooo> Odd. It says monermooo-monero. Missing o. Might be why it didn't get signed if it got modified after signing.
22:52:31 <moneromooo> Could be a test run. Modify something innocuous, see if it gets found.
22:53:05 * moneromooo looks around for occam's razor
22:53:56 <moneromooo> pushed
23:21:42 <TheCharlatan> weird mistake with the name. please blink luigi.
23:25:42 <selsta> moneromooo: does 6745 need a minor rpc bump?
23:26:51 <moneromooo> Oh yes, it does touch RPC a bit... I'll add.
23:28:18 <moneromooo> done
23:51:58 <selsta> luigi1111: please merge the merge list + branch release-v0.17 whenever you have time
23:57:19 <moneromooo> 6820 is another patch to make merge mining easier. Also comes with the same conflict of interest note as the previous one.
23:57:28 <moneromooo> This one's teeny tiny though.
23:57:50 <moneromooo> It will save verification hashing time on whoever's merge mining.