-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[clapack, lapack-reference] Add conflict messages #13200
[clapack, lapack-reference] Add conflict messages #13200
Conversation
Co-authored-by: Jack·Boos·Yu <[email protected]>
Co-authored-by: Jack·Boos·Yu <[email protected]>
Since there might appear even other provider of lapack in future (there are many vendor), what about using the strategy adopted in opencv (where there are already v2, v3 and v4 here on vcpkg, that have to be exclusively installed)? |
@cenit But these two ports are different, not a higher or lower version of the same port. |
No I mean... Let's assume port A, B and C provide lapack.lib. If you have installed B and then you ask to install A, in order to tell you that you have to remove port B, vcpkg needs to check for the presence of share/B. This is the only robust way to identify colliding ports. |
@cenit make sense. |
That's the only similarity with opencv2/3/4 ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NancyLi1013 @JackBoosY When we edit alternatives like this make sure to double check the ones that CI is not testing manually.
Thanks for your contribution! |
I have a problem related to this, can't install Clapack due to lapack.lib being installed already from lapack-reference. |
@JoaoAVDuarte That is not something vcpkg can fix: only one of those libraries can be in use at a time because they both try to provide the same symbols and library names. You need to choose one. |
Describe the pull request
clapack
andlapack-reference
cannot be installed at the same time, since both of them will install lib/lapack.lib.Note: No need to test features.