MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010417CMakeModulespublic2010-03-13 16:362010-11-09 22:57
ReporterClinton Stimpson 
Assigned ToDavid Cole 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionCMake 2.8.3Fixed in VersionCMake 2.8.3 
Summary0010417: BundleUtilities/GetPrerequisites not doing bundleutilities example correctly
DescriptionThere is an example here:
http://www.cmake.org/Wiki/BundleUtilitiesExample [^]

That doesn't work correctly.

There are two problems.

1.
otool -L also returns the install id, if it exists, which means get_prerequistes can return the target itself as a dependency of the target. This causes problems for BundleUtilities.cmake.
The attached patch fixes this.

2.
BundleUtilities.cmake makes the assumptions that plugins have to reside in the same directory as the executable. In the example, the Qt plugins have a special layout.

Attached patch fixes both issues.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
child of 0009744closed David Cole BundleUtilities making extra copies of plugins 
Attached Filespatch bundle_and_gp.patch (2,455) 2010-03-13 16:36
https://public.kitware.com/Bug/file/2968/bundle_and_gp.patch
Issue History
Date ModifiedUsernameFieldChange
2010-03-13 16:36Clinton StimpsonNew Issue
2010-03-13 16:36Clinton StimpsonFile Added: bundle_and_gp.patch
2010-07-13 09:30Bill HoffmanStatusnew => assigned
2010-07-13 09:30Bill HoffmanAssigned To => David Cole
2010-08-05 17:28Clinton StimpsonRelationship addedchild of 0009744
2010-08-12 09:13Mike McQuaidNote Added: 0021730
2010-08-18 00:36David ColeNote Added: 0021790
2010-08-18 04:53Mike McQuaidNote Added: 0021798
2010-08-31 17:58David ColeTarget Version => CMake 2.8.3
2010-09-07 22:27David ColeNote Added: 0022120
2010-09-07 22:27David ColeStatusassigned => resolved
2010-09-07 22:27David ColeResolutionopen => fixed
2010-09-09 23:59David ColeFixed in Version => CMake 2.8.3
2010-11-09 22:57Philip LowmanStatusresolved => closed

Notes
(0021730)
Mike McQuaid   
2010-08-12 09:13   
It would be great to help with this, if I can at all?
(0021790)
David Cole   
2010-08-18 00:36   
Mike, have you tried the attached patch? Works for you? Thanks...
(0021798)
Mike McQuaid   
2010-08-18 04:53   
Sorry, yes, this attached patch works great for me.
(0022120)
David Cole   
2010-09-07 22:27   
Pushed to CMake 'next' just now:
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=e93a4b4d3421ced7b8c852b76c0dcb427f798df8 [^]

Should make it into the upcoming 2.8.3 release.