Re: [CMake] [New Module] FindEXIF.cmake

2011-01-21 Thread Simone Rossetto
That's good point. Actually on lines 43-45 the variable EXIF_ROOT_DIR is set according to $ENV{EXIF_ROOT_DIR} only if the first header-file of libexif is found in that folder. The same behaviour if the variable EXIF_ROOT_DIR is set with -D option, so if you don't set the env one

Re: [CMake] [New Module] FindEXIF.cmake

2011-01-21 Thread Michael Jackson
The only thing I might add to it would be something like the following: # Only set EXIF_ROOT_DIR to the environment variable if # the user has not specifically set it as a "-D" variable # on the command line or in CMake-GUI as a CMake Variable. if ("${EXIF_ROOT_DIR}" STREQUAL "") SET (EXIF_RO

[CMake] [New Module] FindEXIF.cmake

2011-01-21 Thread Simone Rossetto
Hello to everyone. I've written the new module FindEXIF.cmake that looks for libexif and I'd like to be the maintainer for this module. Attached to this email the FindEXIF.cmake. It works correctly on GNU/Linux and MacOSX, while on Windows it requires the environment variable EXIF_ROOT_DIR to b