ENH: use the standard find_package_handle_standard_args() for lua 5.0 and

5.1

Alex

1 files changed, 13 insertions, 11 deletions

diff --git a/Modules/FindLua51.cmake b/Modules/FindLua51.cmake
index 14a12fc..0a82e8c 100644
--- a/Modules/FindLua51.cmake
+++ b/Modules/FindLua51.cmake
@@ -1,13 +1,13 @@
 # Locate Lua library
 # This module defines
-# LUA_LIBRARIES
-# LUA_FOUND, if false, do not try to link to Lua 
-# LUA_INCLUDE_DIR, where to find lua.h 
+#  LUA_LIBRARIES
+#  LUA_FOUND, if false, do not try to link to Lua 
+#  LUA_INCLUDE_DIR, where to find lua.h 
 #
 # Note that the expected include convention is
-# #include "lua.h"
+#  #include "lua.h"
 # and not
-# #include <lua/lua.h>
+#  #include <lua/lua.h>
 # This is because, the lua location is not standardized and may exist
 # in locations other than lua/
 
@@ -57,16 +57,18 @@ FIND_LIBRARY(LUA_LIBRARY
 IF(LUA_LIBRARY)
   # include the math library for Unix
   IF(UNIX AND NOT APPLE)
-    FIND_LIBRARY(MATH_LIBRARY_FOR_LUA m)
-    SET( LUA_LIBRARIES "${LUA_LIBRARY};${MATH_LIBRARY_FOR_LUA}" CACHE STRING "Lua Libraries")
+    FIND_LIBRARY(LUA_MATH_LIBRARY m)
+    SET( LUA_LIBRARIES "${LUA_LIBRARY};${LUA_MATH_LIBRARY}" CACHE STRING "Lua Libraries")
   # For Windows and Mac, don't need to explicitly include the math library
   ELSE(UNIX AND NOT APPLE)
     SET( LUA_LIBRARIES "${LUA_LIBRARY}" CACHE STRING "Lua Libraries")
   ENDIF(UNIX AND NOT APPLE)
 ENDIF(LUA_LIBRARY)
 
-SET(LUA_FOUND "NO")
-IF(LUA_LIBRARIES AND LUA_INCLUDE_DIR)
-  SET(LUA_FOUND "YES")
-ENDIF(LUA_LIBRARIES AND LUA_INCLUDE_DIR)
+INCLUDE(FindPackageHandleStandardArgs)
+# handle the QUIETLY and REQUIRED arguments and set LUA_FOUND to TRUE if 
+# all listed variables are TRUE
+FIND_PACKAGE_HANDLE_STANDARD_ARGS(Lua50  DEFAULT_MSG  LUA_LIBRARIES LUA_INCLUDE_DIR)
+
+MARK_AS_ADVANCED(LUA_INCLUDE_DIR LUA_LIBRARIES LUA_LIBRARY LUA_MATH_LIBRARY)