diff options
| author | Brad King <brad.king@kitware.com> | 2020-01-14 13:55:36 +0000 |
|---|---|---|
| committer | Kitware Robot <kwrobot@kitware.com> | 2020-01-14 08:55:58 -0500 |
| commit | 11b6a315afb3c69642aba6f8fb93a7bdddf4141a (patch) | |
| tree | 69c35e823e000da52bc2e3a49711e220d716aabb /Modules | |
| parent | 3617e26d8c962528f0656242b69e2e9f6cd55372 (diff) | |
| parent | b704d58f41def99de41485af364d9c7ea46ed475 (diff) | |
| download | cmake-11b6a315afb3c69642aba6f8fb93a7bdddf4141a.tar.gz | |
Merge topic 'FindMatlab-only-CXX'
b704d58f41 FindMatlab: in matlab_add_mex use the correct version file
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Patrik Huber <patrikhuber@gmail.com>
Merge-request: !4206
Diffstat (limited to 'Modules')
| -rw-r--r-- | Modules/FindMatlab.cmake | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/Modules/FindMatlab.cmake b/Modules/FindMatlab.cmake index c8cae2e131..ca56dd51ee 100644 --- a/Modules/FindMatlab.cmake +++ b/Modules/FindMatlab.cmake @@ -974,10 +974,19 @@ function(matlab_add_mex) endif() if(NOT Matlab_VERSION_STRING VERSION_LESS "9.1") # For 9.1 (R2016b) and newer, add version source file - # TODO: check the file extensions in ${${prefix}_SRC} to see if they're C or C++ files - # Currently, the C and C++ versions of the version files are identical, so this doesn't matter. - set(MEX_VERSION_FILE "${Matlab_ROOT_DIR}/extern/version/c_mexapi_version.c") - #set(MEX_VERSION_FILE "${Matlab_ROOT_DIR}/extern/version/cpp_mexapi_version.cpp") + # Add the correct version file depending on which languages are enabled in the project + if(CMAKE_C_COMPILER_LOADED) + # If C is enabled, use the .c file as it will work fine also with C++ + set(MEX_VERSION_FILE "${Matlab_ROOT_DIR}/extern/version/c_mexapi_version.c") + elseif(CMAKE_CXX_COMPILER_LOADED) + # If C is not enabled, check if CXX is enabled and use the .cpp file + # to avoid that the .c file is silently ignored + set(MEX_VERSION_FILE "${Matlab_ROOT_DIR}/extern/version/cpp_mexapi_version.cpp") + else() + # If neither C or CXX is enabled, warn because we cannot add the source. + # TODO: add support for fortran mex files + message(WARNING "[MATLAB] matlab_add_mex requires that at least C or CXX are enabled languages") + endif() endif() if(NOT Matlab_VERSION_STRING VERSION_LESS "9.4") # For 9.4 (R2018a) and newer, add API macro |