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

Search build directory first when linking #523

Merged
merged 1 commit into from
Sep 3, 2019

Conversation

ilammy
Copy link
Collaborator

@ilammy ilammy commented Sep 2, 2019

Make sure that we look into our build directory before /usr/local/lib or any other system directories. Developer builds should use newly built libraries even if the system already has Themis installed.

If we look into /usr/local/lib first then developer builds may fail on macOS which has Themis installed to /usr/local/lib by default. This is important for correct dependency information to be recorded in the libraries (especially if the ABI changes one day).

Make sure that we look into our build directory before /usr/local/lib
or any other system directories. Developer builds should use newly
built libraries even if the system already has Themis installed.

If we look into /usr/local/lib first then developer builds may fail
on macOS which has Themis installed to /usr/local/lib by default.
This is important for correct dependency information to be recorded
in the libraries (especially if the ABI changes one day).
@ilammy ilammy added the core Themis Core written in C, its packages label Sep 2, 2019
@vixentael vixentael added the infrastructure Automated building and packaging label Sep 2, 2019
@vixentael
Copy link
Contributor

worked for me!

@ilammy ilammy merged commit da8b52c into cossacklabs:master Sep 3, 2019
@ilammy ilammy deleted the ilammy/macos-dev-build branch September 4, 2019 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Themis Core written in C, its packages infrastructure Automated building and packaging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants