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

fix(bitcode): Remove bitcode from libraries. #2

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

lhunath
Copy link

@lhunath lhunath commented Nov 6, 2024

Boost and curl libraries were compiled in such a way that they included bitcode markers as well as invalid OSO symbols referring to bitcode_strip tmp files that resulted in dsymutil warnings.

These updates remove the bitcode markers from the libs entirely as well as the OSO symbols referencing them.

Boost and curl libraries were compiled in such a way that they included
bitcode markers as well as invalid OSO symbols referring to
bitcode_strip tmp files that resulted in dsymutil warnings.
@lhunath lhunath self-assigned this Nov 6, 2024
@lhunath lhunath requested a review from eakoli November 6, 2024 15:37
Boost/BoostConfig.cmake Outdated Show resolved Hide resolved
CURL/build.sh Outdated Show resolved Hide resolved
Boost/build.sh Outdated Show resolved Hide resolved
@lhunath lhunath requested a review from L0PiTaL November 7, 2024 16:32
@lhunath
Copy link
Author

lhunath commented Nov 13, 2024

Note: This PR updates libcurl to 8.11.0. Before merging, please merge https://github.com/NetsoftHoldings/hubstaff-client/pull/2033 since a code change is required in backend/NetworkAccess.cpp.

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

Successfully merging this pull request may close these issues.

2 participants