ci: final round of APK CI fixes
authorPaul Spooren <mail@aparcar.org>
Sat, 17 Aug 2024 14:37:04 +0000 (16:37 +0200)
committerRosen Penev <rosenp@gmail.com>
Sun, 18 Aug 2024 03:26:59 +0000 (20:26 -0700)
Using -ne wasn't such a good idea since it would do a numeric compare.
Now use -n which checks if the length is non-zero. APK prints the
problematic version if - problematic - so this should now do the trick.

Signed-off-by: Paul Spooren <mail@aparcar.org>
.github/workflows/check-apk-valid-version.yml

index c16ca9cce0564e05cb848ae1d61f8a2bf2ae534b..def88c8648f96cb8091fc70afc6273d16d30e4c7 100644 (file)
@@ -6,7 +6,7 @@ on:
 
 jobs:
   build:
-    name: Check autorelease deprecation
+    name: Check APK compatible version/release
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
@@ -48,15 +48,15 @@ jobs:
                 PKG_RELEASE=$(grep -E '^PKG_RELEASE' "$ROOT/Makefile" | cut -f 2 -d '=')
                 if [ -n "$PKG_RELEASE" ]; then
                   if [[ "$PKG_RELEASE" == '^[0-9]+$' ]]; then
-                    echo "PKG_RELEASE is not an integer"
+                    echo "PKG_RELEASE is not an integer: $PKG_RELEASE"
                     INCOMPATIBLE_VERSION+=" $ROOT"
                     break
                   fi
                 fi
                 PKG_VERSION=$(grep -E '^PKG_VERSION' "$ROOT/Makefile" | cut -f 2 -d '=')
                 if [ -n "$PKG_VERSION" ]; then
-                  if [[ $($GITHUB_WORKSPACE/apk version --check "$PKG_VERSION") -ne "" ]]; then
-                    echo "PKG_VERSION is not compatible"
+                  if [[ -n $($GITHUB_WORKSPACE/apk version --check "$PKG_VERSION") ]]; then
+                    echo "PKG_VERSION is not compatible: $PKG_VERSION"
                     INCOMPATIBLE_VERSION+=" $ROOT"
                   fi
                 fi