-
Notifications
You must be signed in to change notification settings - Fork 31
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
Conversation
Signed-off-by: CrazyMax <[email protected]>
Signed-off-by: CrazyMax <[email protected]>
8e1864c
to
0ca9013
Compare
Signed-off-by: CrazyMax <[email protected]>
0ca9013
to
19fe5ec
Compare
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 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use the https://github.com/alpinelinux/aports/releases/tag/v20230901 tag
There was a problem hiding this comment.
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.
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