View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0013261CMakeCMakepublic2012-05-30 13:432012-10-01 13:23
ReporterCharlie Sharpsteen 
Assigned ToDavid Cole 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformAppleOSOS XOS Version
Product VersionCMake 2.8.8 
Target VersionCMake 2.8.9Fixed in VersionCMake 2.8.9 
Summary0013261: Default value of CMAKE_FIND_FRAMEWORK can not be overridden from the command line
Description`CMAKE_FIND_FRAMEWORK` always has a value of `FIRST` even if a user passes an argument such as `-DCMAKE_FIND_FRAMEWORK=LAST` when configuring a project. This is due to an unguarded default in the platform file `Darwin.cmake` that always overrides user input.

See the following thread on the mailing list for more details:

    http://cmake.3232098.n2.nabble.com/Why-does-CMAKE-FIND-FRAMEWORK-always-default-to-FIRST-ignoring-command-line-definitions-tp7521759.html [^]


Patch attached which adds guards to the default in `Darwin.cmake`. No changes in the CMake testsuite were noticed after applying the patch.
TagsNo tags attached.
Attached Filespatch file icon 0001-Add-guards-to-CMAKE_FIND_FRAMEWORK-default.patch [^] (1,025 bytes) 2012-05-30 13:43 [Show Content]
? file icon 0001-Add-guards-to-CMAKE_FIND_FRAMEWORK-default.patch-2 [^] (1,662 bytes) 2012-05-30 14:16

 Relationships

  Notes
(0029587)
Brad King (manager)
2012-05-30 13:54

I think Darwin-icc.cmake may need this too.
(0029588)
Charlie Sharpsteen (reporter)
2012-05-30 14:08

Good catch. Unfortunately, I cannot figure out how to delete my original attachement so that I can upload a corrected patch.
(0029589)
Brad King (manager)
2012-05-30 14:10

Re 0013261:0029588: Just add a "-2" to the end of the file name and add a second attachment.
(0029590)
Charlie Sharpsteen (reporter)
2012-05-30 14:19

Done. Although it is a bummer that the bugtracker won't let me edit my own bug---it is rare for humans to "get it perfect the first time".
(0029591)
David Cole (manager)
2012-05-31 08:28

I modified the patch a little bit to remove the verbose endif() and to do the similar thing for the variable 'CMAKE_FIND_APPBUNDLE' as well...

Fix pushed to 'next':

  http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=8b2fb3310b4b8f1d218e479e3f5eafe04e8b8077 [^]
(0031138)
David Cole (manager)
2012-10-01 13:23

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

 Issue History
Date Modified Username Field Change
2012-05-30 13:43 Charlie Sharpsteen New Issue
2012-05-30 13:43 Charlie Sharpsteen File Added: 0001-Add-guards-to-CMAKE_FIND_FRAMEWORK-default.patch
2012-05-30 13:54 Brad King Assigned To => David Cole
2012-05-30 13:54 Brad King Status new => assigned
2012-05-30 13:54 Brad King Note Added: 0029587
2012-05-30 14:08 Charlie Sharpsteen Note Added: 0029588
2012-05-30 14:10 Brad King Note Added: 0029589
2012-05-30 14:16 Charlie Sharpsteen File Added: 0001-Add-guards-to-CMAKE_FIND_FRAMEWORK-default.patch-2
2012-05-30 14:19 Charlie Sharpsteen Note Added: 0029590
2012-05-31 08:27 David Cole Target Version => CMake 2.8.9
2012-05-31 08:28 David Cole Note Added: 0029591
2012-05-31 08:28 David Cole Status assigned => resolved
2012-05-31 08:28 David Cole Fixed in Version => CMake 2.8.9
2012-05-31 08:28 David Cole Resolution open => fixed
2012-10-01 13:23 David Cole Note Added: 0031138
2012-10-01 13:23 David Cole Status resolved => closed


Copyright © 2000 - 2018 MantisBT Team