Skip to content

Commit

Permalink
Ensure libstdc++ exists on Ruby Alpine images
Browse files Browse the repository at this point in the history
Recent images appear to have removed libstdc++ from some Ruby
versions. Add it to ensure the re2 shared library can be loaded.
  • Loading branch information
stanhu committed Jan 21, 2025
1 parent dafffb6 commit ec2fd57
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -335,7 +335,7 @@ jobs:
docker run --rm -v "$(pwd):/re2" -w /re2 \
--platform=linux/arm64 \
ruby:${{ matrix.ruby }}-alpine \
./scripts/test-gem-install ${{ matrix.rubygems }}
/bin/sh -c "apk update && apk add libstdc++ && ./scripts/test-gem-install ${{ matrix.rubygems }}"
test-precompiled-arm-linux-gnu:
needs: "precompile-arm-linux-gnu"
Expand Down Expand Up @@ -393,7 +393,7 @@ jobs:
docker run --rm -v "$(pwd):/re2" -w /re2 \
--platform=linux/arm/v7 \
ruby:${{ matrix.ruby }}-alpine \
./scripts/test-gem-install ${{ matrix.rubygems }}
/bin/sh -c "apk update && apk add libstdc++ && ./scripts/test-gem-install ${{ matrix.rubygems }}"
test-precompiled-x86-linux-gnu:
needs: "precompile-x86-linux-gnu"
Expand Down Expand Up @@ -451,7 +451,7 @@ jobs:
docker run --rm -v "$(pwd):/re2" -w /re2 \
--platform=linux/386 \
ruby:${{ matrix.ruby }}-alpine \
./scripts/test-gem-install ${{ matrix.rubygems }}
/bin/sh -c "apk update && apk add libstdc++ && ./scripts/test-gem-install ${{ matrix.rubygems }}"
test-precompiled-x86_64-linux-gnu:
needs: "precompile-x86_64-linux-gnu"
Expand Down Expand Up @@ -506,6 +506,7 @@ jobs:
with:
name: cruby-x86_64-linux-musl-gem
path: pkg
- run: apk update && apk add libstdc++
- run: ./scripts/test-gem-install ${{ matrix.rubygems }}

test-precompiled-arm64-darwin:
Expand Down

0 comments on commit ec2fd57

Please sign in to comment.