You don't need to specify it for RHEL.

LZ

On Mon, Jun 03, 2013 at 06:52:13AM -0700, Dominic Cleal wrote:
> > +%endif
> > +Requires(post):     /usr/sbin/semodule, /sbin/restorecon, /usr/sbin/setsebool, /usr/sbin/selinuxenabled, /usr/sbin/semanage
> > +Requires(post):     policycoreutils-python
> > +Requires(post):     selinux-policy-targeted
> > +Requires(postun):   /usr/sbin/semodule, /sbin/restorecon
> > +Requires(pre):      %{modulename}
> > +
> > +%description
> > +SELinux policy module for Foreman
> > +
> > +%prep
> > +%setup -q
> > +
> > +%build
> > +# create selinux-friendly version from VR and replace it inplace
> > +perl -i -pe 'BEGIN { $VER = join ".", grep /^\d+$/, split /\./, "%{version}.%{release}"; } s!\@\@VERSION\@\@!$VER!g;' %{modulename}.te
> 
> Does perl need listing as a BR, or is it a known default?  (it's often not on EL systems)
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/theforeman/foreman-selinux/pull/1/files#r4499610

-- 
Later,

 Lukas "lzap" Zapletal
 irc: lzap #theforeman