-
r4v3r23[m]
UkoeHBcan you confirm how cold-signing will work in seraphis? i remember you said somewhere key images won't need to be imported/exported, but will outputs/enotes still need to be? or will it be simplified down to just unsigned tx/signed tx?
-
UkoeHB
r4v3r23[m]: yes just unsigned tx / signed tx
-
r4v3r23[m]
thanks
-
rbrunner
Where do I start my search for the problem if a release build is successful but a debug build fails, while linking as it seems, with some missing symbol
-
rbrunner
I try to compile UkoeHB's latest seraphis_lib branch, "make" goes through, "make debug" gives me this error:
paste.debian.net/1285853
-
moneromoooo
Check the lib with work_while_waiting is linked in after scan_ledger_chunk_async.cpp.
-
moneromoooo
If you're unlucky, there's a circular dep.
-
rbrunner
What is "work_while_waiting"?
-
moneromoooo
I can only guess. I did not work with that code.
-
rbrunner
Ah, you reference a symbol in that code. I was taking it as the name of some analysis tool, linker option, or whatever :)
-
moneromoooo
Yes, it's the one the error complains about.
-
rbrunner
Alright, thanks.
-
UkoeHB
rbrunner: nuke you build directory and try again, might fix it
-
UkoeHB
this line is quite suspicious src/wallet/CMakeFiles/wallet_rpc_server.dir/wallet_rpc_server.cpp.o
-
rbrunner
Well, that's more or less random, I compiled with 4 threads, and that's just what was compiled in one of them before another one had the error
-
rbrunner
But good idea anyway, C++ dev's equivalent of "Have you already tried switching it off and on again" :)
-
UkoeHB
ah righ
-
recanman[m]
Would this regex be correct for a standard monero address? /(?:^[48][0-9AB][1-9A-HJ-NP-Za-km-z]{93}$)/
-
recanman[m]
* Would this regex be correct for a standard monero address? /(?:^\[48\]\[0-9AB\]\[1-9A-HJ-NP-Za-km-z\]{93}$)/
-
recanman[m]
* Would this regex be correct for a standard monero address?
-
recanman[m]
`/(?:^\[48\]\[0-9AB\]\[1-9A-HJ-NP-Za-km-z\]{93}$)/`
-
recanman[m]
I found a few online but some didn't work with subaddresses, so I changed them around a bit
-
recanman[m]
recanman[m]: > <@recanman:agoradesk.com> Would this regex be correct for a standard monero address?
-
recanman[m]
> `/(?:^\[48\]\[0-9AB\]\[1-9A-HJ-NP-Za-km-z\]{93}$)/`
-
recanman[m]
I don't mean to match integrated addresses
-
recanman[m]
Is the only monero javascript library this onehttps://github.com/monero-ecosystem/monero-javascript?
-
recanman[m]
s/onehttps/one https/
-
sech1
-
sech1
I checked a few addresses and it matched them
-
recanman[m]
Thank you, I just wanted to make sure I didn't miss any edge cases
-
sech1
[0-9AB] can be changed to [1-9AB]
-
sech1
because base58 doesn't have '0'
-
recanman[m]
Thank you
-
UkoeHB
rbrunner: it is also failing for me, I must have broken something
-
hyc
jeffro256[m]: of course not.
-
woodser[m]
I'm wondering if this is a valid assertion added to the release-v0.18 branch to check if outputs are frozen before using them:
github.com/monero-project/monero/bl…50be6d/src/wallet/wallet2.cpp#L2008
-
woodser[m]
for example, can key images be imported to the multisig participants in different orders and times, so the key image ordering could be different?
-
woodser[m]
asking because this wasn't asserted in the code prior, and my multisig wallets were working
-
woodser[m]
whereas now I hit that exception on signing the multisig tx
-
jberman[m]
I recall double checking in the scan_tx PR that multisig does rely on consistently ordered key image imports
-
jberman[m]
need to re-check, not sure why you didn't run into issues prior to that change
-
UkoeHB
I don't think you can rely on ordering, an oversight in that PR?
-
jberman[m]
^that makes more sense. Nevermind what I said. I had checked that multisig relies on consistent ordering of m_transfers which key image imports shouldn't affect
-
UkoeHB
rbrunner: ok should be fixed
-
moneromoooo
m_transfers should be ordered in (canonical) blockchain order.
-
rbrunner
UkoeHB: Thanks, will have a look probably on Sunday.