[SSSD] sss.so idmapd plugin manual page... / was: Re: [PATCH v3 0/4] NFSv4 rpc.idmapd plugin

Roland Mainz rmainz at redhat.com
Tue Sep 2 10:53:27 UTC 2014



----- Original Message -----
> From: "Noam Meltzer" <tsnoam at gmail.com>
> To: "Development of the System Security Services Daemon" <sssd-devel at lists.fedorahosted.org>
> Cc: "Roland Mainz" <rmainz at redhat.com>
> Sent: Tuesday, September 2, 2014 12:07:20 PM
> Subject: Re: [SSSD] [PATCH v3 0/4] NFSv4 rpc.idmapd plugin
> 
> On Tue, Sep 2, 2014 at 12:01 PM, Jakub Hrozek <jhrozek at redhat.com> wrote:
> 
> > On Mon, Sep 01, 2014 at 04:35:13PM -0400, Roland Mainz wrote:
> > > Yes... the code seems to be OK... I've tested last weeks code (e.g.
> > prior option rename&co.) in detail in combination with Linux&&Solaris and
> > various NFSv4 server/client setups and it seems to work fine...
> > > ... r=rmainz at redhat.com for the patch series...
> >
> > Thank you for the review and testing. I pushed the patches to master:
> >     68b608d90c716c1aa475bfcb29141bcc6286fe37
> >     b9c8ce2bdd4045782c243605a1b999098bedcffc
> >     4466604d78e5ffd017e69e6861f7d78242b351fb
> >     e9553c2961fa4f25b9d004a6a65b90837a13d8e1
> >
> > And thanks Noam again for the contribution and patience while the
> > patches were merged.
> >
> 
> it's my pleasure to contribute.
> 
> now that the code is committed, I believe it's a good time to remind about
> the man pages for the plugin.
> it was previously split into a separate commit in order to bypass string
> commit freeze.
> 
> here's a link to the patchset:
> https://lists.fedorahosted.org/pipermail/sssd-devel/2014-June/020389.html

Mhhh... some nitpicking after looking at https://lists.fedorahosted.org/pipermail/sssd-devel/2014-June/020390.html ...
1. the manpage has an email address:
-- snip --
+    <refentryinfo>
+        <productname>sss_nfs</productname>
+        <orgname>Noam Meltzer, Primary Data Inc. &lt;noam at primarydata.com&gt; (2013-2014)</orgname>
+        <orgname>Noam Meltzer &lt;tsnoam at gmail.com&gt; (2014-)</orgname>
+    </refentryinfo>
-- snip --

... the <email> tag can't be used within <orgname>, right ? Arguably this looks a bit like an abuse of <orgname> but I don't have an idea yet how to make it better... my DocBook books are 60km away from here...

... my first guess is that this might work (use <authorgroup> to add multiple <authors>):
-- snip --
<refentryinfo>
<author>
<firstname>Noam</firstname>
<surname>Melzer</surname>
<email>noam at primarydata.com</email>
<affiliation>Primary Data Inc.</affiliation>
</author>
</refentryinfo>
-- snip --
If you don't know how to do it then I can try my luck...

2. What about adding the example idmapd.conf file we used for testing to an EXAMPLES section ?

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) rmainz at redhat.com
  \__\/\/__/  IPA/Kerberos5 team
  /O /==\ O\  
 (;O/ \/ \O;)
 



More information about the sssd-devel mailing list