reasonml / ReasonNativeProject

Reason native compilation starter project
MIT License
203 stars 45 forks source link

Syntax error on the ReasonML code #37

Closed alinpopa closed 6 years ago

alinpopa commented 6 years ago

Trying to build this simply by running make build, and I'm getting this:

% make build
jbuilder build
       refmt bin/test.re.ml (exit 1)
(cd _build/default && /Users/alin/.opam/4.02.3.reason/bin/refmt --print binary bin/test.re) > _build/default/bin/test.re.ml
File "bin/test.re", line 4, characters 13-23:
Error: 2146: syntax error, consider adding a `;' before
make: *** [build] Error 1

This is my current opam setup:

# Installed packages for 4.02.3.reason:
base-bigarray                  base  Bigarray library distributed with the OCaml compiler
base-bytes                     base  Bytes library distributed with the OCaml compiler
base-ocamlbuild                base  OCamlbuild binary and libraries distributed with the OCaml compiler
base-threads                   base  Threads library distributed with the OCaml compiler
base-unix                      base  Unix library distributed with the OCaml compiler
biniou                        1.2.0  Binary data format designed for speed, safety, ease of use and backward compatibility as protocols evolve
camomile                      0.8.7  A comprehensive Unicode library
conf-m4                           1  Virtual package relying on m4
conf-which                        1  Virtual package relying on which
cppo                          1.6.0  Equivalent of the C preprocessor for OCaml programs
easy-format                   1.3.1  High-level and functional interface to the Format module of the OCaml standard library
jbuilder                 1.0+beta16  Fast, portable and opinionated build system
lambda-term                  1.12.0  Terminal manipulation library for OCaml
lwt                           3.2.1  Promises, concurrency, and parallelized I/O
lwt_react                     1.1.0  Helpers for using React with Lwt
menhir                     20171013  LR(1) parser generator
merlin                        3.0.5  Editor helper, provides completion, typing and source browsing in Vim and Emacs
merlin-extend                   0.3  A protocol to provide custom frontend to Merlin
ocaml-migrate-parsetree       1.0.7  Convert OCaml parsetrees between different versions
ocamlbuild                        0  Build system distributed with the OCaml compiler since OCaml 3.10.0
ocamlfind                   1.7.3-1  A library manager for OCaml
ppx_tools_versioned             5.1  A variant of ppx_tools based on ocaml-migrate-parsetree
re                            1.7.1  RE is a regular expression library for OCaml
react                         1.2.1  Declarative events and signals for OCaml
reason                        3.0.4  Reason: Syntax & Toolchain for OCaml
result                          1.2  Compatibility Result module
topkg                         0.9.1  The transitory OCaml software packager
utop                          2.0.2  Universal toplevel for OCaml
yojson                        1.4.0  Yojson is an optimized parsing and printing library for the JSON format
zed                             1.6  Abstract engine for text edition in OCaml

Am I missing something?

jordwalke commented 6 years ago

Updated the project and README. Please let me know if you see any other problems.

alinpopa commented 6 years ago

@jordwalke are you sure it's opam add ..., and not opam install ...? opam 1.2.2 doesn't have the add command, don't know about opam 2.0, tho.

jordwalke commented 6 years ago

Ah, my mistake. I will update the instructions for opam as yoI suggested (I use the esy workflow so I forgot the correct opam command)