Overview
Request 992544 superseded
- Created by mcepl
- In state superseded
- Superseded by 993539
-
Open review for
opensuse-review-team
-
Open review for
factory-staging
Request History
mcepl created request
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
RBrownFactory added openSUSE:Factory:Staging:adi:32 as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:32"
RBrownFactory accepted review
Picked "openSUSE:Factory:Staging:adi:32"
dimstar declined review
21+Requires: (python-cached-property and python-base < 3.8)
Not sure what exactly you try - but this definitively cannot work
a) python-base is translated to python310-base - and that is never < 3.8
b) from the idea, I think what you want is 'if' (with would be a different error)
dimstar declined request
21+Requires: (python-cached-property and python-base < 3.8)
Not sure what exactly you try - but this definitively cannot work
a) python-base is translated to python310-base - and that is never < 3.8
b) from the idea, I think what you want is 'if' (with would be a different error)
coolo-factory reopened request
Reopened via staging workflow.
coolo-factory added factory-staging as a reviewer
Being evaluated by group "factory-staging"
coolo-factory accepted review
Unstaged from project "openSUSE:Factory:Staging:adi:32"
coolo-factory declined request
Declined via staging workflow.
@dimstar:
the
Requires: python-cached-propertyis needed only with Python lower than 3.8 (we need it on the SLE/Leap side).on SLE side we have those versions of
python-rpm-macros:and rpm (both SP3 and SP4) is only version 4.13.3
Do you have any alternative idea?
Requires: (python-cached-property if python-base < 3.8)'with' as documented in the spec would not do the right thing, as this would require a package python-cached-property which, the SAME package, provides python-base < 3.8. definitively not what is intended
'and' as used in your SR would be the same as splitting it simply up to
so also not what you intended.
with 'if' the boolean dep does what you want: if python-base < 3.8 is installed, we require python-cached-property (keep in mind that rpm-single-spec rewrites python topython3, python38, python39, python310, as appropriate)
Luckily, 'if' exists since RPM 4.13