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: [email protected]
--------------------------------------------------------------------------
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg)
_______________________________________________
Spacewalk-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/spacewalk-devel
