nfrechette / rtm

Realtime Math
MIT License
732 stars 44 forks source link

#223: Generic CMake Unix Platform Support #224

Closed bedwardly-down closed 1 month ago

bedwardly-down commented 1 month ago

There's no full need to individually check each platform, so use a generic variable provided by CMake to cover a large scope (until specific ones need to be checked for): https://cmake.org/cmake/help/latest/variable/UNIX.html .

CLAassistant commented 1 month ago

CLA assistant check
All committers have signed the CLA.

nfrechette commented 1 month ago

Great idea, thanks! I'll merge this once CI completes.

nfrechette commented 1 month ago

CI failure is sadly expected. It is the result of a toolchain code generation bug. Thank you for the contribution!