On Thu, Apr 25, 2013 at 02:09:43PM +0200, Michal Židek wrote:
> On 04/24/2013 08:57 PM, Jakub Hrozek wrote:
> >On Tue, Apr 23, 2013 at 11:40:38AM -0400, Stephen Gallagher wrote:
> >>-----BEGIN PGP SIGNED MESSAGE-----
> >>Hash: SHA1
> >>
> >>On 04/18/2013 09:38 AM, Michal Židek wrote:
> >>>On 04/18/2013 02:20 PM, Jakub Hrozek wrote:
> >>>>diff --git a/src/man/sssd-ad.5.xml b/src/man/sssd-ad.5.xml index
> >>>>6ab133d5204fb2adcb65bafb33e8242a5afcaa70..cfd03b8d30eaf8ad6bb8c333119dc6df8ff381d8
> >>>>
> >>>>
> >>100644
> >>>>--- a/src/man/sssd-ad.5.xml +++ b/src/man/sssd-ad.5.xml @@ -68,6
> >>>>+68,7 @@ <programlisting> ldap_id_mapping = False
> >>>></programlisting> +            The AD provider is also
> >>>>case-insensitive by default.
> >>>
> >>>I think it would be better to say, that AD provider is "always"
> >>>case insensitive (not just "by default").
> >>>
> >>
> >>As a native speaker, I'd suggest that this would read better as
> >>
> >>"Users, groups and other entities served by SSSD are always treated as
> >>case-insensitive in the AD provider for compatibility with Active
> >>Directory's LDAP implementation."
> >>
> >>It provides a little more information (so people don't have to go
> >>looking for the case-sensitivity section of the docs).
> >
> >Thanks for the suggestion, new patches are attached.
> >
> 
> The wording looks good to me and man pages build fine with these patches.
> 
> Ack!
> 
> Michal

Thanks for the review.

Pushed to master.
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to