remove erroneous else conditions to make what is going on clearer, add a comment
authornemo
Thu, 24 Oct 2019 09:41:10 -0400
changeset 15503 58ce582ae87d
parent 15502 9f98eece9841
child 15504 4cc9ec732392
remove erroneous else conditions to make what is going on clearer, add a comment
cmake_modules/paths.cmake
--- a/cmake_modules/paths.cmake	Wed Oct 23 17:45:28 2019 -0400
+++ b/cmake_modules/paths.cmake	Thu Oct 24 09:41:10 2019 -0400
@@ -61,7 +61,11 @@
     #install_name_tool for libraries
     set(CMAKE_BUILD_WITH_INSTALL_NAME_DIR TRUE)
     set(CMAKE_INSTALL_NAME_DIR "@executable_path/../Frameworks")
-else(APPLE AND NOT (${CMAKE_INSTALL_PREFIX} MATCHES "/usr"))
+# should this be a separate if block like so
+#if(NOT APPLE AND NOT (${CMAKE_INSTALL_PREFIX} MATCHES "/usr"))
+# there were some conditions here that implied not setting the RPATH if installed to /usr 
+# but it was not being applied due to else not actually taking parameters  (HT wuzzy)
+else()
     #paths where to find libraries (final slash not optional):
     # - the first is relative to the executable
     # - the second is the same directory of the executable (so it runs in bin/)