Remove CMake-language block-end command arguments

Ancient versions of CMake required else(), endif(), and similar block
termination commands to have arguments matching the command starting the
block.  This is no longer the preferred style.

Run the following shell code:

for c in else endif endforeach endfunction endmacro endwhile; do
    echo 's/\b'"$c"'\(\s*\)(.\+)/'"$c"'\1()/'
done >convert.sed &&
git ls-files -z -- bootstrap '*.cmake' '*.cmake.in' '*CMakeLists.txt' |
egrep -z -v '^(Utilities/cm|Source/kwsys/)' |
egrep -z -v 'Tests/CMakeTests/While-Endwhile-' |
xargs -0 sed -i -f convert.sed &&
rm convert.sed

Author: kwrobot

CMakeCPack.cmake

@@ -23,7 +23,7 @@ if(EXISTS "${CMAKE_ROOT}/Modules/CPack.cmake")
     endif()
 
     include(${CMake_SOURCE_DIR}/Modules/InstallRequiredSystemLibraries.cmake)
-  endif(EXISTS "${CMAKE_ROOT}/Modules/InstallRequiredSystemLibraries.cmake")
+  endif()
 
   set(CPACK_PACKAGE_DESCRIPTION_SUMMARY "CMake is a build tool")
   set(CPACK_PACKAGE_VENDOR "Kitware")
@@ -56,37 +56,37 @@ if(EXISTS "${CMAKE_ROOT}/Modules/CPack.cmake")
     # cygwin is good for the system name
     if("${CMAKE_SYSTEM_NAME}" STREQUAL "CYGWIN")
       set(CPACK_SYSTEM_NAME Cygwin)
-    else("${CMAKE_SYSTEM_NAME}" STREQUAL "CYGWIN")
+    else()
       set(CPACK_SYSTEM_NAME ${CMAKE_SYSTEM_NAME}-${CMAKE_SYSTEM_PROCESSOR})
-    endif("${CMAKE_SYSTEM_NAME}" STREQUAL "CYGWIN")
-  endif(NOT DEFINED CPACK_SYSTEM_NAME)
+    endif()
+  endif()
   if(${CPACK_SYSTEM_NAME} MATCHES Windows)
     if(CMAKE_CL_64)
       set(CPACK_SYSTEM_NAME win64-x64)
-    else(CMAKE_CL_64)
+    else()
       set(CPACK_SYSTEM_NAME win32-x86)
-    endif(CMAKE_CL_64)
-  endif(${CPACK_SYSTEM_NAME} MATCHES Windows)
+    endif()
+  endif()
 
   if(NOT DEFINED CPACK_PACKAGE_FILE_NAME)
     # if the CPACK_PACKAGE_FILE_NAME is not defined by the cache
     # default to source package - system, on cygwin system is not
     # needed
     if(CYGWIN)
       set(CPACK_PACKAGE_FILE_NAME "${CPACK_SOURCE_PACKAGE_FILE_NAME}")
-    else(CYGWIN)
+    else()
       set(CPACK_PACKAGE_FILE_NAME
         "${CPACK_SOURCE_PACKAGE_FILE_NAME}-${CPACK_SYSTEM_NAME}")
-    endif(CYGWIN)
-  endif(NOT DEFINED CPACK_PACKAGE_FILE_NAME)
+    endif()
+  endif()
 
   set(CPACK_PACKAGE_CONTACT "[email protected]")
 
   if(UNIX)
     set(CPACK_STRIP_FILES "bin/ccmake;bin/cmake;bin/cpack;bin/ctest")
     set(CPACK_SOURCE_STRIP_FILES "")
     set(CPACK_PACKAGE_EXECUTABLES "ccmake" "CMake")
-  endif(UNIX)
+  endif()
 
   # cygwin specific packaging stuff
   if(CYGWIN)
@@ -110,7 +110,7 @@ if(EXISTS "${CMAKE_ROOT}/Modules/CPack.cmake")
     # configures some files and adds some install targets
     # this file uses some of the package file name variables
     include(Utilities/Release/Cygwin/CMakeLists.txt)
-  endif(CYGWIN)
+  endif()
 
   # Set the options file that needs to be included inside CMakeCPackOptions.cmake
   set(QT_DIALOG_CPACK_OPTIONS_FILE ${CMake_BINARY_DIR}/Source/QtDialog/QtDialogCPack.cmake)
@@ -120,4 +120,4 @@ if(EXISTS "${CMAKE_ROOT}/Modules/CPack.cmake")
 
   # include CPack model once all variables are set
   include(CPack)
-endif(EXISTS "${CMAKE_ROOT}/Modules/CPack.cmake")
+endif()

CMakeCPackOptions.cmake.in

@@ -32,7 +32,7 @@ if(CPACK_GENERATOR MATCHES "NSIS")
   set(CPACK_NSIS_URL_INFO_ABOUT "http://www.kitware.com")
   set(CPACK_NSIS_CONTACT @CPACK_PACKAGE_CONTACT@)
   set(CPACK_NSIS_MODIFY_PATH ON)
-endif(CPACK_GENERATOR MATCHES "NSIS")
+endif()
 
 # include the cpack options for qt dialog if they exisit
 # they might not if qt was not enabled for the build
@@ -42,12 +42,12 @@ if(CPACK_GENERATOR MATCHES "CygwinSource")
   # when packaging source make sure the .build directory is not included
     set(CPACK_SOURCE_IGNORE_FILES
       "/CVS/" "/\\.build/" "/\\.svn/" "\\.swp$" "\\.#" "/#" "~$")
-endif(CPACK_GENERATOR MATCHES "CygwinSource")
+endif()
 
 if("${CPACK_GENERATOR}" STREQUAL "PackageMaker")
   if(CMAKE_PACKAGE_QTGUI)
     set(CPACK_PACKAGE_DEFAULT_LOCATION "/Applications")
-  else(CMAKE_PACKAGE_QTGUI)
+  else()
     set(CPACK_PACKAGE_DEFAULT_LOCATION "/usr")
-  endif(CMAKE_PACKAGE_QTGUI)
-endif("${CPACK_GENERATOR}" STREQUAL "PackageMaker")
+  endif()
+endif()

CMakeLists.txt

@@ -67,7 +67,7 @@ macro(CMAKE_HANDLE_SYSTEM_LIBRARIES)
     else()
       set(CMAKE_USE_SYSTEM_LIBRARY_${util} OFF)
     endif()
-  endforeach(util)
+  endforeach()
   if(CMAKE_BOOTSTRAP)
     unset(CMAKE_USE_SYSTEM_LIBRARIES CACHE)
   endif()
@@ -87,23 +87,23 @@ macro(CMAKE_HANDLE_SYSTEM_LIBRARIES)
   foreach(util ${UTILITIES})
     if(CMAKE_USE_SYSTEM_${util})
       message(STATUS "Using system-installed ${util}")
-    endif(CMAKE_USE_SYSTEM_${util})
-  endforeach(util)
+    endif()
+  endforeach()
 
   # Inform utility library header wrappers whether to use system versions.
   configure_file(${CMake_SOURCE_DIR}/Utilities/cmThirdParty.h.in
     ${CMake_BINARY_DIR}/Utilities/cmThirdParty.h
     @ONLY)
 
-endmacro(CMAKE_HANDLE_SYSTEM_LIBRARIES)
+endmacro()
 
 
 
 
 set(CMAKE_BUILD_ON_VISUAL_STUDIO 0)
 if(WIN32 AND NOT UNIX AND NOT BORLAND AND NOT MINGW )
   set(CMAKE_BUILD_ON_VISUAL_STUDIO 1)
-endif(WIN32 AND NOT UNIX AND NOT BORLAND AND NOT MINGW )
+endif()
 
 
 #-----------------------------------------------------------------------
@@ -113,7 +113,7 @@ endif(WIN32 AND NOT UNIX AND NOT BORLAND AND NOT MINGW )
 macro(CMAKE_SETUP_TESTING)
   if (NOT DART_ROOT)
     set(MAKEPROGRAM ${CMAKE_MAKE_PROGRAM})
-  endif (NOT DART_ROOT)
+  endif ()
 
   if(BUILD_TESTING)
     set(CMAKE_TEST_GENERATOR "" CACHE STRING
@@ -123,28 +123,28 @@ macro(CMAKE_SETUP_TESTING)
     if(NOT CMAKE_TEST_GENERATOR)
       set(CMAKE_TEST_GENERATOR "${CMAKE_GENERATOR}")
       set(CMAKE_TEST_MAKEPROGRAM "${MAKEPROGRAM}")
-    else(NOT CMAKE_TEST_GENERATOR)
+    else()
       set(CMAKE_TEST_DIFFERENT_GENERATOR TRUE)
-    endif(NOT CMAKE_TEST_GENERATOR)
+    endif()
 
     # Are we testing with the MSVC compiler?
     set(CMAKE_TEST_MSVC 0)
     if(MSVC AND NOT CMAKE_TEST_DIFFERENT_GENERATOR)
       set(CMAKE_TEST_MSVC 1)
-    else(MSVC AND NOT CMAKE_TEST_DIFFERENT_GENERATOR)
+    else()
       if("${CMAKE_TEST_GENERATOR}" MATCHES "NMake" OR
           "${CMAKE_TEST_GENERATOR}" MATCHES "Visual Studio")
         set(CMAKE_TEST_MSVC 1)
       endif("${CMAKE_TEST_GENERATOR}" MATCHES "NMake" OR
         "${CMAKE_TEST_GENERATOR}" MATCHES "Visual Studio")
-    endif(MSVC AND NOT CMAKE_TEST_DIFFERENT_GENERATOR)
+    endif()
 
     set(CMAKE_TEST_SYSTEM_LIBRARIES 0)
     foreach(util CURL EXPAT XMLRPC ZLIB)
       if(CMAKE_USE_SYSTEM_${util})
         set(CMAKE_TEST_SYSTEM_LIBRARIES 1)
-      endif(CMAKE_USE_SYSTEM_${util})
-    endforeach(util)
+      endif()
+    endforeach()
 
     # This variable is set by cmake, however to
     # test cmake we want to make sure that
@@ -154,7 +154,7 @@ macro(CMAKE_SETUP_TESTING)
     set(CMAKE_CTEST_COMMAND "${CMake_BIN_DIR}/ctest")
     set(CMAKE_CMAKE_COMMAND "${CMake_BIN_DIR}/cmake")
     set(CMAKE_CPACK_COMMAND "${CMake_BIN_DIR}/cpack")
-  endif(BUILD_TESTING)
+  endif()
 
   # configure some files for testing
   configure_file("${CMAKE_CURRENT_SOURCE_DIR}/Templates/CTestScript.cmake.in"
@@ -171,10 +171,10 @@ macro(CMAKE_SETUP_TESTING)
   if(BUILD_TESTING AND DART_ROOT)
     configure_file(${CMake_SOURCE_DIR}/CMakeLogo.gif
       ${CMake_BINARY_DIR}/Testing/HTML/TestingResults/Icons/Logo.gif COPYONLY)
-  endif(BUILD_TESTING AND DART_ROOT)
+  endif()
   mark_as_advanced(DART_ROOT)
   mark_as_advanced(CURL_TESTING)
-endmacro(CMAKE_SETUP_TESTING)
+endmacro()
 
 
 # Provide a way for Visual Studio Express users to turn OFF the new FOLDER
@@ -205,7 +205,7 @@ macro(CMAKE_SET_TARGET_FOLDER tgt folder)
   else()
     set_property(GLOBAL PROPERTY USE_FOLDERS OFF)
   endif()
-endmacro(CMAKE_SET_TARGET_FOLDER)
+endmacro()
 
 
 #-----------------------------------------------------------------------
@@ -266,15 +266,15 @@ macro (CMAKE_BUILD_UTILITIES)
     if(NOT ZLIB_FOUND)
       message(FATAL_ERROR
         "CMAKE_USE_SYSTEM_ZLIB is ON but a zlib is not found!")
-    endif(NOT ZLIB_FOUND)
+    endif()
     set(CMAKE_ZLIB_INCLUDES ${ZLIB_INCLUDE_DIR})
     set(CMAKE_ZLIB_LIBRARIES ${ZLIB_LIBRARIES})
-  else(CMAKE_USE_SYSTEM_ZLIB)
+  else()
     set(CMAKE_ZLIB_INCLUDES ${CMake_SOURCE_DIR}/Utilities)
     set(CMAKE_ZLIB_LIBRARIES cmzlib)
     add_subdirectory(Utilities/cmzlib)
     CMAKE_SET_TARGET_FOLDER(cmzlib "Utilities/3rdParty")
-  endif(CMAKE_USE_SYSTEM_ZLIB)
+  endif()
 
   #---------------------------------------------------------------------
   # Build Curl library for CTest.
@@ -283,26 +283,26 @@ macro (CMAKE_BUILD_UTILITIES)
     if(NOT CURL_FOUND)
       message(FATAL_ERROR
         "CMAKE_USE_SYSTEM_CURL is ON but a curl is not found!")
-    endif(NOT CURL_FOUND)
+    endif()
     set(CMAKE_CURL_INCLUDES ${CURL_INCLUDE_DIRS})
     set(CMAKE_CURL_LIBRARIES ${CURL_LIBRARIES})
-  else(CMAKE_USE_SYSTEM_CURL)
+  else()
     set(CURL_SPECIAL_ZLIB_H ${CMAKE_ZLIB_HEADER})
     set(CURL_SPECIAL_LIBZ_INCLUDES ${CMAKE_ZLIB_INCLUDES})
     set(CURL_SPECIAL_LIBZ ${CMAKE_ZLIB_LIBRARIES})
     option(CMAKE_BUILD_CURL_SHARED "Should curl be built shared" FALSE)
     if(NOT CMAKE_BUILD_CURL_SHARED)
       add_definitions(-DCURL_STATICLIB)
-    endif(NOT CMAKE_BUILD_CURL_SHARED)
+    endif()
     set(CMAKE_CURL_INCLUDES)
     set(CMAKE_CURL_LIBRARIES cmcurl)
     if(CMAKE_TESTS_CDASH_SERVER)
       set(CMAKE_CURL_TEST_URL "${CMAKE_TESTS_CDASH_SERVER}/user.php")
-    endif(CMAKE_TESTS_CDASH_SERVER)
+    endif()
     add_subdirectory(Utilities/cmcurl)
     CMAKE_SET_TARGET_FOLDER(cmcurl "Utilities/3rdParty")
     CMAKE_SET_TARGET_FOLDER(LIBCURL "Utilities/3rdParty")
-  endif(CMAKE_USE_SYSTEM_CURL)
+  endif()
 
   #---------------------------------------------------------------------
   # Build Compress library for CTest.
@@ -334,7 +334,7 @@ macro (CMAKE_BUILD_UTILITIES)
     endif()
     set(CMAKE_TAR_INCLUDES ${LibArchive_INCLUDE_DIRS})
     set(CMAKE_TAR_LIBRARIES ${LibArchive_LIBRARIES})
-  else(CMAKE_USE_SYSTEM_LIBARCHIVE)
+  else()
     set(ZLIB_INCLUDE_DIR ${CMAKE_ZLIB_INCLUDES})
     set(ZLIB_LIBRARY ${CMAKE_ZLIB_LIBRARIES})
     add_definitions(-DLIBARCHIVE_STATIC)
@@ -346,7 +346,7 @@ macro (CMAKE_BUILD_UTILITIES)
     add_subdirectory(Utilities/cmlibarchive)
     CMAKE_SET_TARGET_FOLDER(cmlibarchive "Utilities/3rdParty")
     set(CMAKE_TAR_LIBRARIES cmlibarchive ${BZIP2_LIBRARIES})
-  endif(CMAKE_USE_SYSTEM_LIBARCHIVE)
+  endif()
 
   #---------------------------------------------------------------------
   # Build expat library for CMake and CTest.
@@ -355,15 +355,15 @@ macro (CMAKE_BUILD_UTILITIES)
     if(NOT EXPAT_FOUND)
       message(FATAL_ERROR
         "CMAKE_USE_SYSTEM_EXPAT is ON but a expat is not found!")
-    endif(NOT EXPAT_FOUND)
+    endif()
     set(CMAKE_EXPAT_INCLUDES ${EXPAT_INCLUDE_DIRS})
     set(CMAKE_EXPAT_LIBRARIES ${EXPAT_LIBRARIES})
-  else(CMAKE_USE_SYSTEM_EXPAT)
+  else()
     set(CMAKE_EXPAT_INCLUDES)
     set(CMAKE_EXPAT_LIBRARIES cmexpat)
     add_subdirectory(Utilities/cmexpat)
     CMAKE_SET_TARGET_FOLDER(cmexpat "Utilities/3rdParty")
-  endif(CMAKE_USE_SYSTEM_EXPAT)
+  endif()
 
   #---------------------------------------------------------------------
   # Build XMLRPC library for CMake and CTest.
@@ -372,10 +372,10 @@ macro (CMAKE_BUILD_UTILITIES)
     if(NOT XMLRPC_FOUND)
       message(FATAL_ERROR
         "CTEST_USE_XMLRPC is ON but xmlrpc is not found!")
-    endif(NOT XMLRPC_FOUND)
+    endif()
     set(CMAKE_XMLRPC_INCLUDES ${XMLRPC_INCLUDE_DIRS})
     set(CMAKE_XMLRPC_LIBRARIES ${XMLRPC_LIBRARIES})
-  endif(CTEST_USE_XMLRPC)
+  endif()
 
   #---------------------------------------------------------------------
   # Use curses?
@@ -386,20 +386,20 @@ macro (CMAKE_BUILD_UTILITIES)
       find_package(Curses QUIET)
       if (CURSES_LIBRARY)
         option(BUILD_CursesDialog "Build the CMake Curses Dialog ccmake" ON)
-      else (CURSES_LIBRARY)
+      else ()
         message("Curses libraries were not found. Curses GUI for CMake will not be built.")
         set(BUILD_CursesDialog 0)
-      endif (CURSES_LIBRARY)
-    else(NOT "${CMAKE_SYSTEM_NAME}" MATCHES syllable)
+      endif ()
+    else()
       set(BUILD_CursesDialog 0)
-    endif(NOT "${CMAKE_SYSTEM_NAME}" MATCHES syllable)
-  else (UNIX)
+    endif()
+  else ()
     set(BUILD_CursesDialog 0)
-  endif (UNIX)
+  endif ()
   if(BUILD_CursesDialog)
     add_subdirectory(Source/CursesDialog/form)
-  endif(BUILD_CursesDialog)
-endmacro (CMAKE_BUILD_UTILITIES)
+  endif()
+endmacro ()
 
 #-----------------------------------------------------------------------
 if(CMAKE_CXX_PLATFORM_ID MATCHES "OpenBSD")
@@ -509,7 +509,7 @@ CHECK_SYMBOL_EXISTS(environ "stdlib.h" HAVE_ENVIRON_NOT_REQUIRE_PROTOTYPE)
 #
 if("x${CMAKE_TESTS_CDASH_SERVER}" STREQUAL "x")
   set(CMAKE_TESTS_CDASH_SERVER "http://www.cdash.org/CDash")
-endif("x${CMAKE_TESTS_CDASH_SERVER}" STREQUAL "x")
+endif()
 
 # build the utilities (a macro defined in this file)
 CMAKE_BUILD_UTILITIES()
@@ -524,8 +524,8 @@ if(BUILD_CursesDialog)
   set(CURSES_NEED_RPATH FALSE)
   if(NOT "${_CURSES_DIR}" STREQUAL "/lib" AND NOT "${_CURSES_DIR}" STREQUAL "/usr/lib" AND NOT "${_CURSES_DIR}" STREQUAL "/lib64" AND NOT "${_CURSES_DIR}" STREQUAL "/usr/lib64")
     set(CURSES_NEED_RPATH TRUE)
-  endif(NOT "${_CURSES_DIR}" STREQUAL "/lib" AND NOT "${_CURSES_DIR}" STREQUAL "/usr/lib" AND NOT "${_CURSES_DIR}" STREQUAL "/lib64" AND NOT "${_CURSES_DIR}" STREQUAL "/usr/lib64")
-endif(BUILD_CursesDialog)
+  endif()
+endif()
 
 if(BUILD_QtDialog)
   if(APPLE)
@@ -538,16 +538,16 @@ if(BUILD_QtDialog)
     string(SUBSTRING "${CMAKE_INSTALL_PREFIX}" ${LEN} 1 ENDCH)
     if(NOT "${ENDCH}" STREQUAL "/")
       set(CMAKE_INSTALL_PREFIX "${CMAKE_INSTALL_PREFIX}/")
-    endif(NOT "${ENDCH}" STREQUAL "/")
+    endif()
     set(CMAKE_INSTALL_PREFIX
       "${CMAKE_INSTALL_PREFIX}${CMAKE_BUNDLE_NAME}.app/Contents")
-  endif(APPLE)
+  endif()
 
   set(QT_NEED_RPATH FALSE)
   if(NOT "${QT_LIBRARY_DIR}" STREQUAL "/lib" AND NOT "${QT_LIBRARY_DIR}" STREQUAL "/usr/lib" AND NOT "${QT_LIBRARY_DIR}" STREQUAL "/lib64" AND NOT "${QT_LIBRARY_DIR}" STREQUAL "/usr/lib64")
     set(QT_NEED_RPATH TRUE)
-  endif(NOT "${QT_LIBRARY_DIR}" STREQUAL "/lib" AND NOT "${QT_LIBRARY_DIR}" STREQUAL "/usr/lib" AND NOT "${QT_LIBRARY_DIR}" STREQUAL "/lib64" AND NOT "${QT_LIBRARY_DIR}" STREQUAL "/usr/lib64")
-endif(BUILD_QtDialog)
+  endif()
+endif()
 
 
 # The same might be true on other systems for other libraries.
@@ -563,7 +563,7 @@ if (UNIX)
     set(CMAKE_BUILD_WITH_INSTALL_RPATH TRUE)
   endif(CMAKE_USE_SYSTEM_CURL   OR  CMAKE_USE_SYSTEM_ZLIB
         OR  CMAKE_USE_SYSTEM_EXPAT  OR  CTEST_USE_XMLRPC  OR  CURSES_NEED_RPATH  OR  QT_NEED_RPATH)
-endif (UNIX)
+endif ()
 
 
 # add the uninstall support