Hi,

Am Montag, 11. April 2011, 16:53:41 schrieb Miroslav Suchý:
> 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

ok.
 
> +%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.

Ahh, different philosophies apply here :-)
So let me explain.

OBS does not build this RPM, because during build, every directory is checked 
if it is owned by some package. rhn-client-tools is in Requires, but not 
installed during the build process. This is the reason, why I get this error 
message. So I have two possibilities:

1) add a BuildRequires: rhn-client-tools

2) add the %dir statement like above, maybe with %if suse_version around. RPM 
work fine with directories owned by multiple packages.

At SUSE we prefer to add the %dir statement to not accidently introduce build 
cycles. 

Please tell me what you would prefer. I will send a new patch later this week. 

-- 
MFG

        Michael Calmer

--------------------------------------------------------------------------
Michael Calmer
SUSE LINUX Products GmbH, Maxfeldstr. 5, D-90409 Nuernberg
T: +49 (0) 911 74053 0
F: +49 (0) 911 74053575  - e-mail: michael.cal...@suse.com
--------------------------------------------------------------------------
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg)

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

Reply via email to