Support configurable `ocaml-flags`

bsansouci / bsb-native

Build system for OCaml/Reason

Other

245 stars 10 forks source link

Support configurable `ocaml-flags` #8

Closed anmonteiro closed 6 years ago

anmonteiro commented 6 years ago

This is probably incomplete, but I'm opening the PR anyways to get early feedback.

TODO / questions:

[x] Choose sane defaults for the flags, I'm thinking just -thread -no-alias-deps.
[x] Make flags additive to the defaults
[ ] Update the README to reflect the new option

bsansouci commented 6 years ago

Let's go with -thread -no-alias-deps for the defaults.'

Let's go with additive for now, i think it's a better and simpler experience for everyone. I don't see many reasons to disable -no-alias-deps. For -threads maybe you'd want to optimize the release binary but we'll figure that one out later.

Thanks!

bsansouci commented 6 years ago

Thanks!