08:42:20 <br-m> <rbrunner7> @tobtoht:monero.social: I have a new problem with the Polyseed code that must have crept in quite recently. Symptom is a problem with the Polyseed dependencies. By setting a breakpoint in the Dependency constructor, I can confirm that it runs and initializes static dependency deps; in polyseed.cpp. But if a bit later a P [... too long, see https://mrelay.p2pool.observer/e/5IHvmaELRWcydzZK ]
08:43:38 <br-m> <rbrunner7> One suspicion that I have but that I could not yet verify so far is that after the recent Polyseed PR (see https://github.com/tevador/polyseed/pull/20/changes) we somehow have that static deps variable twice, and only one of the two gets initialized correctly.
08:44:32 <br-m> <rbrunner7> (twice in memory, in 2 different parts of the resulting binary)
08:46:56 <br-m> <rbrunner7> If that does not look like the probable culprint, any idea where and how to continue the search for the root of the problem? This goes quite a bit over my low-level C/C++ knowledge :)
09:00:50 <xmr-pr> bas-kirill opened issue #11126: Suggest `hw_reconnect` when hardware wallet HID communication fails du...
09:00:51 <xmr-pr> > https://github.com/monero-project/monero/issues/11126
09:14:34 <br-m> <tobtoht> @rbrunner7:monero.social: Is the issue still present in your rebase? I can look at it now.
09:15:44 <br-m> <rbrunner7> Yes, I rebased now to test again, just to make sure that I don't chase any gosts. The issue persists.
11:28:47 <selsta> .merge+ 11081 11080 11079 11078 11077 11075 11073 11062
11:28:47 <xmr-pr> Added
14:18:26 <br-m> <sneedlewoods_xmr:matrix.org> I tested it with polyseed_static changed back to polyseed in src/mnemonics/polyseed/CMakeLists.txt and it works. > <@rbrunner7> One suspicion that I have but that I could not yet verify so far is that after the recent Polyseed PR (see https://github.com/tevador/polyseed/pull/20/changes) we somehow have that static deps variable twice, and only one of the two gets initialized correctly.
14:18:26 <br-m> <sneedlewoods_xmr:matrix.org> With ldd monero-wallet-cli I see that libpolyseed.so.2 is missing when using polyseed_static in target_link_libraries(). The binary size is 23MB for non-static and 24MB for static.
15:17:37 <br-m> <tobtoht> Reverting to dynamic linking is not a solution. It must be linked statically for release builds.
15:17:58 <br-m> <syntheticbird> dynamic linking in general is a failure
15:20:47 <br-m> <sneedlewoods_xmr:matrix.org> AFAICT polyseed_static works for release builds but not for debug builds, which is what confuses me
15:34:10 <br-m> <rbrunner7> Now, that's interesting. Can confirm: A release build works, a debug build doesn't.
15:39:36 <br-m> <rbrunner7> Well, could we take out the hammer and use polyseed_static for release builds, and polyseed for debug builds? Surely that CMake thingy is smart enough for something like that? :)
15:50:03 <jpk68> .merges
15:50:03 <xmr-pr> 10865 10872 10886 10887 10964 11046 11066 11077 11078
15:51:01 <br-m> <syntheticbird> .merged
15:51:10 <br-m> <syntheticbird> damn i can't even joke without a typo
16:45:50 <xmr-pr> Ap4sh opened pull request #11127: epee: parse HTTP methods case-sensitively
16:45:51 <xmr-pr> > https://github.com/monero-project/monero/pull/11127
16:46:59 <br-m> <syntheticbird> LETS FUCKING GO
16:47:10 <br-m> <syntheticbird> sorry i overreacted
17:16:36 <br-m> <basses:matrix.org> only syn issues get fixed in less than a day
17:17:23 <br-m> <syntheticbird> my issues are small tbf