selsta

Results 960 comments of selsta

It did build the docker image correctly, it failed to run it but that appears to be unrelated. Will open a PR with the changes.

https://github.com/google/oss-fuzz/pull/12138

I don't have a Google account unfortunately to sign the CLA. Can you add the commit to this PR, or should I ask someone else from the monero project to...

Will be re-opened by a different person who has the CLA signed.

Since this is a debug build, do the same issues happen with a release build? Also can you share which command you used to build?

The second build error you posted is expected without applying https://github.com/monero-project/monero/pull/9450 / https://github.com/monero-project/monero/pull/9462

It seems the issue is specifically debug related, can you update the issue title to add debug build?

My preference would be to have -dev for discussing short term changes (existing bugs, already PRd changes, releases), while for long term development (Seraphis, FCMP) to have a separate channel.

The sweep transaction was sent from a different wallet cache, the transaction key isn't known.

The three points you have written above all sound good and were necessary to get this merged. In file src/wallet/wallet_rpc_server_commands_defs.h, please update `WALLET_RPC_VERSION_MINOR` to indicate API changes.