Re: Please stop re-adding gtk-update-icon-cache scriptlets (for Fedora)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 18/01/18 19:45, Sérgio Basto wrote:
> On Thu, 2018-01-18 at 20:17 +0100, Igor Gnatenko wrote:
>> Hello,
>>
>> I'm working on removing all this cruft from all our packages (and
>> creating
>> conditionals for all packages which have epel branch).
> 
>> Unfortunately some maintainers adding them back with conditionals
>> like:
>> %if 0%{?fedora} < 28 || 0%{?rhel} < 8
>>
>> 1. Those scriptlets are not needed since ~ F24 era
>> 2. You might not know, but 0%{?rhel} on Fedora evaluates to "0" which
>> is "< 8",
>> so those scriptlets are active.
>>
>> Also forgive me if your package had some EL* specific conditions and
>> I removed
>> scriptlets (because there was no epel* branch) -- please use
>> %if 0%{?rhel} && 0%{?rhel} <= 7
>>
>> for them.
> 
> Hello , 
> BTW I have some questions on how exactly we should deal with EPEL 7 and
> 6, from old wiki page [1] desktop-database and  mimeinfo  have been
> removed and Icon cache too ?  IMHO these info should still be in wiki
> and not deleted, maybe also should explain the status on EPEL versions 
> ... 
> 
> Maybe more correct scriptlet is: 
> %if 0%{?fedora} < 25 && 0%{?rhel} < 8

I'm not even the best at bash, much less the scripting in spec files,
but is it possible this would be a problem because e.g. Fedora 26 would
be evaluated as 026, which would evaluate as octal, and therefore the
conditional evaluate as 22 < 25, true, when it was meant to be false?

> 
> And about appdata how exclude it from EPEL 6 ? 
> 
> %if 0%{?rhel} > 6 || 0%{?fedora} ? 
> 
> I'd like that we have some "official" scriptlets for that. 
> 
> Best regards and thanks.
> 
> [1]
> https://fedoraproject.org/w/index.php?title=Packaging:Scriptlets&oldid=468484#mimeinfo
> 


-- 
J. Randall Owens | http://www.GhiaPet.net/

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux