On 05/21/2010 01:24 AM, Kevin Kofler wrote:
Toshio Kuratomi wrote:
> This is also wrong because we want 0%{?rhel} <= 5.
>
> To avoid having to specify negation I'd also reverse the condition:
>
> %if 0%{?rhel} >= 5 || 0%{?fedora}
> BuildRequires: tcp_wrappers-devel
> %else
> BuildRequires: tcp_wrappers
> %endif
I'd spell the condition as:
%if 0%{?rhel} && 0{%?rhel} < 6
BuildRequires: tcp_wrappers
%else
BuildRequires: tcp_wrappers-devel
%endif
are you sure rhel is defined on rhel-5? imho not by default!
--
Levente "Si vis pacem para bellum!"