Skip to content

Commit 74f5101

Browse files
committed
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.
1 parent a0ec2d6 commit 74f5101

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

.github/workflows/tests.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -285,7 +285,7 @@ jobs:
285285
docker run --rm -v "$(pwd):/re2" -w /re2 \
286286
--platform=linux/arm64 \
287287
ruby:${{ matrix.ruby }}-alpine \
288-
sh -c "gem update --system && apk add bash && ./scripts/test-gem-install"
288+
./scripts/test-gem-install
289289
290290
test-precompiled-arm-linux-gnu:
291291
needs: "precompile-arm-linux-gnu"
@@ -327,7 +327,7 @@ jobs:
327327
docker run --rm -v "$(pwd):/re2" -w /re2 \
328328
--platform=linux/arm/v7 \
329329
ruby:${{ matrix.ruby }}-alpine \
330-
sh -c "gem update --system && apk add bash && ./scripts/test-gem-install"
330+
./scripts/test-gem-install
331331
332332
test-precompiled-x86-linux-gnu:
333333
needs: "precompile-x86-linux-gnu"
@@ -369,7 +369,7 @@ jobs:
369369
docker run --rm -v "$(pwd):/re2" -w /re2 \
370370
--platform=linux/386 \
371371
ruby:${{ matrix.ruby }}-alpine \
372-
sh -c "gem update --system && apk add bash && ./scripts/test-gem-install"
372+
./scripts/test-gem-install
373373
374374
test-precompiled-x86_64-linux-gnu:
375375
needs: "precompile-x86_64-linux-gnu"
@@ -407,8 +407,6 @@ jobs:
407407
with:
408408
name: cruby-x86_64-linux-musl-gem
409409
path: pkg
410-
- run: gem update --system
411-
- run: apk add bash
412410
- run: ./scripts/test-gem-install
413411

414412
test-precompiled-arm64-darwin:

scripts/test-gem-install

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#!/usr/bin/env bash
1+
#!/usr/bin/env sh
22

33
set -eu
44

0 commit comments

Comments
 (0)