[api-major] Output JavaScript modules in the builds (issue 10317)#17055
[api-major] Output JavaScript modules in the builds (issue 10317)#17055timvandermeij merged 3 commits intomozilla:masterfrom
Conversation
Snuffleupagus
force-pushed
the
output-modules
branch
3 times, most recently
from
b856022 to
469789e
Compare
|
Given the errors when loading the previews above, my guess would be that the server running on the bots doesn't understand the Line 26 in be53c7d @calixteman Can you please check if this can be fixed on the bots? |
|
/botio-linux preview |
Snuffleupagus
force-pushed
the
output-modules
branch
2 times, most recently
from
acf25b4 to
9c8232b
Compare
|
So it works but I had to modify |
|
What version of the Thanks for the help so far, since at least it works now which unblocks this PR :-) |
|
It's a dep of |
Snuffleupagus
force-pushed
the
output-modules
branch
2 times, most recently
from
32116ac to
87562a6
Compare
Snuffleupagus
force-pushed
the
output-modules
branch
from
87562a6 to
269a5c3
Compare
Snuffleupagus
force-pushed
the
output-modules
branch
from
378f833 to
0b5f70d
Compare
From: Bot.io (Linux m4)ReceivedCommand cmd_preview from @Snuffleupagus received. Current queue size: 0 |
From: Bot.io (Linux m4)SuccessTotal script time: 1.39 mins Published
|
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 |
From: Bot.io (Linux m4)FailedTotal script time: 25.07 mins
Image differences available at: http://54.241.84.105:8877/44c40b0752466fc/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedTotal script time: 35.80 mins
Image differences available at: http://54.193.163.58:8877/7c6a24eda821954/reftest-analyzer.html#web=eq.log |
Snuffleupagus
force-pushed
the
output-modules
branch
from
0b5f70d to
927e50f
Compare
|
/botio-linux preview |
From: Bot.io (Linux m4)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 |
From: Bot.io (Linux m4)SuccessTotal script time: 1.40 mins Published
|
|
/botio integrationtest |
From: Bot.io (Linux m4)ReceivedCommand cmd_integrationtest from @timvandermeij received. Current queue size: 0 |
From: Bot.io (Windows)ReceivedCommand cmd_integrationtest from @timvandermeij received. Current queue size: 0 |
From: Bot.io (Linux m4)SuccessTotal script time: 5.45 mins
|
From: Bot.io (Windows)SuccessTotal script time: 16.94 mins
|
4 participants
At this point in time all browsers, and also Node.js, support standard
import/exportstatements and we can now finally consider outputting modern JavaScript modules in the builds.[1]In order for this to work we can only use proper
import/exportstatements throughout the main code-base, and (as expected) our Node.js support made this much more complicated since both the official builds and the GitHub Actions-based tests must keep working.[2]One remaining issue is that the
pdf.scripting.jsfile cannot be built as a JavaScript module, since doing so breaks PDF scripting. It's not clear to me if this is a limitation of the QuickJS Javascript Engine itself, or "just" an issue with how the Emscripten compiler was configured.Note that my initial goal was to try and split these changes into a couple of commits, however that unfortunately didn't really work since it turned out to be difficult for smaller patches to work correctly and pass (all) tests that way.[3]
This is a classic case of every change requiring a couple of other changes, with each of those changes requiring further changes in turn and the size/scope quickly increasing as a result.
One possible "issue" with these changes is that we'll now only output JavaScript modules in the builds, which could perhaps be a problem with older tools. However it unfortunately seems far too complicated/time-consuming for us to attempt to support both the old and modern module formats, hence the alternative would be to do "nothing" here and just keep our "old" builds.[4]
[1] The final blocker was module support in workers in Firefox, which was implemented in Firefox 114; please see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import#browser_compatibility
[2] It's probably possible to further improve/simplify especially the Node.js-specific code, but it does appear to work as-is.
[3] Having partially "broken" patches, that fail tests, as part of the commit history is really not a good idea in general.
[4] Outputting JavaScript modules was first requested almost five years ago, see issue #10317, and nowadays there should be much better support for JavaScript modules in various tools.