[Bazel] Main bootstrap script for Wasm Audio Worklets not generated about emsdk HOT 3 CLOSED

A PR would definitely be welcome!

What information do you need, that you expected to see in a CONTRIBUTING file? (we can add that too if it would be generally useful)

from emsdk.

What information do you need, that you expected to see in a CONTRIBUTING file? (we can add that too if it would be generally useful)

Just wanted to know if there's anything a newcomer needs to know about the whole process or if there are any requirements to submit a PR (i.e. some repos requires a contributor licence agreement).

Anyways I guess I'll find out as we go 😄

About the issue at hand before attempting a fix I'd like to know more about the rationale of the implementation of this part of the linker_wrapper.py: why is there a set of valid file extensions in the first place? IIUC the outdir should contain the files generated by the compiler, so it seems to me that the implementation could be simplified by putting all files contained in the outdir in the list of files to be added to the files array. Am I missing something here?

Of course the easy way to fix the issue is to just add the .aw.js extension to the set mention before, but on the other hand I think simplifying the logic of the wrapper, if possible, can be more valuable in the long run. Maybe the original author of the rule, @walkingeyerobot, can give me more context.

Thanks!

from emsdk.

Of course PRs are welcome :)

The toolchain is much older than github history would lead you to believe. Originally, we wanted to build existing cc_binary targets under emscripten, so wasm_cc_binary was written as a wrapper rule that simply transitioned the cc toolchain. This means that the cc_binary is still the thing that does the actual building. cc_binary only has exactly one output, whereas emscripten outputs many files. To work around this, we put all emscripten's output files into a tarball, and then after everything is built wasm_cc_binary extracts this tarball and exposes all possible output files. Because bazel isn't capable of optional outputs (or at least wasn't at the time wasm_cc_binary was written), we needed to make sure all the files were created even if they were empty.

I suspect (but I'm not sure) that simplifying or eliminating this tarball process is possible but a huge amount of work. I would recommend just adding the .aw.js output file to the relevant scripts / rules.

from emsdk.