On Tue, Sep 2, 2014 at 1:53 PM, Roland Mainz <rmainz(a)redhat.com> wrote:
----- Original Message -----
> From: "Noam Meltzer" <tsnoam(a)gmail.com>
> To: "Development of the System Security Services Daemon" <
sssd-devel(a)lists.fedorahosted.org>
> Cc: "Roland Mainz" <rmainz(a)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(a)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(a)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. <noam at
primarydata.com> (2013-2014)</orgname>
+ <orgname>Noam Meltzer <tsnoam at gmail.com>
(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(a)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...
about "orgname":
I wasn't familiar with DocBook before, so I took example from pam_sss.8.xml
Anyhow, my goal is to provide two AUTHORS for the manpage. Since I'm no
longer an employee of PrimaryData, I need to provide my current email
address. However, it is important to me to give them the due credit.
However2, (to my defense) this DocBook compiles into a valid manpage...
2. What about adding the example idmapd.conf file we used for testing
to
an EXAMPLES section ?
sure.
if you can send me the idmapd.conf you've used for testing, I'll add it to
the EXAMPLES section.
----
Bye,
Roland
--
__ . . __
(o.\ \/ /.o) rmainz(a)redhat.com
\__\/\/__/ IPA/Kerberos5 team
/O /==\ O\
(;O/ \/ \O;)