Skip to content

Commit a72d768

Browse files
committed
fix environment variable issues too
1 parent 6889fc3 commit a72d768

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

jscomp/others/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ else
1818
BS_FLAGS= $(BS_COMMON_FLAGS) -bs-package-output lib/js
1919
endif
2020

21-
COMPFLAGS += $(BS_FLAGS) -I ../runtime -I ../stdlib -w -40 -w -49
21+
COMPFLAGS += $(BS_FLAGS) -I ../runtime -I ../stdlib -w -40 -w -49 -bin-annot
2222

2323

2424

jscomp/runtime/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ else
3131
BS_FLAGS= $(BS_COMMON_FLAGS) -bs-package-output lib/js
3232
endif
3333

34-
COMPFLAGS += $(BS_FLAGS) -I ../stdlib -nostdlib -nopervasives -open Pervasives -unsafe -w -40-49
34+
COMPFLAGS += $(BS_FLAGS) -I ../stdlib -nostdlib -nopervasives -open Pervasives -unsafe -w -40-49 -bin-annot
3535

3636

3737
$(RUNTIME): $(COMPILER)

scripts/config.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,9 @@ var map = {
5757
var is_windows = ! (os.type().indexOf('Windows') < 0)
5858

5959
process.env.BS_RELEASE_BUILD = 1
60-
process.env.OCAMLPARAM = '_,bin-annot=1'
60+
delete process.env.OCAMLPARAM // stdlib is already compiled using -bin-annot
61+
// delete process.env.OCAMLLIB // this will not work on Windows
62+
// delete process.env.CAMLLIB
6163
process.env.OCAMLRUNPARAM = 'b'
6264

6365
// return False if it does not exist otherwise the map

0 commit comments

Comments
 (0)