Skip to content
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

Consider Changing -fPIC to PRIVATE in CMakeLists.txt #5188

Open
souvik1997 opened this issue Mar 13, 2025 · 2 comments · May be fixed by #5189
Open

Consider Changing -fPIC to PRIVATE in CMakeLists.txt #5188

souvik1997 opened this issue Mar 13, 2025 · 2 comments · May be fixed by #5189

Comments

@souvik1997
Copy link

Hi,

I noticed that in the CMake configuration, the -fPIC compile option is added as a PUBLIC option:

target_compile_options(${PROJECT_NAME} PUBLIC -fPIC)

Would it be possible to change this to PRIVATE? Making it PUBLIC propagates -fPIC to all targets linking s2n, which might be unnecessary or even problematic for some consumers of the library.

@souvik1997
Copy link
Author

#5189

@boquan-fang boquan-fang linked a pull request Mar 13, 2025 that will close this issue
@boquan-fang
Copy link
Contributor

Hi Souvik,

Thanks for cutting this issue to us. I have seen your PR: #5189.

Reviewers are already assigned for this PR. Please continue to work with @camshaft and @lrstewart to resolve this issue.

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants