-
Notifications
You must be signed in to change notification settings - Fork 363
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
Treat tilde versions as higher for purpose of dependency comparison #715
Comments
Err, no. If its sorts lower than 2.0.0 then it must match the < 2.0.0 dependency, anything else would be a mess that nobody will understand. How exactly is this a problem for you? |
To rephrase a bit, I can see why you'd sometimes want 2.0.0~alpha to be considered 2.0.0, but also that in other cases not, and rpm cannot magically know which one you want for a particular case. It'd be nice if there was a way to explicitly express both cases. This sort of reminds me of this (otherwise unrelated) case: http://lists.rpm.org/pipermail/rpm-maint/2010-November/002948.html |
I guess there is no real way around using |
@pmatilai yeah, this is somewhat similar. Unfortunately I don't know how we could nicely express both. @ffesti yeah, the only problem is that semver.org says you can have versions like |
So what's the actual use-case here? Converting upstream dependency tracking data to rpm? |
For this specific one is to be used in |
whoops, wrong ticlket. For this one is yes -- converting upstream dependency data into the RPM one... which works fine except for this small thingy. |
Since what you really want is a significantly different total ordering, you really need a different set of binary relations. I certainly don't want to be the one to propose |
As pointed out by @jasontibbitts we really don't want to go down this rabbit whole. I am closing this - even if this is not great. Sorry! |
In Rust packaging we've found issue that we have
Requires: (foo >= 1.0.0 with foo < 2.0.0)
and package with version2.0.0~alpha
is actually is matching this range. I think it should be treated for dependency check purpose./cc @ffesti
The text was updated successfully, but these errors were encountered: