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
app-text/sablotron: Fix clang16 build #32773
Conversation
|
||
dnl here we collect needed libraries | ||
-OTHER_LIBS= | ||
+OTHER_LIBS="-lsupc++ " |
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.
@thesamesam It this patch actually necessary? (And if it is, it this the correct place to add it?)
If I bind with CC=gcc CXX=gcc
this fails because gcc does not automatically bind with lsupc++.
If I bind with CC=clang CXX=clang
it fails as well.
But CC=clang CXX=clang++
works as well as CC=gcc CXX=g++
Then again, you wrote - Set CC=clang and CXX=clang in make.conf
in your bashrc hook.
Bug: https://bugs.gentoo.org/894710 Signed-off-by: Pascal Jäger <pascal.jaeger@leimstift.de>
283e919
to
be677cf
Compare
Pull Request assignmentSubmitter: @Schievel1 app-text/sablotron: @gentoo/proxy-maint (maintainer needed) Linked bugsBugs linked: 894710 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-14 08:20 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Pull request CI reportReport generated at: 2023-09-14 08:35 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
oh darn #32726 |
hehe, try to pick up other ones from https://bugs.gentoo.org/870412. P.S. There are plenty :) |
I know I know, I’ve been doing a few already |
Bug: https://bugs.gentoo.org/894710