View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014686CMakeCMakepublic2014-01-07 19:412014-06-02 08:38
Reporterazdagron 
Assigned ToBrad King 
PrioritynormalSeverityfeatureReproducibilityalways
StatusclosedResolutionfixed 
PlatformWindowsOSWindows 7OS Version6.1.7601
Product VersionCMake 2.8.12 
Target VersionCMake 3.0Fixed in VersionCMake 3.0 
Summary0014686: FindBoost.cmake does not provide a way to use libraries built by boost with the runtime-debugging=off bjam parameter
DescriptionFindBoost.cmake unconditionally adds the "g" portion to the debug ABI tag indicating that it wants the versions of the libraries using the debug runtime. When bjam is provided the runtime-debugging=off command-line option, the release version of the runtime is used instead
Steps To Reproduce1) bjam link=static runtime-link=shared runtime-debugging=off (produces a binary on windows with no "-*g-" in the name, e.g. libboost_date_time-vc110-mt-d-1_55.lib)
2) use FindBoost as follows:
set(Boost_USE_STATIC_LIBS ON)
set(Boost_USE_MULTITHREADED ON)
set(Boost_USE_STATIC_RUNTIME OFF)
find_package(Boost 1.55.0 REQUIRED COMPONENTS date_time)

Additional InformationI propose the inclusion of a Boost_USE_RUNTIME_DEBUGGING option to mirror that provided to bjam with conditionally skips the addition of "g" to the debug ABI tag:

+if(Boost_USE_RUNTIME_DEBUGGING)
  if(WIN32)
    if(MSVC OR "${CMAKE_CXX_COMPILER}" MATCHES "icl"
            OR "${CMAKE_CXX_COMPILER}" MATCHES "icpc")
      set(_boost_DEBUG_ABI_TAG "${_boost_DEBUG_ABI_TAG}g")
    endif()
  endif()
+endif()
TagsNo tags attached.
Attached Files

 Relationships

  Notes
(0034912)
azdagron (reporter)
2014-01-07 19:44

 Oops, i meant "g" instead of "d" in the ABI tag name.
(0034914)
Brad King (manager)
2014-01-08 09:14

 I fixed the description to use "g" instead of "d". Please read back through it to make sure it is now correct.

The proposed

 +if(Boost_USE_RUNTIME_DEBUGGING)

option will need to be on by default to avoid changing existing behavior.
(0034915)
azdagron (reporter)
2014-01-08 11:19

 Looks good, thanks for updating it.

I agree that it should be on-by-default.
(0034916)
Brad King (manager)
2014-01-08 11:35

 I added an option with a name more consistent with existing options:

 FindBoost: Add Boost_USE_DEBUG_RUNTIME option
 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=d729899a [^]

Due to documentation reformatting since 2.8.12 the help text may conflict.
(0036072)
Robert Maynard (manager)
2014-06-02 08:38

 Closing resolved issues that have not been updated in more than 4 months.

 Issue History
Date Modified Username Field Change
2014-01-07 19:41 azdagron New Issue
2014-01-07 19:44 azdagron Note Added: 0034912
2014-01-08 09:12 Brad King Description Updated
2014-01-08 09:12 Brad King Steps to Reproduce Updated
2014-01-08 09:12 Brad King Additional Information Updated
2014-01-08 09:14 Brad King Note Added: 0034914
2014-01-08 11:19 azdagron Note Added: 0034915
2014-01-08 11:35 Brad King Note Added: 0034916
2014-01-09 10:09 Brad King Assigned To => Brad King
2014-01-09 10:09 Brad King Status new => resolved
2014-01-09 10:09 Brad King Resolution open => fixed
2014-01-09 10:09 Brad King Fixed in Version => CMake 3.0
2014-01-09 10:09 Brad King Target Version => CMake 3.0
2014-06-02 08:38 Robert Maynard Note Added: 0036072
2014-06-02 08:38 Robert Maynard Status resolved => closed

Copyright © 2000 - 2018 MantisBT Team