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