Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I tried building the WASM bundle following the instructions in @mymonero/mymonero-monero-client, but ran into errors.
This pull request fixes the problems I ran into with the prepare script,
and then hacks around the remaining C++ compiler errors. The C++ hacks are almost certainly the wrong solution, but I cannot find the matching branch in mymonero-core-cpp with the correct solution. If you know where that code is, I would be happy to update the PR.Edit: I found the correct code in the
webassembly-cleanup
branch, so I dropped the hacks from the PR. This is now clean & ready to merge.I also have some comment updates and an unnecessary header removal. These are useful in any case, so I could submit them as a separate PR if you don't want to mix these cleanups with the build fixes.