search

lf-lang / reactor-c

A reactor runtime written in C
Other
10 stars 23 forks source link

Fix cmake syntax #440

Closed edwardalee closed 1 month ago

edwardalee commented 1 month ago

The CMake syntax previously given does not parse when NUMBER_OF_WORKERS is not defined (apparently, it tries to parse the whole thing with NUMBER_OF_WORKERS replaced by nothing), at least with cmake version 3.27.6. This fixes that problem.

This also fixes lingua-franca-ref.txt to point to master.

edwardalee commented 1 month ago

Looks good. Is there a specific reason to do the check in cmake and not in the LF validator?

No reason that I know of. But it's interesting that CMake semantics of AND doesn't match that of most programming languages because, apparently, it evaluates its arguments before it parses the expression.