Skip to content

Switch to hidden symbol visibility as default #3687

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

Open
pmatilai opened this issue Mar 26, 2025 · 0 comments
Open

Switch to hidden symbol visibility as default #3687

pmatilai opened this issue Mar 26, 2025 · 0 comments
Labels
API API related cleanup

Comments

@pmatilai
Copy link
Member

pmatilai commented Mar 26, 2025

C++ symbol visibility is a different beast from that of C as I found out in #3679. We need to move to hidden visibility as default, and explicit exports for exporting symbols to the ABI.

We should've really done that a long ago already, but that got mixed up with symbol versioning (#1127) which is a more intrusive change and requires a soname bump to go with it. Changing symbol visibility doesn't, as long as the exported symbols just stay the same.

This doesn't affect the desire to introduce symbol versioning later on.

@pmatilai pmatilai added this to RPM Mar 26, 2025
@github-project-automation github-project-automation bot moved this to Backlog in RPM Mar 26, 2025
@pmatilai pmatilai added API API related cleanup labels Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API API related cleanup
Projects
Status: Backlog
Development

No branches or pull requests

1 participant