Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support multithreading #97

Closed
wants to merge 5 commits into from
Closed

Support multithreading #97

wants to merge 5 commits into from

Conversation

woodser
Copy link
Owner

@woodser woodser commented Jul 8, 2022

Closes #96

This PR updates most of the configuration to support multithreading.

The build completes most of the way but fails with the following error during final linking:

[100%] Linking CXX executable monero_wallet_full.js
Error running link command: No such file or directory
make[2]: *** [monero_wallet_full.js] Error 2
make[1]: *** [CMakeFiles/monero_wallet_full.dir/all] Error 2
make: *** [all] Error 2
emmake: error: 'cmake --build . -j16' failed (returned 2)

Any help would be greatly appreciated. Starting points could be reading the latest documentation to support pthreads in emscripten, and playing with the shared options in build_openssl_emscripten.sh.

Before attempting to build, these folders should first be deleted so the full stack is built with pthreads enabled:

./build
./external/monero-cpp/external/boost-sdk
./external/monero-cpp/external/openssl-sdk

@woodser woodser marked this pull request as draft July 8, 2022 18:04
@woodser woodser added the help wanted Extra attention is needed label Jul 8, 2022
@woodser woodser changed the title update configuration to support multithreading Support multithreading Jul 8, 2022
bump version to v0.7.3
add to build script: git submodule update --init --force
rename emsdk_inc.sh to download_deps.sh
@woodser
Copy link
Owner Author

woodser commented Aug 30, 2022

Supeseded by #102

@woodser woodser closed this Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support multithreading in wasm
1 participant