Don't create symlinks in the build tree
authorTeemu Murtola <teemu.murtola@gmail.com>
Sun, 8 Dec 2013 05:05:16 +0000 (07:05 +0200)
committerTeemu Murtola <teemu.murtola@gmail.com>
Sun, 8 Dec 2013 05:05:16 +0000 (07:05 +0200)
This requires an update to the regression tests and/or Jenkins.
Either the regression tests must be run against installed code, or they
must be modified to not depend on the symlinks.  But that would be for
the better, as the present approach (that is removed here) is a hack
that doesn't always work, and since we want to get rid of the symlinks
in the long run, it doesn't make sense to try to make the symlinking
more elaborate.

Change-Id: I90859f1a8c15d8ac3090e31f522aae2af7ba4a67

src/programs/CMakeLists.txt

index d3f57884c080de48279f393db170ba317b7a685d..4a68bf512dbecaa30e34c7641353ffc410188a1c 100644 (file)
@@ -78,22 +78,6 @@ else()
 
     configure_file(CreateLinks.cmake.cmakein CreateLinks.cmake @ONLY)
     set(CREATE_LINKS_SCRIPT ${CMAKE_CURRENT_BINARY_DIR}/CreateLinks.cmake)
-    # TODO: Currently this rule is always run and recreates the symlinks;
-    # it would be best to just adapt the rest of the build such that this is
-    # never needed (for now, this makes the binary directory seem like it was
-    # earlier).
-    add_custom_target(links ALL
-        ${CMAKE_COMMAND}
-            -DBINARY_DIRECTORY=${CMAKE_RUNTIME_OUTPUT_DIRECTORY}
-            -DQUIETLY=TRUE
-            -P ${CREATE_LINKS_SCRIPT}
-        DEPENDS gmx
-        COMMENT "Creating symlinks for alternative binary names"
-        VERBATIM)
-
-    # TODO: Create man pages (currently not working since the old mechanism
-    # requires a target for each binary to which one can add a post-build role, and
-    # the symlinks do not have such targets).
 
     install(TARGETS gmx
             RUNTIME DESTINATION ${BIN_INSTALL_DIR})