build: Use ccomp_type to determine C compiler

janestreet / bin_prot

Binary protocol generator

MIT License

73 stars 21 forks source link

build: Use ccomp_type to determine C compiler #7

Closed seliopou closed 9 years ago

seliopou commented 9 years ago

This library did not build properly on FreeBSD because of the hard-coded C compiler in myocamlbuild.ml. The file now checks ccomp_type to determine the C compiler to use, which should make the library more portable.

Tested on FreeBSD 10.0.

Closes #6

ghost commented 9 years ago

I'm not sure ccomp_type is supposed to be a command. It happens to be always cc on Unix but I wouldn't rely on that. I'd rather use cc directly, or bytecomp_c_compiler.

seliopou commented 9 years ago

@diml Sorry for the delay, just pushed the change.