emscripten-core/emscripten

Faulty assertion in node on .mjs files

Ouverte

#17 465 ouverte le 18 juil. 2022

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)C++ (3 519 forks)batch import
JavaScriptgood first bughelp wanted

Métriques du dépôt

Stars
 (27 361 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

It turns out that node will disable the global require function on foo.mjs - that is just not defined there, unlike for foo.js. I guess mjs files are modules and require is not supposed to be used there. This ends up a problem for us since we have an assertion on it:

https://github.com/emscripten-core/emscripten/blob/a3d8979b38d2f655f59b040bfd32f562ebc92d03/src/shell.js#L196-L198

Removing that assertion is necessary to run an .mjs file in node with -sASSERTIONS.

We should probably just remove it?

This happens on binaryen CI which will manually builds an .mjs file by concatenating the emcc output with some extra code.

Guide contributeur

Direction de recherche
Inspectez l'assertion dans shell.js lignes 196-198 et supprimez la. Vérifiez que la suppression n'affecte pas d'autres utilisations et testez avec des fichiers .mjs dans Node avec sASSERTIONS.
Stack technique
javascriptnodejs
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80