07:02:54 Hereby I officially apply to add my PR #8046 to the merge queue, to have it merged into master with the next merge round 07:03:13 It's tested, it's reviewed, it's ready, IMHO. See also https://github.com/monero-project/monero/pull/8076#issuecomment-1459635389 07:03:58 @selsta, please have a look. 07:04:28 Ah, PR number is wrong, it's not 8046 of course, it's 8076: https://github.com/monero-project/monero/pull/8076. 07:09:56 please approve again jberman[m] 07:19:32 done :) 11:37:17 * crowstick[m] uploaded an image: (140KiB) < https://libera.ems.host/_matrix/media/v3/download/matrix.org/DyJWXkSUHnEaczHfcXgUcBfd/Screenshot%202023-03-08%20at%2014.28.11.png > 11:37:18 I have not opened my GUI for a while but I am going back now to add some XMR and I get the following problem which I never had before. Can anyone help? 11:46:17 Are you modifying the code or trying to ? If not and you're trying to use monero as is and it doesn't work, try asking in #monero. 11:46:50 Also, I suspect you might have posted an image. A link does not show up on IRC. 12:02:09 Many thanks 12:26:30 No images matrix side 12:27:59 Oops, this is -dev 16:03:18 jeffro256[m]: this is an old callback - resolve.cpp looks up both IPv4 and IPv6 16:03:33 the code in common is more "basic" it just does one task at a time 16:04:27 so if you just want an IP 4 or 6, you have to manually do it anyway 16:04:59 you can put the code in /common with the other stuff, but since I created a /net folder long ago I decided to just start moving it into there 16:06:19 I mean you can chop the code, but then you end up re-writing get me v4 and then v6 code. and you can put in /common, but it makes more sense to be in /net and it simply has to call just a few functions in that DNSResolver class 19:33:38 .merge+ 8076 19:33:38 Added 19:54:59 Wow, thanks. 20:27:37 only to master for now, I think plowsof11 wanted to do a bit more testing 20:27:52 I'm cool with adding #8706 to master under the condition that the changes don't make it to the release branch without another PR adding code to invalidate the wallet refresh cache when switching daemons, which shouldn't be a big change at all 20:28:00 Again, great work rbrunner and jberman 20:28:17 Sorry for being so picky 20:56:48 No problem, I am cool with that, I have no doubt that further progress is possible or even necessary before release, I just did not see my PR as the good vehicle for that