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
sys-boot/refind: fix compilation with clang #32710
Conversation
Pull request CI reportReport generated at: 2023-09-10 14:10 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
5f85f77
to
836d1e1
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.
I would always try to perserve the PR message details inside the commit message (although your commit message isn't bad, except for missing the Bug: tags).
Pull request CI reportReport generated at: 2023-09-10 14:35 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
What should I do about the bugs? Should I close them as WONTFIX or FIXED? Because in the end it cannot be built with lld or with llvm-objcopy, I only check them and try binutils. |
I'd say |
836d1e1
to
5e2be34
Compare
Pull Request assignmentSubmitter: @ceamac sys-boot/refind: @ceamac, @sveyret, @gentoo/proxy-maint Linked bugsBugs linked: 732256, 832018, 881131 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
Pull request CI reportReport generated at: 2023-09-10 18:40 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Compiling with clang works. Linking with lld produces broken executables (including the drivers). llvm-objcopy does not support the output format needed by refind. Try to force ld.bfd and binutils objcopy. Fail otherwise. Bug: https://bugs.gentoo.org/732256 Bug: https://bugs.gentoo.org/832018 Bug: https://bugs.gentoo.org/881131 Signed-off-by: Viorel Munteanu <ceamac@gentoo.org>
5e2be34
to
ccf5902
Compare
Pull request CI reportReport generated at: 2023-09-11 05:30 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Related bugs:
llvm-objcopy
, only binutilsobjcopy
works.