build: Don't write site_config.hpp if it hasn't changed.

Author: memmett

CMakeLists.txt

@@ -25,7 +25,7 @@ option(pfasst_DEFAULT_RAND_SEED "Using a hardcoded random seed"
 
 if(${pfasst_WITH_MPI})
     find_package(MPI REQUIRED)
-    if(NOT "${CMAKE_CXX_COMPILER}" STREQUAL "${MPI_CXX_COMPILER}" OR 
+    if(NOT "${CMAKE_CXX_COMPILER}" STREQUAL "${MPI_CXX_COMPILER}" OR
        NOT "${CMAKE_C_COMPILER}" STREQUAL "${MPI_C_COMPILER}")
         message(STATUS "C++ Compiler: ${CMAKE_CXX_COMPILER}")
         message(STATUS "MPI C++ Compiler Wrapper: ${MPI_CXX_COMPILER}")
@@ -156,7 +156,7 @@ message(STATUS "Configuring sources")
 update_site_config()
 
 add_custom_target(set_version ALL
-    COMMAND ${pfasst_SOURCE_DIR}/tools/get_pfasst_version.sh
+    COMMAND ${pfasst_SOURCE_DIR}/tools/get_pfasst_version.py
     WORKING_DIRECTORY ${pfasst_SOURCE_DIR}
     COMMENT "Updating PFASST++ version number"
     USES_TERMINAL

cmake/site_config.hpp.in

@@ -12,7 +12,7 @@ using namespace std;
 
 namespace pfasst
 {
-  static constexpr const char* VERSION = "pfasst_VERSION";
+  static constexpr const char* VERSION = "@pfasst_VERSION@";
 }  // ::pfasst
 
 #endif  // _PFASST__SITE_CONFIG_HPP

cmake/utility_functions.cmake

@@ -33,6 +33,9 @@ macro(update_site_config)
     if(pfasst_RANDOM_SEED)
         add_definitions(-DPFASST_DEFAULT_RANDOM_SEED)
     endif()
+    execute_process(COMMAND git describe --dirty
+                    OUTPUT_VARIABLE pfasst_VERSION
+                    OUTPUT_STRIP_TRAILING_WHITESPACE)
     configure_file(
         "${pfasst_SOURCE_DIR}/cmake/site_config.hpp.in"
         "${CMAKE_CURRENT_SOURCE_DIR}/include/pfasst/site_config.hpp"

tools/get_pfasst_version.py

@@ -0,0 +1,27 @@
+#!/usr/bin/env python
+#
+#
+#
+
+import re
+import subprocess
+
+from os.path import dirname, abspath, join
+
+# get git version
+version = str(subprocess.check_output(['git', 'describe', '--dirty'])).strip()
+
+# read in site_config.hpp
+base = dirname(dirname(abspath(__file__)))
+site_config = abspath(join(base, 'include', 'pfasst', 'site_config.hpp'))
+
+with open(site_config, 'r') as f:
+    config = f.read()
+
+# reset version
+new_config = re.sub('VERSION = "[^"]*"', 'VERSION = "{version}"'.format(version=version), config)
+
+# write site_config if it has changed
+if config != new_config:
+    with open(site_config, 'w') as f:
+        f.write(new_config)