Skip to content

Use libucontext in 2.7 to fix alpine based builds on arm32v6/7 and s390x#364

Merged
tianon merged 1 commit intodocker-library:masterfrom
infosiftr:arches
Oct 26, 2021
Merged

Use libucontext in 2.7 to fix alpine based builds on arm32v6/7 and s390x#364
tianon merged 1 commit intodocker-library:masterfrom
infosiftr:arches

Conversation

@yosifkit
Copy link
Member

Alpine also uses ucontext on all arches for their Ruby 2.7.

libffi-dev \
libxml2-dev \
libxslt-dev \
libucontext-dev \
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Harmless without LIBS also being set.

checking for getcontext... no
checking for setcontext... no

3.0 and 2.6 build fine without the LIBS variable, so I figured it is likely safer to not include it for them.

@tianon
Copy link
Member

tianon commented Oct 26, 2021

I've updated #360 to hopefully put this sha256sum issue to rest.

@tianon
Copy link
Member

tianon commented Oct 26, 2021

Now that #360 is in, can we rebase? 🙏 😇

@tianon tianon merged commit 4d6eafa into docker-library:master Oct 26, 2021
@tianon tianon deleted the arches branch October 26, 2021 23:18
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Oct 26, 2021
Changes:

- docker-library/ruby@4d6eafa: Merge pull request docker-library/ruby#364 from infosiftr/arches
- docker-library/ruby@53a12c2: Use libucontext in 2.7 to fix alpine based builds on arm32v6/7 and s390x
- docker-library/ruby@2caaf19: Merge pull request docker-library/ruby#360 from docker-library/revert-358-confused
- docker-library/ruby@659f4ab: Remove sha256sum from GitHub patch file
@tianon
Copy link
Member

tianon commented Oct 27, 2021

Heads up for folks who haven't seen it yet -- this broke things (#365), and there's a new patch coming Soon. 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants