Re: Appreciate advice on %if %foo || %bar syntax

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

 



On Thu, 2007-08-30 at 08:12 +0200, Tony Earnshaw wrote:
> Michael Jennings skrev, on 29-08-2007 23:48:
> 
> >> I have a spec file which tests for various distro's:
> >>
> >> %define is_fc45 %(test -f /etc/fedora-release && cat /etc/fedora-release | egrep -q '4|5' && echo 1 || echo 0)
> >> %define is_fc6 %(test -f /etc/fedora-release && grep -c Zod /etc/fedora-release || echo 0)
> >> %define is_f7 %(test -f /etc/fedora-release && grep -c Moonshine /etc/fedora-release || echo 0)
> >> %define is_rhel4 %(test -f /etc/redhat-release && cat /etc/redhat-release | egrep -q 'release 4' && echo 1 || echo 0)
> >> %define is_rhel5 %(test -f /etc/redhat-release && cat /etc/redhat-release | egrep -q 'release 5' && echo 1 || echo 0)
> >>
> >> Now I would like to use something like this:
> >>
> >> %if %is_fc6 || %is_f7 || %is_rhel5
> >> ..do something
> >> %endif
> > 
> > %if %{is_fc6}%{is_f7}%{is_rhel5}
> > 
> > will expand to
> > 
> > %if 010
> > 
> > and will do exactly what you want.
> 
> Thanks from somebody else :)
> 
> Quite another thing is, that a more rational test would be something like:
> 
> "test -f /etc/redhat-release && grep 'release 5' /etc/redhat-release && 
> echo 1 || echo 0"
> 
> Redundant use of "cat|grep" and egrep ...

Thanks Tony, I see what you mean. Iirc I copied it a long time ago from
another spec file. It just works so I never bothered to look at it in
more detail.

Regards,
Patrick

_______________________________________________
Rpm-list mailing list
Rpm-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/rpm-list

[Index of Archives]     [RPM Ecosystem]     [Linux Kernel]     [Red Hat Install]     [PAM]     [Red Hat Watch]     [Red Hat Development]     [Red Hat]     [Gimp]     [Yosemite News]     [IETF Discussion]

  Powered by Linux