On Mon, Feb 06, 2023 at 07:56:50AM -0800, Gordon Messmer wrote:
> On 2023-02-06 07:27, Richard W.M. Jones wrote:
> > Why was this line added?
> > 
> > + Requires: (fedora-release > 38-0.23 if fedora-release)
> > 
> > I suppose it's related to this change in fedora-release:
> > 
> > https://src.fedoraproject.org/rpms/fedora-release/c/096fc7da49de5655db7aad17057e32191f2afe59?branch=rawhide
> 
> 
> Yes.
> 
> 
> > Did you mean >= in the conditional?
> 
> 
> Also yes.
> 
> https://src.fedoraproject.org/rpms/iscsi-initiator-utils/pull-request/11

My bad. I suggested '>'. Usually this would work because there'd be a dist tag,
but fedora-release doesn't have the dist tag.

There's another issue: the dependency should be on fedora-release-common
instead -> 
https://src.fedoraproject.org/rpms/iscsi-initiator-utils/pull-request/12.

Zbyszek
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to