Boost C++ Libraries: Ticket #2189: install rules broken

Tue, 12 Aug 2008 17:37:36 GMT

I guess what we want is

      # Install this module's headers
      install(DIRECTORY ${Boost_SOURCE_DIR}/boost/${libname}
        DESTINATION ${BOOST_HEADER_DIR}
        COMPONENT ${libname}_headers
        PATTERN "CVS" EXCLUDE
        PATTERN ".svn" EXCLUDE)

Note that ${Boost_SOURCE_DIR} isn't quoted, since cmake generated path variables (i.e. CMAKE_SOURCE_DIR) are escaped proper.

Patch is attached.

attachment set

Tue, 12 Aug 2008 17:38:32 GMT

attachment → BoostCore.cmake.patch

Thu, 11 Sep 2008 10:00:38 GMT

ping

Douglas Gregor — Fri, 12 Sep 2008 19:51:27 GMT

I don't think this is the right patch. The header-installation logic here is meant to apply to a "modularized" Boost, where the headers do reside in include/boost. It's true that this will fail with a non-modularized Boost, but this isn't the fix for that: in that case, we shouldn't create ANY install rule for the library, because the install(DIRECTORY) rule for the Boost headers in the top-level CMakeLists.txt will handle it.

Sat, 13 Sep 2008 07:21:37 GMT

OK.

So.. why have the include/boost folders been removed from the svn-repo? Just to make clear that this work has do be done?

Douglas Gregor — Sat, 13 Sep 2008 12:11:50 GMT

If you run "make modularize", CMake will move headers into the appropriate places for a modularized Boost.

Sat, 13 Sep 2008 13:48:53 GMT

Thank you. So what we really want is to make sure target modularize being done before make install.

status changed; resolution set

troy d. straszheim — Fri, 20 Nov 2009 23:06:49 GMT

status new → closed
resolution → fixed

modularization temporarily disabled