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
net-nds/gssproxy: Fix build with musl llvm #31238
Conversation
Pull Request assignmentSubmitter: @listout net-nds/gssproxy: @gentoo/base-system Linked bugsBugs linked: 898064 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-05-31 05:18 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Small suggestion, but adding the upstreamed commit url to the patch file and adding |
Upstream-Commit: gssapi/gssproxy@f6ab319 Closes: https://bugs.gentoo.org/898064 Signed-off-by: Brahmajit Das <brahmajit.xyz@gmail.com>
0a75c6d
to
710ba06
Compare
done 👍 |
Pull request CI reportReport generated at: 2023-05-31 13:48 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
ping! upstream is commit is merged cc: @juippis @thesamesam can you take a look, thanks in advance. |
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 confirm this patch works with clang:16 and/or musl+clang:16. I'm not eligible to merge @gentoo/base-system packages though.
Closes: https://bugs.gentoo.org/898064