Weak Requirement Range Causes Conflict #1824
Labels
Comments
|
This is expected AFAIK: >>> ver_range = rez.version.VersionRange('houdini-19+<=20.5')
>>> ver_range.contains_version(rez.version.Version('20.5.278'))
False20.5.278 is greater than 20.5 here. |
|
Fair enough. Might be a point for the docs, unless I'm the only one who was caught out by this :) |
JeanChristopheMorinPerso
added
documentation
help wanted
|
Sure thing @Anti-Distinctlyminty. Would you like to submit a pull request to improve our docs in that area? |
|
Absolutely. I'll add it to my todo list :) |
|
@Anti-Distinctlyminty Would you like to proceed with a PR, or should I close this issue? |
sarah-kamall
pushed a commit
to sarah-kamall/rez
that referenced
this issue
Nov 12, 2024
…areFoundation#1824) Adjusted weak statement to specify that upper limit also includes patch versions.
sarah-kamall
pushed a commit
to sarah-kamall/rez
that referenced
this issue
Nov 12, 2024
…areFoundation#1824) Adjusted weak statement to specify that upper limit also includes patch versions. Signed-off-by: Sarah Soliman
sarah-kamall
added a commit
to sarah-kamall/rez
that referenced
this issue
Nov 12, 2024
…areFoundation#1824) Adjusted weak statement to specify that upper limit also includes patch versions. Signed-off-by: Sarah Soliman
sarah-kamall
added a commit
to sarah-kamall/rez
that referenced
this issue
Nov 12, 2024
…areFoundation#1824) Adjusted weak statement to specify that upper limit also includes patch versions. Signed-off-by: sarah-kamall <[email protected]>
|
It is on my todo list. I would like to contontribute.
…On Tue, 12 Nov 2024 at 19:35, Archit H Barve ***@***.***> wrote:
@Anti-Distinctlyminty <https://github.com/Anti-Distinctlyminty> Would you
like to proceed with a PR, or should I close this issue?
—
Reply to this email directly, view it on GitHub
<#1824 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKQRM73DFJ35EDAILME4QL2AJC65AVCNFSM6AAAAABNWQLRTCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZRGI4DKMZYG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I think you can close as Sarah's PR addresses it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
and others
The weak requirement
~houdini-19+<=20.5causes a conflict when using the packagehoudini-20.5.I thought that by default if a minor/patch version is not specified, then it means "all are ok". But this appears to not be the case.
Environment
To Reproduce
houdini-20.5, the other with the requirement~houdini-19+<=20.5.rez-envExpected behavior
houdini-20.5should be accepted by the requirement~houdini-19+<=20.5.Actual behavior
The text was updated successfully, but these errors were encountered: