From c0fc9bc422ab62933d26d724d67ce6298a15fda7 Mon Sep 17 00:00:00 2001
From: Sergey Nikolaev <prostuda@academ.org>
Date: Fri, 17 May 2024 11:26:54 +0700
Subject: [PATCH] Switching to a newer testing image with a fresher php (#61)

---
 .github/workflows/build_template.yml |  2 +-
 .github/workflows/test.yml           |  1 -
 .github/workflows/test_template.yml  |  4 ++--
 cmake/rev.cmake                      | 14 ++++++++------
 4 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/.github/workflows/build_template.yml b/.github/workflows/build_template.yml
index 748e9591..fb438fab 100644
--- a/.github/workflows/build_template.yml
+++ b/.github/workflows/build_template.yml
@@ -10,7 +10,7 @@ on:
       DISTR:
         required: false
         type: string
-        default: bionic
+        default: jammy
       CTEST_CMAKE_GENERATOR:
         required: false
         type: string
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index b3a6be84..67bef495 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -58,7 +58,6 @@ jobs:
         run: |
           wget https://repo.manticoresearch.com/repository/ci/boost_1_75_0.tgz
           tar -xf boost_1_75_0.tgz
-
   build_linux_release:
     name: Linux release build
     uses: ./.github/workflows/build_template.yml
diff --git a/.github/workflows/test_template.yml b/.github/workflows/test_template.yml
index 041fd4ca..38bc6570 100644
--- a/.github/workflows/test_template.yml
+++ b/.github/workflows/test_template.yml
@@ -40,7 +40,7 @@ jobs:
       run:
         shell: bash
     container:
-      image: manticoresearch/ubertests_ctest:3263 #
+      image: manticoresearch/ubertests_ctest:3263_mar_2024
       env:
         DIAGNOSTIC: 1
         CACHEB: ../cache
@@ -62,7 +62,7 @@ jobs:
       - name: Download build artifacts
         uses: manticoresoftware/download_artifact_with_retries@v2
         with:
-          name: build_bionic_${{ inputs.CTEST_CONFIGURATION_TYPE }}_x86_64
+          name: build_jammy_${{ inputs.CTEST_CONFIGURATION_TYPE }}_x86_64
           path: .
 
       - name: Check out main cache before building
diff --git a/cmake/rev.cmake b/cmake/rev.cmake
index cb2138d2..404e60d3 100644
--- a/cmake/rev.cmake
+++ b/cmake/rev.cmake
@@ -11,6 +11,10 @@ function ( guess_from_git )
 		return ()
 	endif ()
 
+	# without this in some environments you can get error "detected dubious ownership in repository"
+	# `git config --global --add safe.directory '*'` in the docker image it runs in may not help. TODO: check why
+  execute_process ( COMMAND "${GIT_EXECUTABLE}" config --global --add safe.directory "${columnar_SOURCE_DIR}")
+
 	# extract short hash as GIT_COMMIT_ID
 	execute_process ( COMMAND "${GIT_EXECUTABLE}" log -1 --format=%h
 			WORKING_DIRECTORY "${columnar_SOURCE_DIR}"
@@ -21,16 +25,14 @@ function ( guess_from_git )
 	set ( GIT_COMMIT_ID "${GIT_COMMIT_ID}" PARENT_SCOPE )
 
 	# extract timestamp and make number YYMMDD from it
-	# it would be --date=format:%y%m%d, but old git on centos doesn't understand it
-	execute_process ( COMMAND "${GIT_EXECUTABLE}" log -1 --date=short --format=%cd
+	execute_process ( COMMAND "${GIT_EXECUTABLE}" log -1 --date=format-local:"%y%m%d%H" --format=%cd
 			WORKING_DIRECTORY "${columnar_SOURCE_DIR}"
 			RESULT_VARIABLE res
 			OUTPUT_VARIABLE GIT_TIMESTAMP_ID
-			ERROR_QUIET
 			OUTPUT_STRIP_TRAILING_WHITESPACE )
-	string ( REPLACE "-" "" GIT_TIMESTAMP_ID "${GIT_TIMESTAMP_ID}" )
-	string ( SUBSTRING "${GIT_TIMESTAMP_ID}" 2 -1 GIT_TIMESTAMP_ID )
-	set ( GIT_TIMESTAMP_ID "${GIT_TIMESTAMP_ID}" PARENT_SCOPE )
+
+	string ( SUBSTRING "${GIT_TIMESTAMP_ID}" 1 8 GIT_TIMESTAMP_ID )
+	set ( GIT_TIMESTAMP_ID ${GIT_TIMESTAMP_ID} PARENT_SCOPE )
 
 	# timestamp for reproducable packages
 	execute_process ( COMMAND "${GIT_EXECUTABLE}" log -1 --pretty=%ct