Use sh rather than bash for CI script

So we can run test-gem-install on both regular and Alpine Ruby images, use a sh script rather than relying on bash.
mudge · Dec 16, 2024 · 334d2da · 334d2da
1 parent 5e349fd
commit 334d2da
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -285,7 +285,7 @@ jobs:
           docker run --rm -v "$(pwd):/re2" -w /re2 \
             --platform=linux/arm64 \
             ruby:${{ matrix.ruby }}-alpine \
-            sh -c "gem update --system && apk add bash && ./scripts/test-gem-install"
+            ./scripts/test-gem-install
 
   test-precompiled-arm-linux-gnu:
     needs: "precompile-arm-linux-gnu"
@@ -327,7 +327,7 @@ jobs:
           docker run --rm -v "$(pwd):/re2" -w /re2 \
             --platform=linux/arm/v7 \
             ruby:${{ matrix.ruby }}-alpine \
-            sh -c "gem update --system && apk add bash && ./scripts/test-gem-install"
+            ./scripts/test-gem-install
 
   test-precompiled-x86-linux-gnu:
     needs: "precompile-x86-linux-gnu"
@@ -369,7 +369,7 @@ jobs:
           docker run --rm -v "$(pwd):/re2" -w /re2 \
             --platform=linux/386 \
             ruby:${{ matrix.ruby }}-alpine \
-            sh -c "gem update --system && apk add bash && ./scripts/test-gem-install"
+            ./scripts/test-gem-install
 
   test-precompiled-x86_64-linux-gnu:
     needs: "precompile-x86_64-linux-gnu"
@@ -407,8 +407,6 @@ jobs:
         with:
           name: cruby-x86_64-linux-musl-gem
           path: pkg
-      - run: gem update --system
-      - run: apk add bash
       - run: ./scripts/test-gem-install
 
   test-precompiled-arm64-darwin:

diff --git a/scripts/test-gem-install b/scripts/test-gem-install
@@ -1,4 +1,4 @@
-#!/usr/bin/env bash
+#!/usr/bin/env sh
 
 set -eu