00:48:02 <selsta> luigi1111w: do you have time for cli and gui merges tonight or tomorrow? would make it easier to check what's left to do
01:03:37 <luigi1111w> .merges
01:03:37 -xmr-pr- 7805 7822 7825 7847 7850 7893 7894
01:20:43 <selsta> luigi1111w: could you approve the workflow here? https://github.com/monero-project/monero/pull/7897 ?
01:21:08 <luigi1111w> done
01:31:34 <selsta> thnks
04:39:35 <mj-xmr[m]> selsta: what does it mean to approve the workflow in this context? All the checks are failing there for a good reason.
05:02:44 <jon_> What's the status of NFTs and monero currently anything in dev ?
05:37:48 <mj-xmr[m]> selsta: I get it now. It couldn't start before lugi approves.
05:43:24 <mj-xmr[m]> Just a heads up to all patch (/PR) contributors: the new CI features like more granular caching, won't work until you rebase.
05:44:33 <mj-xmr[m]> It is kind of a loophole of GitHub I must admit. The person who has an older CI version in a given PR (before rebasing), could be pushing a breaking change, which was detected outside of the person's PR.
08:01:28 <Space[m]> hello. What I need do with this error?
08:01:29 <Space[m]> https://bin.snopyta.org/?369d2bf7bf4441d5#6q68d6nmDJLYS1PVeaBWM7F2JaieXgJQxPVcYGDWvuqt
08:04:59 <Space[m]> > <@bigmenpixel:matrix.org> hello. What I need do with this error?
08:04:59 <Space[m]> > https://bin.snopyta.org/?369d2bf7bf4441d5#6q68d6nmDJLYS1PVeaBWM7F2JaieXgJQxPVcYGDWvuqt
08:04:59 <Space[m]> this monero 0.17.2.0
12:15:35 <mj-xmr[m]> > <@bigmenpixel:matrix.org> hello. What I need do with this error?
12:15:35 <mj-xmr[m]> > https://bin.snopyta.org/?369d2bf7bf4441d5#6q68d6nmDJLYS1PVeaBWM7F2JaieXgJQxPVcYGDWvuqt
12:15:35 <mj-xmr[m]> Hi. It appears that the <algorithm> header is missing in that file. These problems happen across various compilers, since some toolchains include the header implicitly in other headers, wile others don't.
12:16:25 <mj-xmr[m]> I'd suggest including that header in the file that failed to compile and PR it or pasting a patch that does this.
12:17:26 <mj-xmr[m]> In other words - the code is wrong, not your setup.
12:17:56 <mj-xmr[m]> and "that file" is `src/common/i18n.cpp`
12:35:14 <mj-xmr[m]> Space: also please check if the same problem happens on Master in your setup.
13:02:25 <selsta> .merge+ 7828
13:02:25 <xmr-pr> Added
13:46:28 <selsta> hyc: https://github.com/monero-project/monero/commit/987eb662bfad092ce812041808d4547889edf562
13:47:06 <selsta> do you remember how to reproduce this linking issue?  I undid these changes in https://github.com/monero-project/monero/pull/7898 and can't reproduce the linking issue
13:49:16 <selsta> will try with added readline to CI
14:07:16 <hyc> selsta: sorry no, don't remember
14:07:32 <hyc> hard to believe that's 4yrs ago already
14:08:19 <hyc> you'd need to check a full gitian build; probably the problem shows up on non-linux
14:11:31 <selsta> we do run gitian on all PRs
14:11:45 <selsta> seems to compile fine everywhere
14:12:06 <selsta> wait
14:12:10 <selsta> we run depends, not gitian
14:12:13 <selsta> but shouldn't matter
14:12:45 <selsta> https://github.com/monero-project/monero/actions/runs/1172563687
14:14:33 <jerfo> selsta: Thanks for answering. You said "integrated addresses = payment ids, subaddresses just have an index". And while that is true, all TXs have a payment ID correct? Not that it should necessarily be done, but you can provide payment IDs to subaddresses, correct ?
14:15:06 <selsta> subaddresses don't support payment ids
14:15:41 <selsta> all transactions have a dummy payment id, yes
14:15:52 <hyc> I don't see android on those 
14:17:57 <jerfo> Is it possible to utilize those payment IDs on those TXs without a fork, just wallet changes? I was just wondering because theres talk on GitHub about deprecating integrated addresses, but I see no other way to solve my issue w/o integrated addresses.
14:20:36 <selsta> hyc: wallet_rpc links against `${EPEE_READLINE}` but not `${GNU_READLINE_LIBRARY}` so that could be an issue
14:20:48 <selsta> simplewallet links against both
14:23:57 <hyc> wallet_rpc has no cli so it shouldn't need gnu readline
14:25:03 <selsta> I meant ./monero-wallet-rpc binary, not sure if I looked at the correct cmake target
14:25:15 <selsta> It was added here https://github.com/monero-project/monero/commit/c336d0f217bbb8a8417ebcb362abc9359fcc5d43
14:26:30 <hyc> whatever. if wallet-rpc binary links without errors then it's of no concern
14:26:34 <selsta> ok
14:27:13 <selsta> will test android too then it should be good
14:30:39 <selsta> jerfo: I don't really understand what you want to do
14:32:39 <jerfo> Well before I explain, I guess it would be safe to say that if integrated addresses were deprecated, that the payment ID field would also be deprecated right?
14:45:17 <moneromoooo> Integrated addresses are not deprecated. Long unencrypted payment ids are deprecated. Integrated addresses use short encrypted payment ids.
14:46:43 <moneromoooo> You could write code to add whatever info you like to txes, and code to interpret that info. Consensus won't stop you (though it might in the future).
15:33:55 <jerfo> Sorry I meant hypothetically. I know they aren't deprecated now, I meant to ask that in the future, IF integrated addresses were to bedeprecated, that the payment ID field would also.
15:34:42 <selsta> integrated addresses are just a different way for writing address + payment id
15:34:50 <selsta> so yes
16:08:16 <rbrunner> Maybe jerfo got nervous because of this proposal to drop intergrated addresses: https://github.com/monero-project/monero/issues/7889
16:08:29 <rbrunner> Also discussed on Reddit
17:55:00 <Space[m]> <mj-xmr[m]> "Space: also please check if..." <- Ok, thanks, I'll check tomorrow
23:00:09 <nikg83[m]> Are these even used by monero ? 
23:00:09 <nikg83[m]> https://github.com/monero-project/monero/search?q=smtp&type=
23:11:53 <moneromoooo> No.