<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 20, 2014 at 5:10 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">This patch needs rebasing on current master. For the sake of review, I<br>
rebased it locally -- mostly it was trivial &quot;use both&quot; case.<br>
<br>
The functionality is fine, make, make check and make distcheck all pass.<br>
<div><div class="h5"><br>
On Tue, Mar 04, 2014 at 09:37:54AM +0200, Noam Meltzer wrote:<br>
&gt; ---<br>
&gt;  Makefile.am                 | 19 +++++++++++++++++++<br>
&gt;  <a href="http://configure.ac" target="_blank">configure.ac</a>                | 10 ++++++++++<br>
&gt;  src/conf_macros.m4          | 30 ++++++++++++++++++++++++++++++<br>
&gt;  src/external/libnfsidmap.m4 | 17 +++++++++++++++++<br>
&gt;  4 files changed, 76 insertions(+)<br>
&gt;  create mode 100644 src/external/libnfsidmap.m4<br>
&gt;<br>
&gt; diff --git a/Makefile.am b/Makefile.am<br>
&gt; index 9f010d9..23f6093 100644<br>
&gt; --- a/Makefile.am<br>
&gt; +++ b/Makefile.am<br>
&gt; @@ -47,6 +47,7 @@ localedir = @localedir@<br>
&gt;  nsslibdir = @nsslibdir@<br>
&gt;  pamlibdir = @pammoddir@<br>
&gt;  autofslibdir = @appmodpath@<br>
&gt; +nfslibdir = @nfsidmaplibdir@<br>
&gt;<br>
&gt;  dbpath = @dbpath@<br>
&gt;  pluginpath = @pluginpath@<br>
&gt; @@ -526,6 +527,7 @@ dist_noinst_HEADERS = \<br>
&gt;      src/tests/cmocka/common_mock_sysdb_objects.h \<br>
&gt;      src/sss_client/ssh/sss_ssh_client.h \<br>
&gt;      src/sss_client/sudo/sss_sudo.h \<br>
&gt; +    src/sss_client/nfs/nfsidmap_internal.h \<br>
&gt;      src/lib/idmap/sss_idmap_private.h<br>
&gt;<br>
&gt;<br>
&gt; @@ -1666,6 +1668,23 @@ libnss_sss_la_LDFLAGS = \<br>
&gt;      -version-info 2:0:0 \<br>
&gt;      -Wl,--version-script,$(srcdir)/src/sss_client/sss_nss.exports<br>
&gt;<br>
&gt; +<br>
&gt; +if BUILD_NFS<br>
&gt; +nfslib_LTLIBRARIES = <a href="http://sss_nfs.la" target="_blank">sss_nfs.la</a><br>
&gt; +sss_nfs_la_SOURCES = \<br>
&gt; +    src/sss_client/common.c \<br>
&gt; +    src/sss_client/nss_mc_common.c \<br>
&gt; +    src/util/io.c \<br>
&gt; +    src/util/murmurhash3.c \<br>
&gt; +    src/sss_client/nss_mc_passwd.c \<br>
&gt; +    src/sss_client/nss_mc_group.c \<br>
&gt; +    src/sss_client/nfs/sss_nfs_client.c \<br>
&gt; +    $(NULL)<br>
&gt; +sss_nfs_la_CFLAGS = $(AM_CFLAGS)<br>
&gt; +sss_nfs_la_LIBADD = $(CLIENT_LIBS) $(NFSIDMAP_LIBS)<br>
&gt; +sss_nfs_la_LDFLAGS = -module -avoid-version<br>
<br>
</div></div>I know this is serious bikeshedding, but With CFLAGS, LIBADD and LDFLAGS<br>
could you use the same format we use elsewhere?<br>
<br>
sss_nfs_la_LIBADD = \<br>
    $(CLIENT_LIBS) \<br>
    $(NFSIDMAP_LIBS)<br>
<br>
I&#39;m not too keen about the format itself, but being consistent is good<br>
for maintenance :-)<br></blockquote><div>sure.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The rest looks good to me.<br>
</blockquote></div><br></div></div>