ocaml-obuild / obuild

simple package build system for ocaml
BSD 2-Clause "Simplified" License
55 stars 20 forks source link

not every platform has ocamlopt #173

Closed dougmencken closed 6 years ago

dougmencken commented 6 years ago

suddenly obuild has no support for bytecode-only platforms this commit is the first step to have such support

dougmencken commented 6 years ago

In future, some may unite these two scripts into universally working one

UnixJunkie commented 6 years ago

Hi,

What do you mean by "suddenly obuild has no support for bytecode-only platforms"?

Did it have in the past?

If so, can you locate the commit that broke that (using git bisect)?

Why a new file bootstrap-bytecode?

Can't you just patch the current bootstrap file?

Then, it would be way easier for us to review your proposal.

dougmencken commented 6 years ago

@UnixJunkie sorry, I haven’t tracked obuild from the very beginning of its appearance, but looks like it never had support for bytecode-only platforms since its birth/creation, “suddenly” means “too surprisingly”, "absolutely unexpected", “weird and impossible thing”

I would think about merging of these two by myself

UnixJunkie commented 6 years ago

I guess @andyli's proposal might be accepted soon. Thanks for your contribution!