Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update binutils to 2.41 #127

Merged
merged 3 commits into from
Oct 18, 2023
Merged

Conversation

crazy-max
Copy link
Collaborator

@crazy-max crazy-max commented Oct 17, 2023

closes #121

try a recent version of binutils to check if it fixes segfault when building against riscv64. More info moby/buildkit#4316

edit: looks to fix our issue moby/buildkit#4348

@crazy-max crazy-max force-pushed the binutils-2.41 branch 3 times, most recently from 8e1864c to 0ca9013 Compare October 17, 2023 23:40
Signed-off-by: CrazyMax <[email protected]>
ARG LIBTAPI_VERSION=1100.0.11
ARG CCTOOLS_VERSION=949.0.1-ld64-530
ARG SIGTOOL_VERSION=1dafd2ca4651210ba9acce10d279ace22b50fb01
# BINUTILS_PATCHES_VERSION defines version of aports to use for patches
ARG BINUTILS_PATCHES_VERSION=3.16-stable
ARG BINUTILS_VERSION=2.38
ARG BINUTILS_PATCHES_VERSION=master
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

using master because patches for binutils 2.41 are only on Alpine edge repo.

Copy link
Owner

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes this tag looks good, I'll open a follow-up to pin to it.

@tonistiigi tonistiigi merged commit 20f67db into tonistiigi:master Oct 18, 2023
53 of 56 checks passed
@crazy-max crazy-max deleted the binutils-2.41 branch October 18, 2023 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants