-
Notifications
You must be signed in to change notification settings - Fork 32
Conversation
CMakeLists.txt
Outdated
enable_language(ASM) | ||
|
||
# What compilers are we using? | ||
set(CMAKE_C_COMPILER "clang-3.8") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This I think is the one change I'd like to see. Can you see if you can detect if the cmake compiler is not a variant of clang, and error out? Hard-coding clang 3.8 should not be necessary anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 395631b.
b1a6ae8
to
2423d89
Compare
CMakeLists.txt
Outdated
set(CMAKE_C_COMPILER "clang-3.8") | ||
set(CMAKE_CXX_COMPILER "clang++-3.8") | ||
set(CMAKE_ASM_COMPILER "clang-3.8") | ||
if (NOT ${CMAKE_C_COMPILER} MATCHES ".*clang") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd recommend instead trying this test: https://stackoverflow.com/questions/10046114/in-cmake-how-can-i-test-if-the-compiler-is-clang
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 7fb8fe9.
Mostly a straightforward port of the Makefile. I've tested lightly (running
grrshot
on a binary out of cb-multios, compiled for linux/decree, debug/release) and things appear to be correct.