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!" -- devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/devel