From: <ai...@us...> - 2013-12-13 19:28:00
|
Revision: 12864 http://sourceforge.net/p/plplot/code/12864 Author: airwin Date: 2013-12-13 19:27:57 +0000 (Fri, 13 Dec 2013) Log Message: ----------- Make filter_rpath function robust against the case where none of CMAKE_C_IMPLICIT_LINK_DIRECTORIES, CMAKE_CXX_IMPLICIT_LINK_DIRECTORIES, or CMAKE_FORTRAN_IMPLICIT_LINK_DIRECTORIES are defined. This should never happen according to the CMake documentation, but Phil apparently found a case where it did happen for the "Visual Studio 11" generator (for VS2012). My conclusion is that generator is severely broken, but nevertheless making this function robust for the case when none of these variables are defined cannot be a bad thing to do. Modified Paths: -------------- trunk/cmake/modules/plplot_functions.cmake Modified: trunk/cmake/modules/plplot_functions.cmake =================================================================== --- trunk/cmake/modules/plplot_functions.cmake 2013-12-13 18:27:43 UTC (rev 12863) +++ trunk/cmake/modules/plplot_functions.cmake 2013-12-13 19:27:57 UTC (rev 12864) @@ -207,8 +207,12 @@ ${CMAKE_CXX_IMPLICIT_LINK_DIRECTORIES} ${CMAKE_FORTRAN_IMPLICIT_LINK_DIRECTORIES} ) - list(REMOVE_DUPLICATES directories_to_be_removed) + if(directories_to_be_removed) + list(REMOVE_DUPLICATES directories_to_be_removed) + endif(directories_to_be_removed) + + if(directories_to_be_removed) list(REMOVE_ITEM internal_rpath ${directories_to_be_removed}) endif(directories_to_be_removed) endif(internal_rpath) This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <ai...@us...> - 2013-12-14 21:23:35
|
Revision: 12865 http://sourceforge.net/p/plplot/code/12865 Author: airwin Date: 2013-12-14 21:23:33 +0000 (Sat, 14 Dec 2013) Log Message: ----------- FORTRAN ==> Fortran. This bug fix should make no difference on Linux where all three C, CXX, and Fortran variables are identical, but it will make a (slight) difference on platforms where Fortran has a different set of system locations than the C and C++ compilers. Modified Paths: -------------- trunk/cmake/modules/plplot_functions.cmake Modified: trunk/cmake/modules/plplot_functions.cmake =================================================================== --- trunk/cmake/modules/plplot_functions.cmake 2013-12-13 19:27:57 UTC (rev 12864) +++ trunk/cmake/modules/plplot_functions.cmake 2013-12-14 21:23:33 UTC (rev 12865) @@ -205,7 +205,7 @@ set(directories_to_be_removed ${CMAKE_C_IMPLICIT_LINK_DIRECTORIES} ${CMAKE_CXX_IMPLICIT_LINK_DIRECTORIES} - ${CMAKE_FORTRAN_IMPLICIT_LINK_DIRECTORIES} + ${CMAKE_Fortran_IMPLICIT_LINK_DIRECTORIES} ) if(directories_to_be_removed) This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <ai...@us...> - 2013-12-18 02:14:30
|
Revision: 12877 http://sourceforge.net/p/plplot/code/12877 Author: airwin Date: 2013-12-18 02:14:25 +0000 (Wed, 18 Dec 2013) Log Message: ----------- Refine the filter_rpath function a little more by removing empty elements from the rpath list. Modified Paths: -------------- trunk/cmake/modules/plplot_functions.cmake Modified: trunk/cmake/modules/plplot_functions.cmake =================================================================== --- trunk/cmake/modules/plplot_functions.cmake 2013-12-17 22:45:28 UTC (rev 12876) +++ trunk/cmake/modules/plplot_functions.cmake 2013-12-18 02:14:25 UTC (rev 12877) @@ -201,7 +201,9 @@ #message("DEBUG: ${rpath} = ${${rpath}}") set(internal_rpath ${${rpath}}) if(internal_rpath) + # Remove duplicates and empty elements. list(REMOVE_DUPLICATES internal_rpath) + list(REMOVE_ITEM internal_rpath "") set(directories_to_be_removed ${CMAKE_C_IMPLICIT_LINK_DIRECTORIES} ${CMAKE_CXX_IMPLICIT_LINK_DIRECTORIES} This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |