Eric,

This commit is breaking windows installs:

commit 6a521f8604ee4e6a757109e731a36fdc5575f6c8
Author: Eric NOULARD <[email protected]>
Date:   Mon Aug 23 17:38:33 2010 +0200

    CPack   Enable better handling of absolute installed files

    The idea of the patch is to let the install generator define
    CPACK_ABSOLUTE_INSTALL_FILES then when CMake is installing
    project he will concatenate the list of files and give
    it to specific CPack Generator by defining CPACK_ABSOLUTE_INSTALL_FILES
to be the list of ALL files that were installed using absolute destination.
    An example of use has been applied to RPM generator which now
    tries to automatically build a relocatable package.

It creates a cmake_install file like this:

IF(NOT CMAKE_INSTALL_COMPONENT OR "${CMAKE_INSTALL_COMPONENT}" STREQUAL "Unspecified")
  list(APPEND CPACK_ABSOLUTE_DESTINATION_FILES
   "/usr/local\CTest.c")
FILE(INSTALL DESTINATION "/usr/local" TYPE FILE FILES "C:/Users/hoffman/Max/tm/src/CTest.c") ENDIF(NOT CMAKE_INSTALL_COMPONENT OR "${CMAKE_INSTALL_COMPONENT}" STREQUAL "Unspecified")

Which causes this error:

1>  -- Install configuration: "Debug"
1>  CMake Warning (dev) at cmake_install.cmake:31 (list):
1>    Syntax error in cmake code at
1>
1>      C:/work-c/T/CMake/cmake_install.cmake:32
1>
1>    when parsing string
1>
1>      /usr/local\CTest.conf


Why do you convert the / to an output path?

6a521f86 (Eric NOULARD 2010-08-23 17:38:33 +0200 71) os << dest << cmSystemTools::ConvertToOutputPath("/"); 6a521f86 (Eric NOULARD 2010-08-23 17:38:33 +0200 72) if (rename && *rename)

All paths in the cmake_install file should be cmake paths as cmake is going to consume them.


All that said, I am not sure why no test failed from this ????

-Bill

_______________________________________________
cmake-developers mailing list
[email protected]
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Reply via email to