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
don't use buildmode=pie on ppc64 #42514
Conversation
Why? Does Go not support this build mode on ppc64? |
it does not on big-endian ppc64. |
c4135e3
to
11cb1db
Compare
oops, I rebased too much. will fix soon. |
It's already omitted for ppc64 in hack/dockerfile/install/install.sh not using wildcard, because GOARCH=ppc64le supports pie Signed-off-by: Georgy Yakovlev <gyakovlev@gentoo.org>
11cb1db
to
2cd68d5
Compare
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.
LGTM
CI failure appears to be another instance of #42459 |
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.
LGTM
- What I did
made sure
buildmode=pie
is not passed on ppc64 (big-endian)- How I did it
added exception in
hack/make/.binary
It's already omitted for ppc64 in
hack/dockerfile/install/install.sh
not using wildcard, because GOARCH=ppc64le supports pie
- How to verify it
apply provided patch and #42172
docker now builds and works on big-endian ppc64
- Description for the changelog
don't pass buildmode=pie on ppc64be
- A picture of a cute animal (not mandatory but encouraged)