search

reasonml / ReasonNativeProject

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

Always use -use-ocamlfind in package.json #10

Closed yunxing closed 7 years ago

yunxing commented 7 years ago

-use-ocamlfind has better support with ppx, we should enable it by default.

@vjeux

jordwalke commented 7 years ago

I also wish there was an environment var called $DEPENDENCYPKGFLAGS so that you could change the build command to:

"reasonBuild": "rebuild $DEPENDENCYPKGFLAGS -use-ocamlfind -I src ./src/Test.native",

Instead of having to find out what each of the -pkg flags are.

It's not clear that's even possible, but something like an esy plugin could do it.

vjeux commented 7 years ago

Feel free to commit directly btw :)