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

unexpected build errors when building with latest emsdk #90

Closed
parallastra opened this issue Aug 24, 2023 · 4 comments · Fixed by #93
Closed

unexpected build errors when building with latest emsdk #90

parallastra opened this issue Aug 24, 2023 · 4 comments · Fixed by #93
Labels
enhancement New feature or request

Comments

@parallastra
Copy link

parallastra commented Aug 24, 2023

using the following build procedure:

rm -rf emsdk wasm-audio-decoders
git clone https://github.com/emscripten-core/emsdk.git
(cd emsdk; git pull)
(cd emsdk; ./emsdk install latest)
(cd emsdk; ./emsdk activate latest)
(cd emsdk; source ./emsdk_env.sh)
git clone https://github.com/eshaz/wasm-audio-decoders.git
(cd wasm-audio-decoders; git submodule update --init)
(cd wasm-audio-decoders; make configure; npm i)
(cd wasm-audio-decoders; make clean; make)

the build fails unexpectedly with:

Building Emscripten WebAssembly module src/flac/src/EmscriptenWasm.tmp.js...
emcc: error: legacy setting used in strict mode: LLD_REPORT_UNDEFINED
make: *** [Makefile:212: src/flac/src/EmscriptenWasm.tmp.js] Error 1

$ emcc -v

emcc (Emscripten gcc/clang-like replacement + linker emulating GNU ld) 3.1.45 (ef3e4e3b044de98e1811546e0bc605c65d3412f4)
clang version 18.0.0 (https://github.com/llvm/llvm-project d1e685df45dc5944b43d2547d0138cd4a3ee4efe)
Target: wasm32-unknown-emscripten
Thread model: posix
@eshaz
Copy link
Owner

eshaz commented Aug 24, 2023

Could you try using the Emscripten version listed here: https://github.com/eshaz/wasm-audio-decoders#developing

@parallastra
Copy link
Author

parallastra commented Aug 24, 2023

Did that, found no change in behaviour, will update the original post with the extra steps.
Verified that building with emsdk v 2.0.34 works, will update title accordingly

@parallastra parallastra changed the title unexpected build errors: legacy setting used in strict mode: unexpected build errors when building with latest emsdk Aug 24, 2023
@eshaz
Copy link
Owner

eshaz commented Aug 24, 2023

Ok please use the current version for now listed in the docs to build this project. I'm going to make this a feature request to upgrade to the latest Emscipten toolchain.

@eshaz eshaz added the enhancement New feature or request label Aug 24, 2023
@parallastra
Copy link
Author

Cool, will attempt intermediate versions and try and identify where it breaks.

@eshaz eshaz mentioned this issue Oct 9, 2023
@eshaz eshaz closed this as completed in #93 Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants