On Mon, Jul 14, 2014 at 10:10:59PM +0300, Noam Meltzer wrote:
On Mon, Jul 14, 2014 at 7:30 PM, Jakub Hrozek
<jhrozek(a)redhat.com> wrote:
> On Fri, Jun 27, 2014 at 09:44:38AM +0300, Noam Meltzer wrote:
> > ---
> > contrib/sssd.spec.in | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/contrib/sssd.spec.in b/contrib/sssd.spec.in
> > index b5e7903..8d2f2be 100644
> > --- a/contrib/sssd.spec.in
> > +++ b/contrib/sssd.spec.in
> > @@ -138,6 +138,11 @@ BuildRequires: systemd-devel
> > %if (0%{?with_cifs_utils_plugin} == 1)
> > BuildRequires: cifs-utils-devel
> > %endif
> > +%if ((0%{?fedora} >= 15) || (0%{?rhel} >= 7))
> > +BuildRequires: libnfsidmap-devel
> > +%else
> > +BuildRequires: nfs-utils-lib-devel
> > +%endif
> >
> > # RHEL 5 is too old to support samba4 and the PAC responder
> > %if !0%{?is_rhel5}
> > @@ -451,6 +456,7 @@ autoreconf -ivf
> > --with-krb5-rcache-dir=%{_localstatedir}/cache/krb5rcache \
> > --enable-nsslibdir=/%{_lib} \
> > --enable-pammoddir=/%{_lib}/security \
> > + --enable-nfsidmaplibdir=%{_libdir}/libnfsidmap \
> > --disable-static \
> > --disable-rpath \
> > %{?with_ccache} \
> > @@ -600,6 +606,7 @@ rm -rf $RPM_BUILD_ROOT
> > # 3rd party application libraries
> > %{_libdir}/sssd/modules/libsss_autofs.so
> > %{_libdir}/libsss_sudo.so
> > +%{_libdir}/libnfsidmap/sss_nfs.so
>
> Here I've got a question, the directory %{_libdir}/libnfsidmap seems not
> to be owned by any package, at least on Fedora-20, however the
> individual files are:
> $ rpm -qf /usr/lib64/libnfsidmap/
> file /usr/lib64/libnfsidmap is not owned by any package
> $ rpm -qf /usr/lib64/libnfsidmap/nsswitch.so
> libnfsidmap-0.25-8.fc20.x86_64
>
> The reason I'm curious is that initially I wanted to Require the package
> that owns this directory..
>
> Do you agree this is a packaging bug of libnfsidmap ?
>
I agree
It's been filed for some time already:
https://bugzilla.redhat.com/show_bug.cgi?id=986678
>
> Otherwise ACK to your patch.
Actually, I think we should also add:
Requires: libnfsidmap
to the spec file.
I was wondering for a bit if we should split the plugin to a separate
subpackage because of this new Requirement, but I think it's too much
pain for little gain. On the vast majority of systems, libnfsidmap would
be installed anyway, because it's required by nfs-utils, which provides
some quite basic binaries like mount.nfs. Also, the size of the library
is only about 100 kB..