View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006400CMakeModulespublic2008-02-21 00:382008-07-23 02:53
ReporterPhilip Lowman 
Assigned ToMiguel Figueroa 
PrioritynormalSeverityfeatureReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0006400: FindMagick.cmake Module
DescriptionAttached is a FindMagick.cmake module designed for searching for the Magick and Magick++ development libraries (often distributed with ImageMagick).

I wasn't sure if this should be merged with FindImageMagick.cmake since that module is focused entirely on finding the utility programs distributed with ImageMagick. Most of the time a user is probably going to want to find either one or the other for his build and not both.

In the event that FindMagick.cmake is checked in, it might not be a bad idea to add a note to FindImageMagick.cmake about it.

Module uses FIND_PACKAGE_HANDLE_STANDARD_ARGS() and as far as I now compiles with CMake module submission criteria (if not please let me know what's wrong).
TagsNo tags attached.
Attached Files? file icon FindMagick.cmake [^] (1,385 bytes) 2008-02-21 00:38
? file icon IMPROVED_FindMagick.cmake [^] (1,523 bytes) 2008-02-21 20:40
? file icon FindImageMagick.cmake [^] (7,505 bytes) 2008-07-23 02:23

 Relationships
has duplicate 0004220closedMiguel Figueroa FindImageMagick.cmake should also find ImageMagick's libraries 

  Notes
(0010624)
Alex Neundorf (developer)
2008-02-21 14:29

 Miguel has already a related bug report assigned.

Alex
(0010625)
Philip Lowman (developer)
2008-02-21 20:40

 I investigated Bug 0004220 which was a FindMagick.cmake submitted on 12/16/06.

I examined this module to determine what is done differently:

1. They allow the user to specify a plethora of environment variables to find ImageMagick. This seems silly given the existence of CMAKE_PREFIX_PATH, CMAKE_LIBRARY_PATH and CMAKE_INCLUDE_PATH.

2. They provide multiple include paths which is not needed at least with modern versions of the libraries.

3. They search for the Wand library and expose it via cache to the user, which was something I wasn't doing. I have added this functionality to the attached IMPROVED_FindMagick.cmake (could not delete original).

I suggest that Bug 0004220 be closed.
(0011777)
Philip Lowman (developer)
2008-05-09 13:56

 Any possibility of getting this included with CMake and closing both this issue and Bug 0004220?
(0012805)
Miguel Figueroa (developer)
2008-07-23 02:52

 I've applied a patch to CVS:

/cvsroot/CMake/CMake/Modules/FindImageMagick.cmake,v <-- FindImageMagick.cmake
new revision: 1.5; previous revision: 1.4

Please test that it works as expected.

--Miguel
(0012806)
Miguel Figueroa (developer)
2008-07-23 02:53

 Applied patch:

/cvsroot/CMake/CMake/Modules/FindImageMagick.cmake,v <-- FindImageMagick.cmake
new revision: 1.5; previous revision: 1.4

--Miguel

 Issue History
Date Modified Username Field Change
2008-02-21 00:38 Philip Lowman New Issue
2008-02-21 00:38 Philip Lowman File Added: FindMagick.cmake
2008-02-21 12:11 Alex Neundorf Status new => assigned
2008-02-21 12:11 Alex Neundorf Assigned To => Alex Neundorf
2008-02-21 14:29 Alex Neundorf Note Added: 0010624
2008-02-21 14:29 Alex Neundorf Assigned To Alex Neundorf => Miguel Figueroa
2008-02-21 20:40 Philip Lowman File Added: IMPROVED_FindMagick.cmake
2008-02-21 20:40 Philip Lowman Note Added: 0010625
2008-05-09 13:56 Philip Lowman Note Added: 0011777
2008-07-23 02:23 Miguel Figueroa File Added: FindImageMagick.cmake
2008-07-23 02:27 Miguel Figueroa Note Added: 0012803
2008-07-23 02:36 Miguel Figueroa Relationship added has duplicate 0004220
2008-07-23 02:51 Miguel Figueroa Note Deleted: 0012803
2008-07-23 02:52 Miguel Figueroa Note Added: 0012805
2008-07-23 02:53 Miguel Figueroa Status assigned => closed
2008-07-23 02:53 Miguel Figueroa Note Added: 0012806
2008-07-23 02:53 Miguel Figueroa Resolution open => fixed

Copyright © 2000 - 2018 MantisBT Team