On 04/08/2011 02:57 PM, Michael Calmer wrote:
> Hi,
> 
> here is the patch for rhncfg.
> 
> 0007-build-rhncfg-on-SUSE.patch:
> - only some specfile modifications

If I put aside the fact that I would prefer more commits about this
splitting things which "allow build rhncfg on SUSE" like:
+%if 0%{?rhel}
 Requires: libselinux-python
 %endif
+%endif
from general fixes, like:
+%dir %{_sharedstatedir}/rhncfg

Then I have problem with:
+%dir %{_sharedstatedir}
This is owned by filesystem package on Fedora. If this is not owned by
any base package on SUSE wrap it with if/endif

+%dir %{rhnconf}
This should not be there. This directory is owned by rhn-client-tools
and we Require it.

+%dir %{client_caps_dir}
This is the same. This directory is owned by rhn-client-tools and we
Require it.

-- 
Miroslav Suchy
Red Hat Satellite Engineering

_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to