Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't run step7 #5

Closed
yhsiang opened this issue Feb 1, 2017 · 2 comments
Closed

Can't run step7 #5

yhsiang opened this issue Feb 1, 2017 · 2 comments

Comments

@yhsiang
Copy link

yhsiang commented Feb 1, 2017

Hi, everything is fine when running step1-6, but step7 got errors.
Environment:

MacOS 10.12.2
The OCaml compiler, version 4.02.3
node v6.3.1
npm 3.10.3

Error messages:

File "node_modules/reason-js/src/ReasonJs.re", line 15, characters 7-13:
Error: 438: <SYNTAX ERROR>

File "node_modules/reason-js/src/ReasonJs.re", line 1:
Error: Error while running external preprocessor
Command line: refmt 'node_modules/reason-js/src/ReasonJs.re' > /var/folders/qt/qx5f0wb14b923zrz5m2b7jj00000gn/T/ocamlpp4549dd

File "src/test.re", line 16, characters 0-19:
Error: Unbound module ReasonJs
module.js:442
    throw err;
    ^

Error: Cannot find module '/Users/yhsiang/w/appier/intro-to-reason-compilation/step7/_build/self/test.js'
    at Function.Module._resolveFilename (module.js:440:15)
    at Function.Module._load (module.js:388:25)
    at Module.runMain (module.js:575:10)
    at run (bootstrap_node.js:352:7)
    at startup (bootstrap_node.js:144:9)
    at bootstrap_node.js:467:3

Check reasonJs.re Line 15: module Window, did I miss something to upgrade or ?

@chenglou
Copy link
Owner

chenglou commented Feb 3, 2017

Hey @yhsiang, thanks for the notice! Which version of refmt are you using? I've made an update that should theoretically fix your thing. Tell me if it did!

@yhsiang
Copy link
Author

yhsiang commented Feb 3, 2017

Hi @chenglou ,
My original refmt is 1.3.0, now I use 1.7.3 and fixed by this commit b7a2c4b

Thanks!

@yhsiang yhsiang closed this as completed Feb 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants