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

Conflicting conan dependencies #450

Closed
hplatou opened this issue Oct 30, 2019 · 1 comment · Fixed by #452
Closed

Conflicting conan dependencies #450

hplatou opened this issue Oct 30, 2019 · 1 comment · Fixed by #452
Assignees
Labels
bug Something isn't working conan This issue or PR is related to the conan package manager urgent blocking issue

Comments

@hplatou
Copy link
Contributor

hplatou commented Oct 30, 2019

As conan is moving away from using user/channel the package libzip/1.5.2 has been updated with new dependency on zlib/1.2.11 instead of zlib/1.2.11@conan/stable. This gives the following error when building on cse-core (with clean cache or -u):

ERROR: Conflict in libzip/1.5.2@bincrafters/stable
    Requirement zlib/1.2.11 conflicts with already defined zlib/1.2.11@conan/stable
    To change it, override it in your base requirements

Fixing this is easy, buy changing the dependency in conanfile.py to 'zlib/1.2.11'. Doing so reveals another problem. Now thrift are having problems with OpenSSL. This is because OpenSSL has been migrated from conan-community to conan-center and in that process also been renamed from OpenSSL to openssl. This causes errors when building on Windows:

ERROR: Failed requirement 'OpenSSL/1.0.2r@conan/stable' from 'thrift/0.12.0@bincrafters/stable'
ERROR: Requested 'OpenSSL/1.0.2r@conan/stable' but found case incompatible 'openssl'
Case insensitive filesystem can't manage this

I have filed a PR on conan-thrift (bincrafters/community#1016), and it has already been merged, but only on the testing branch and that is not deployed to any repo.
I will build these packages manually for Windows and Linux and deploy to our osp-conan repo.
Then I will file a PR with updated dependencies. This will likely have to be merged into every existing branch to make the builds pass on Jenkins.

@hplatou hplatou added bug Something isn't working urgent blocking issue conan This issue or PR is related to the conan package manager labels Oct 30, 2019
@hplatou hplatou self-assigned this Oct 30, 2019
@hplatou hplatou changed the title Fix Conflicting conan dependencies Oct 30, 2019
@markaren
Copy link
Contributor

Your PR was merged into the stable branch 19 hours ago 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working conan This issue or PR is related to the conan package manager urgent blocking issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants