chore: Refactor runastyle scripts to support future runners (pipx, uvx)

This refactor introduces a dedicated `ASTYLE_VERSION` variable (formatted as
`X.Y.Z`) and derives `ASTYLE_VERSION_STR` automatically. This separation makes
it clearer where to update the required version and ensures consistency across
platforms.

The change simplifies maintenance and prepares the scripts for use with command
runners such as `pipx` or `uvx`.

Author: jcfr

runastyle

@@ -8,15 +8,17 @@ SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 
 cd $SCRIPT_DIR
 
-# If project management wishes to take a newer astyle version into use
-# just change this string to match the start of astyle version string.
-ASTYLE_VERSION="Artistic Style Version 3.0.1"
+# To require a newer astyle version, update ASTYLE_VERSION below.
+# ASTYLE_VERSION_STR is then constructed to match the beginning of the
+# version string reported by "astyle --version".
+ASTYLE_VERSION="3.0.1"
+ASTYLE_VERSION_STR="Artistic Style Version ${ASTYLE_VERSION}"
 ASTYLE="astyle"
 
-DETECTED_VERSION=`$ASTYLE --version 2>&1`
-if [[ "$DETECTED_VERSION" != ${ASTYLE_VERSION}* ]]; then
-    echo "You should use: ${ASTYLE_VERSION}";
-    echo "Detected: ${DETECTED_VERSION}"
+DETECTED_VERSION_STR=`$ASTYLE --version 2>&1`
+if [[ "$DETECTED_VERSION_STR" != ${ASTYLE_VERSION_STR}* ]]; then
+    echo "You should use: ${ASTYLE_VERSION_STR}";
+    echo "Detected: ${DETECTED_VERSION_STR}"
     exit 1;
 fi
 

runastyle.bat

@@ -10,18 +10,20 @@ SET SCRIPT_DIR=%~dp0
 @REM Change to that directory
 CD /d %SCRIPT_DIR%
 
-@REM If project management wishes to take a newer astyle version into use
-@REM just change this string to match the start of astyle version string.
-@SET ASTYLE_VERSION="Artistic Style Version 3.0.1"
+@REM To require a newer astyle version, update ASTYLE_VERSION below.
+@REM ASTYLE_VERSION_STR is then constructed to match the beginning of the
+@REM version string reported by "astyle --version".
+@SET ASTYLE_VERSION="3.0.1"
+@SET ASTYLE_VERSION_STR="Artistic Style Version %ASTYLE_VERSION%"
 @SET ASTYLE="astyle"
 
-@SET DETECTED_VERSION=""
-@FOR /F "tokens=*" %%i IN ('%ASTYLE% --version') DO SET DETECTED_VERSION=%%i
-@ECHO %DETECTED_VERSION% | FINDSTR /B /C:%ASTYLE_VERSION% > nul && (
-    ECHO "%DETECTED_VERSION%" matches %ASTYLE_VERSION%
+@SET DETECTED_VERSION_STR=""
+@FOR /F "tokens=*" %%i IN ('%ASTYLE% --version') DO SET DETECTED_VERSION_STR=%%i
+@ECHO %DETECTED_VERSION_STR% | FINDSTR /B /C:%ASTYLE_VERSION_STR% > nul && (
+    ECHO "%DETECTED_VERSION_STR%" matches %ASTYLE_VERSION_STR%
 ) || (
-    ECHO You should use: %ASTYLE_VERSION%
-    ECHO Detected: "%DETECTED_VERSION%"
+    ECHO You should use: %ASTYLE_VERSION_STR%
+    ECHO Detected: "%DETECTED_VERSION_STR%"
     GOTO EXIT_ERROR
 )