<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 15, 2014 at 6:28 PM, Jakub Hrozek <span dir="ltr">&lt;<a href="mailto:jhrozek@redhat.com" target="_blank">jhrozek@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Jul 14, 2014 at 10:10:59PM +0300, Noam Meltzer wrote:<br>
&gt; On Mon, Jul 14, 2014 at 7:30 PM, Jakub Hrozek &lt;<a href="mailto:jhrozek@redhat.com">jhrozek@redhat.com</a>&gt; wrote:<br>
&gt;<br>
&gt; &gt; On Fri, Jun 27, 2014 at 09:44:38AM +0300, Noam Meltzer wrote:<br>
&gt; &gt; &gt; ---<br>
&gt; &gt; &gt;  contrib/<a href="http://sssd.spec.in" target="_blank">sssd.spec.in</a> | 7 +++++++<br>
&gt; &gt; &gt;  1 file changed, 7 insertions(+)<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; diff --git a/contrib/<a href="http://sssd.spec.in" target="_blank">sssd.spec.in</a> b/contrib/<a href="http://sssd.spec.in" target="_blank">sssd.spec.in</a><br>
&gt; &gt; &gt; index b5e7903..8d2f2be 100644<br>
&gt; &gt; &gt; --- a/contrib/<a href="http://sssd.spec.in" target="_blank">sssd.spec.in</a><br>
&gt; &gt; &gt; +++ b/contrib/<a href="http://sssd.spec.in" target="_blank">sssd.spec.in</a><br>
&gt; &gt; &gt; @@ -138,6 +138,11 @@ BuildRequires: systemd-devel<br>
&gt; &gt; &gt;  %if (0%{?with_cifs_utils_plugin} == 1)<br>
&gt; &gt; &gt;  BuildRequires: cifs-utils-devel<br>
&gt; &gt; &gt;  %endif<br>
&gt; &gt; &gt; +%if ((0%{?fedora} &gt;= 15) || (0%{?rhel} &gt;= 7))<br>
&gt; &gt; &gt; +BuildRequires: libnfsidmap-devel<br>
&gt; &gt; &gt; +%else<br>
&gt; &gt; &gt; +BuildRequires: nfs-utils-lib-devel<br>
&gt; &gt; &gt; +%endif<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;  # RHEL 5 is too old to support samba4 and the PAC responder<br>
&gt; &gt; &gt;  %if !0%{?is_rhel5}<br>
&gt; &gt; &gt; @@ -451,6 +456,7 @@ autoreconf -ivf<br>
&gt; &gt; &gt;      --with-krb5-rcache-dir=%{_localstatedir}/cache/krb5rcache \<br>
&gt; &gt; &gt;      --enable-nsslibdir=/%{_lib} \<br>
&gt; &gt; &gt;      --enable-pammoddir=/%{_lib}/security \<br>
&gt; &gt; &gt; +    --enable-nfsidmaplibdir=%{_libdir}/libnfsidmap \<br>
&gt; &gt; &gt;      --disable-static \<br>
&gt; &gt; &gt;      --disable-rpath \<br>
&gt; &gt; &gt;      %{?with_ccache} \<br>
&gt; &gt; &gt; @@ -600,6 +606,7 @@ rm -rf $RPM_BUILD_ROOT<br>
&gt; &gt; &gt;  # 3rd party application libraries<br>
&gt; &gt; &gt;  %{_libdir}/sssd/modules/libsss_autofs.so<br>
&gt; &gt; &gt;  %{_libdir}/libsss_sudo.so<br>
&gt; &gt; &gt; +%{_libdir}/libnfsidmap/sss_nfs.so<br>
&gt; &gt;<br>
&gt; &gt; Here I&#39;ve got a question, the directory %{_libdir}/libnfsidmap seems not<br>
&gt; &gt; to be owned by any package, at least on Fedora-20, however the<br>
&gt; &gt; individual files are:<br>
&gt; &gt; $ rpm -qf /usr/lib64/libnfsidmap/<br>
&gt; &gt; file /usr/lib64/libnfsidmap is not owned by any package<br>
&gt; &gt; $ rpm -qf /usr/lib64/libnfsidmap/nsswitch.so<br>
&gt; &gt; libnfsidmap-0.25-8.fc20.x86_64<br>
&gt; &gt;<br>
&gt; &gt; The reason I&#39;m curious is that initially I wanted to Require the package<br>
&gt; &gt; that owns this directory..<br>
&gt; &gt;<br>
&gt; &gt; Do you agree this is a packaging bug of libnfsidmap ?<br>
&gt; &gt;<br>
&gt;<br>
&gt; I agree<br>
<br>
</div></div>It&#39;s been filed for some time already:<br>
    <a href="https://bugzilla.redhat.com/show_bug.cgi?id=986678" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=986678</a><br>
<div class="">&gt;<br>
&gt;<br>
&gt; &gt;<br>
&gt; &gt; Otherwise ACK to your patch.<br>
<br>
</div>Actually, I think we should also add:<br>
    Requires: libnfsidmap<br>
to the spec file.<br></blockquote><div><br></div><div>I agree. Can you do it while merging the code, or do you want me to send PATCHv4?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
I was wondering for a bit if we should split the plugin to a separate<br>
subpackage because of this new Requirement, but I think it&#39;s too much<br>
pain for little gain. On the vast majority of systems, libnfsidmap would<br>
be installed anyway, because it&#39;s required by nfs-utils, which provides<br>
some quite basic binaries like mount.nfs. Also, the size of the library<br>
is only about 100 kB..<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote></div></div></div>