Merge topic 'FindGIFAndFreetypeCleanupAndEnhancements'

81a70c6 Add basic version check for giflib
2462f65 FindGif: add giflib4 as one more name for the library under Windows
f97620e FindGIF/FindFreetype.cmake: remove standard search paths from find-calls
7c38523 Strip trailing whitespace

Modules/FindFreetype.cmake

@@ -44,13 +44,9 @@
 FIND_PATH(FREETYPE_INCLUDE_DIR_ft2build ft2build.h
   HINTS
   $ENV{FREETYPE_DIR}
-  PATH_SUFFIXES include
   PATHS
   /usr/local/X11R6/include
   /usr/local/X11/include
-  /usr/X11/include
-  /sw/include
-  /opt/local/include
   /usr/freeware/include
 )
 
@@ -60,9 +56,6 @@ FIND_PATH(FREETYPE_INCLUDE_DIR_freetype2 freetype/config/ftheader.h
   PATHS
   /usr/local/X11R6/include
   /usr/local/X11/include
-  /usr/X11/include
-  /sw/include
-  /opt/local/include
   /usr/freeware/include
   PATH_SUFFIXES freetype2
 )
@@ -75,8 +68,6 @@ FIND_LIBRARY(FREETYPE_LIBRARY
   PATHS
   /usr/local/X11R6
   /usr/local/X11
-  /usr/X11
-  /sw
   /usr/freeware
 )
 

Modules/FindGIF.cmake

@@ -1,7 +1,11 @@
-# This module defines
+# This module searches giflib and defines
 # GIF_LIBRARIES - libraries to link to in order to use GIF
 # GIF_FOUND, if false, do not try to link
 # GIF_INCLUDE_DIR, where to find the headers
+# GIF_VERSION, reports either version 4 or 3 (for everything before version 4)
+#
+# The minimum required version of giflib can be specified using the
+# standard syntax, e.g. FIND_PACKAGE(GIF 4)
 #
 # $GIF_DIR is an environment variable that would
 # correspond to the ./configure --prefix=$GIF_DIR
@@ -28,14 +32,11 @@ FIND_PATH(GIF_INCLUDE_DIR gif_lib.h
   PATH_SUFFIXES include
   PATHS
   ~/Library/Frameworks
-  /Library/Frameworks
+  /usr/freeware
-  /sw/include # Fink
-  [HKEY_LOCAL_MACHINE\\SYSTEM\\CurrentControlSet\\Control\\Session\ Manager\\Environment;OSG_ROOT]/include
-  /usr/freeware/include
 )
 
 # the gif library can have many names :-/
-SET(POTENTIAL_GIF_LIBS gif libgif ungif libungif giflib)
+SET(POTENTIAL_GIF_LIBS gif libgif ungif libungif giflib giflib4)
 
 FIND_LIBRARY(GIF_LIBRARY
   NAMES ${POTENTIAL_GIF_LIBS}
@@ -44,23 +45,36 @@ FIND_LIBRARY(GIF_LIBRARY
   PATH_SUFFIXES lib64 lib
   PATHS
   ~/Library/Frameworks
-  /Library/Frameworks
-  /usr/local
-  /usr
-  /sw
-  /opt/local
-  /opt/csw
-  /opt
-  [HKEY_LOCAL_MACHINE\\SYSTEM\\CurrentControlSet\\Control\\Session\ Manager\\Environment;OSG_ROOT]
   /usr/freeware
 )
 
 # see readme.txt
 SET(GIF_LIBRARIES ${GIF_LIBRARY})
 
+# Very basic version detection.
+# The GIF_LIB_VERSION string in gif_lib.h seems to be unreliable, since it seems
+# to be always " Version 2.0, " in versions 3.x of giflib.
+# In version 4 the member UserData was added to GifFileType, so we check for this
+# one.
+# http://giflib.sourcearchive.com/documentation/4.1.4/files.html
+IF(GIF_INCLUDE_DIR)
+  INCLUDE(CMakePushCheckState)
+  INCLUDE(CheckStructHasMember)
+  CMAKE_PUSH_CHECK_STATE()
+  SET(GIF_VERSION 3)
+  SET(CMAKE_REQUIRED_INCLUDES "${GIF_INCLUDE_DIR}")
+  CHECK_STRUCT_HAS_MEMBER(GifFileType UserData gif_lib.h GIF_GifFileType_UserData )
+  IF(GIF_GifFileType_UserData)
+    SET(GIF_VERSION 4)
+  ENDIF()
+  CMAKE_POP_CHECK_STATE()
+ENDIF()
+
+
 # handle the QUIETLY and REQUIRED arguments and set GIF_FOUND to TRUE if
 # all listed variables are TRUE
 INCLUDE(${CMAKE_CURRENT_LIST_DIR}/FindPackageHandleStandardArgs.cmake)
-FIND_PACKAGE_HANDLE_STANDARD_ARGS(GIF  DEFAULT_MSG  GIF_LIBRARY  GIF_INCLUDE_DIR)
+FIND_PACKAGE_HANDLE_STANDARD_ARGS(GIF  REQUIRED_VARS  GIF_LIBRARY  GIF_INCLUDE_DIR
+                                       VERSION_VAR GIF_VERSION )
 
 MARK_AS_ADVANCED(GIF_INCLUDE_DIR GIF_LIBRARY)