bsc crashes with `Fatal error: exception Not_found` on `Some(x)` when the type of `x` comes from a transitive dependency
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 76/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Bereich
- compilers
Rechercherichtung
Start in compiler/ml/typeopt.ml at type_cannot_contain_undefined and inspect the Env.find_type lookup, then compare the related code from ReScript 11.1.4 and the call path through compiler/ml/env.ml. Reproduce with the provided npm workspace using npm install and npx rescript build. Done means the transitive-dependency example compiles without Fatal error: exception Not_found and preserves the existing successful cases.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Thank you for filing! Check list:
- Is it a bug? Usage questions should often be asked in the forum instead.
- Concise, focused, friendly issue title & description.
- A minimal, reproducible example.
- OS and browser versions, if relevant.
- Is it already fixed in master? Instructions
bsc exits with a bare Fatal error: exception Not_found (exit code 2) when it compiles Some(x) and the type of x is declared in a package that is only a transitive dependency.
The message names no file, so it is hard to find the module that fails.
Snippet
Three packages in npm workspaces: app (the root) depends on facade, and facade depends on lib-a.
package.json:
{
"name": "app",
"private": true,
"workspaces": ["packages/*"],
"dependencies": {"rescript": "12.3.1"}
}
rescript.json:
{
"name": "app",
"sources": [{"dir": "src"}],
"dependencies": ["facade"]
}
src/App.res:
let x = Some(Facade.v)
packages/facade/package.json:
{"name": "facade", "version": "1.0.0", "dependencies": {"lib-a": "1.0.0"}}
packages/facade/rescript.json:
{"name": "facade", "sources": [{"dir": "src"}], "dependencies": ["lib-a"]}
packages/facade/src/Facade.res:
let v = A.v
packages/lib-a/package.json:
{"name": "lib-a", "version": "1.0.0"}
packages/lib-a/rescript.json:
{"name": "lib-a", "sources": [{"dir": "src"}]}
packages/lib-a/src/A.res:
type t = X
let v = X
npm install
npx rescript build
Actual
Cleaned 0/0
Parsed 3 source files
Compiled 3 modules
Fatal error: exception Not_found
Incremental build failed. Error: Failed to Compile. See Errors Above
The same with the bsc command line from npx rescript compiler-args src/App.res (paths made relative) and a backtrace:
cd lib/bs
OCAMLRUNPARAM=b ../../node_modules/@rescript/linux-x64/bin/bsc.exe -I ../ocaml -runtime-path ../../node_modules/@rescript/runtime -I ../../packages/facade/lib/ocaml -bs-package-name app -bs-package-output commonjs:src:.js src/App.ast
Fatal error: exception Not_found
Raised at Env.find_pers_struct in file "compiler/ml/env.ml", line 699, characters 12-27
Called from Env.find_pers_struct in file "compiler/ml/env.ml" (inlined), line 740, characters 28-54
Called from Env.find_module_descr in file "compiler/ml/env.ml", line 787, characters 8-42
Called from Env.find in file "compiler/ml/env.ml", line 804, characters 25-50
Called from Env.find_type in file "compiler/ml/env.ml" (inlined), line 862, characters 26-48
Called from Typeopt.type_cannot_contain_undefined in file "compiler/ml/typeopt.ml", line 51, characters 17-36
Called from Translcore.transl_exp0 in file "compiler/ml/translcore.ml", lines 829-830, characters 19-32
Called from Translcore.transl_let.transl in file "compiler/ml/translcore.ml", line 1081, characters 18-33
Called from Translmod.transl_structure in file "compiler/ml/translmod.ml", line 371, characters 7-56
Called from Translmod.transl_struct in file "compiler/ml/translmod.ml" (inlined), line 288, characters 2-73
Called from Translmod.transl_implementation in file "compiler/ml/translmod.ml", line 463, characters 16-77
Called from Js_implementation.after_parsing_impl in file "compiler/core/js_implementation.ml", line 147, characters 11-72
Called from Dune__exe__Rescript_compiler_main in file "compiler/bsc/rescript_compiler_main.ml", line 447, characters 6-75
Re-raised at Location.report_exception_rec in file "compiler/ml/location.ml", line 282, characters 14-25
Re-raised at Location.report_exception_rec in file "compiler/ml/location.ml", line 282, characters 14-25
Re-raised at Location.report_exception_rec in file "compiler/ml/location.ml", line 282, characters 14-25
Re-raised at Location.report_exception_rec in file "compiler/ml/location.ml", line 282, characters 14-25
Re-raised at Location.report_exception_rec in file "compiler/ml/location.ml", line 282, characters 14-25
Re-raised at Location.report_exception_rec in file "compiler/ml/location.ml", line 282, characters 14-25
Called from Location.report_exception in file "compiler/ml/location.ml" (inlined), line 288, characters 31-61
Called from Dune__exe__Rescript_compiler_main in file "compiler/bsc/rescript_compiler_main.ml", line 452, characters 4-35
With -I ../../packages/lib-a/lib/ocaml added, the same command exits with 0 and emits let x = Facade.v;.
It crashes the same way when A.t is a record, an abstract type, an @unboxed variant, or type t = int.
It also crashes when Facade re-exports a function of A with an optional labeled argument, and App.res passes Facade.v to it: Facade.f(~a=Facade.v) for let f = (~a: option<t>=?) => a.
It compiles when App.res uses Facade.v without Some: let x = Facade.v or let x = [Facade.v].
Expected
App.res compiles, as it did with ReScript 11.1.4 (see below).
Possible cause: in type_cannot_contain_undefined (compiler/ml/typeopt.ml), Env.find_type p env runs before match decl.type_kind with, so the | exception _ -> false case does not catch its Not_found.
In 11.1.4, the lookup was inside the match expression; it moved out in #7458.
Worked in ReScript 11
The same files with ReScript 11.1.4, and bs-dependencies instead of dependencies in both rescript.json files, build without errors.
lib/js/src/App.js:
// Generated by ReScript, PLEASE EDIT WITH CARE
'use strict';
var Facade = require("facade/lib/js/src/Facade.js");
var Caml_option = require("rescript/lib/js/caml_option.js");
var x = Caml_option.some(Facade.v);
exports.x = x;
/* No side effect */
Environment
- ReScript 12.3.1, 13.0.0-alpha.6, and master
58c6c89(from pkg.pr.new, reports13.0.0-alpha.7): the same crash on all three. - Node.js 24.9.0
- Linux x86_64
Context
We hit it in a test file: a package with test helpers re-exports a test library with include (Lib: module type of Lib), and the test passed a value of an @unboxed variant type from that library to an optional labeled argument.
The tested package lists only the helper package in its dependencies.
Workaround
Add the package that declares the type (lib-a) to the dependencies of the package that fails to compile (app).
- Vorherrschende Sprache
- OCaml
- Sterne
- 7.5k
- Forks
- 485
- Ø Merge
- 1 T. 6 Std.
- Gemergte PRs (30 T.)
- 52
Entwicklungsumgebung
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus rescript-lang/rescript
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100
rescript-lang/rescript#8688 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
rescript-lang/rescript#8659 · 2 Kommentare · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
rescript-lang/rescript#8647 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Untagged variants: boolean literal/payload overlap is rejected while string, int, float and bigint overlap is allowedEvtl. vergeben @cknitt hat das vor 24 Tagen übernommen. Offen
rescript-lang/rescript#8632 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 28/100
rescript-lang/rescript#8624 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in rescript-lang/rescript
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
objectionary/jeo-maven-plugin#1827 ·
Maintainer antworten meist innerhalb von 4 Tagen
-
forwarded-to-js-devs
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
backend:DirectX
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
llvm/llvm-project#227530 ·
Maintainer antworten meist innerhalb von 1 Tag
-
`enzymexla.linalg.lu` lowering fails for a tall matrix: the permutation is built with the pivot typeOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
EnzymeAD/Enzyme-JAX#3286 ·
Maintainer antworten meist innerhalb von 1 Tag
-
The unsafe/foreign gate misses an @unsafe dependency that appears only in a datatype's is Kind(..)Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
HigherOrderCO/Bend#1205 ·