ksherlock / mpw

Macintosh Programmer's Workshop (mpw) compatibility layer
238 stars 21 forks source link

CMakeLists.txt doesn't check for ragel #48

Open ryandesign opened 1 year ago

ryandesign commented 1 year ago

If ragel is not installed, running cmake succeeds but running make fails with:

/bin/sh: ragel: command not found

Although ragel is mentioned as a requirement in mpw's readme, it would be an improvement if CMakeLists.txt checked for ragel and any other dependencies.

ksherlock commented 1 year ago

The CMakeList has been updated to check for ragel and give an error if not found. the only other odd dependency is lemon and that only runs if it's found.

843ea2764c582cc2112e47478b1b79b3f29a8c1d

ryandesign commented 1 year ago

The CMakeList has been updated to check for ragel and give an error if not found.

Thanks, that works.

the only other odd dependency is lemon and that only runs if it's found.

Nitpick: If ragel is now always run quoted as "${RAGEL}", should lemon always be run quoted as "${LEMON}"?

Is there a way to tell the build system not to run lemon, even if it is found? This is important for reproducibility.