----- Original Message -----
From: "Jakub Hrozek" <jhrozek(a)redhat.com>
To: sssd-devel(a)lists.fedorahosted.org
Sent: Wednesday, January 14, 2015 4:19:15 PM
Subject: Re: [SSSD] [PATCH] MAN: Remove indentation in element programlistening
On Wed, Jan 14, 2015 at 09:10:29AM -0500, Roland Mainz wrote:
>
>
> ----- Original Message -----
> > From: "Lukas Slebodnik" <lslebodn(a)redhat.com>
> > To: sssd-devel(a)lists.fedorahosted.org
> > Sent: Wednesday, January 14, 2015 2:11:22 PM
> > Subject: [SSSD] [PATCH] MAN: Remove indentation in element
> > programlistening
> >
> > ehlo,
> >
> > I noticed that indentation of examples for ad_gpo_map_* is realy weird.
>
> That's the expected behaviour with <programlisting> vs. space/tab/etc
> characters - leading spaces/tabs/etc. are part of the formatting.
>
> > attached is a simple patch which just remove whitespaces.
> [snip]
>
> Patch looks good to me, r=rmainz(a)redhat.com
Thank you for the review, pushed to master as
ecf9e7a870945ecfba8eb751d344de3601de9424
BTW: In the future it may be interesting to use XInclude for longer <programlisting>
chunks to prevent the quoting (and layout) hell - see
https://lists.fedorahosted.org/pipermail/sssd-devel/2015-January/022352.html ("[SSSD]
Using XInclude for DocBook/XML manpages ... / was: Re: xml schema used for generating man
pages") for examples...
----
Bye,
Roland
--
__ . . __
(o.\ \/ /.o) rmainz(a)redhat.com
\__\/\/__/ IPA/Kerberos5 team
/O /==\ O\
(;O/ \/ \O;)