cmakelists: Fix CMAKE_COMPILER_ID => CMAKE_<lang>_COMPILER_ID#6438
cmakelists: Fix CMAKE_COMPILER_ID => CMAKE_<lang>_COMPILER_ID#6438SirLynix wants to merge 1 commit intoxmake-io:devfrom
Conversation
|
Which one is more correct? #6439 |
|
I think it would be mine because I use CMAKE_C_COMPILER_ID when it's not sure and CMAKE_CXX_LINKER_ID, what do you think @PierreEVEN? |
I was wondering if "C" is better than "CXX". I saw "CXX" everywhere else so I choose to keep it consistent but you may be right. By the way the value for |
#6439 (mine) works as long as CXX is enabled. but it has the same problem as this pr if C is not enabled. I'm fixing it to handle both cases... |
|
Closing because #6439 will have a better fix |
https://cmake.org/cmake/help/latest/variable/CMAKE_LANG_COMPILER_ID.html