emscripten-core/emscripten

Faulty assertion in node on .mjs files

Aberta

#17.465 aberto em 18 de jul. de 2022

Ver no GitHubGuia do colaborador
 (4 comentários) (0 reação) (0 responsável)C++ (3.519 forks)batch import
JavaScriptgood first bughelp wanted

Métricas do repositório

Stars
 (27.361 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Inspecione a asserção em shell.js nas linhas 196-198 e remova a. Verifique se a remoção não afeta outros usos e teste com arquivos .mjs no Node com sASSERTIONS.
Pilha de tecnologia
javascriptnodejs
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptNode.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80