Hi,

On 19-09-2024 18:49, Julian Gilbey wrote:
Ah!  That would work.  But I've now found the source of the problem,
and have a suggestion for how to fix it.  Shall I commit directly to
the emscripten salsa repo?

emscripten is QA maintained, so if you feel like you can responsibly change the package, please go ahead.

I'd like to note that I also believe that the current version of emscripten also need stuff from llvm-toolchain-19 instead of 18. The autopkgtest failure of node-inwasm shows that the current version of emscripten uses a too new option --no-growable-memory

98s emcc: warning: LLVM version for clang executable "/usr/bin/clang-18" appears incorrect (seeing "18.1", expected "20") [-Wversion-check]
 98s wasm-ld-18: error: unknown argument: --no-growable-memory

Fixing the autopkgtest of emscripten to run in test instead of in tests and rebuilding [1] with 18 replaced by 19 in d/control shows that's not enough:
"""
emcc: warning: LLVM version for clang executable "/usr/bin/clang-19" appears incorrect (seeing "19.1", expected "20") [-Wversion-check] error: undefined symbol: _emscripten_tempret_set (referenced by $setTempRet0, referenced by setTempRet0, referenced by root reference (e.g. compiled C/C++ code)) warning: To disable errors for undefined symbols use `-sERROR_ON_UNDEFINED_SYMBOLS=0` warning: __emscripten_tempret_set may need to be added to EXPORTED_FUNCTIONS if it arrives from a system library
Error: Aborting compilation due to previous errors
emcc: error: '/usr/bin/node /usr/share/emscripten/src/compiler.mjs /tmp/tmpinds__jv.json' failed (returned 1)
"""

Paul

[1] https://debomatic-amd64.debian.net/distribution#unstable/emscripten/3.1.66~dfsg-3~debo1/autopkgtest

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature

Reply via email to