From e9045ece5d0a4c37ae9a4b3b30007a116602e913 Mon Sep 17 00:00:00 2001
From: derekduan <derekduan@tencent.com>
Date: Fri, 14 Jun 2024 18:29:25 +0800
Subject: [PATCH] Fix set version logic error.

The current set version check shows that the test logic of srcroot is
reversed. When srcroot is not empty, the `test "${srcroot}"` itself is
true and should continue to be executed backwards.
---
 configure.ac | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure.ac b/configure.ac
index ff493e1d..14a84a61 100644
--- a/configure.ac
+++ b/configure.ac
@@ -622,7 +622,7 @@ AC_ARG_WITH([version],
     fi
   ], [
     dnl Set VERSION if source directory is inside a git repository.
-    if test "x`test ! \"${srcroot}\" && cd \"${srcroot}\"; git rev-parse --is-inside-work-tree 2>/dev/null`" = "xtrue" ; then
+    if test "x`test \"${srcroot}\" && cd \"${srcroot}\"; git rev-parse --is-inside-work-tree 2>/dev/null`" = "xtrue" ; then
       dnl Pattern globs aren't powerful enough to match both single- and
       dnl double-digit version numbers, so iterate over patterns to support up
       dnl to version 99.99.99 without any accidental matches.
@@ -631,7 +631,7 @@ AC_ARG_WITH([version],
                      '[0-9][0-9].[0-9].[0-9]' '[0-9][0-9].[0-9].[0-9][0-9]' \
                      '[0-9][0-9].[0-9][0-9].[0-9]' \
                      '[0-9][0-9].[0-9][0-9].[0-9][0-9]']; do
-        (test ! "${srcroot}" && cd "${srcroot}"; git describe --long --abbrev=40 --match="${pattern}") > "${objroot}VERSION.tmp" 2>/dev/null
+        (test "${srcroot}" && cd "${srcroot}"; git describe --long --abbrev=40 --match="${pattern}") > "${objroot}VERSION.tmp" 2>/dev/null
         if test $? -eq 0 ; then
           mv "${objroot}VERSION.tmp" "${objroot}VERSION"
           break