[PATCH] Wrong option for parameter ldap ssl in smb.conf man page

[Marc Muehlfeld]

Am 16.07.2016 um 20:58 schrieb Rowland Penny:
> Hi Marc, you can have my read by, except, if you are fixing these, you
> should also fix this line:
> 
> Please note that this parameter does only affect <emphasis>rpc</emphasis>
> 
> Better English would be:
> 
> Please note that this parameter will only affect <emphasis>rpc</emphasis>


Attached the new patch. Please review and push.


Regards,
Marc

-------------- next part --------------
From cec197ce31314ec66a74a0f093f47a4bc546853f Mon Sep 17 00:00:00 2001
From: Marc Muehlfeld <mmuehlfeld at samba.org>
Date: Sun, 17 Jul 2016 17:56:24 +0200
Subject: [PATCH] Wrong option for parameter ldap ssl in smb.conf man page

- "ldap ssl" does not accept the value "yes"
- Replaced "start_tls" with "start tls"
- Rephrased sentence

Bug: https://bugzilla.samba.org/show_bug.cgi?id=12023
Signed-off-by: Marc Muehlfeld <mmuehlfeld at samba.org>
---
 docs-xml/smbdotconf/ldap/ldapssl.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/docs-xml/smbdotconf/ldap/ldapssl.xml b/docs-xml/smbdotconf/ldap/ldapssl.xml
index a50c88e..081ea03 100644
--- a/docs-xml/smbdotconf/ldap/ldapssl.xml
+++ b/docs-xml/smbdotconf/ldap/ldapssl.xml
@@ -14,7 +14,7 @@
 
 	<para>LDAP connections should be secured where possible. This may be
 	done setting <emphasis>either</emphasis> this parameter to
-	<parameter moreinfo="none">Start_tls</parameter>
+	<parameter moreinfo="none">start tls</parameter>
 	<emphasis>or</emphasis> by specifying <parameter moreinfo="none">ldaps://</parameter> in
         the URL argument of <smbconfoption name="passdb backend"/>.</para>
 
@@ -33,10 +33,10 @@
 		</listitem>
 	</itemizedlist>
 	<para>
-	Please note that this parameter does only affect <emphasis>rpc</emphasis>
+	Please note that this parameter will only affect <emphasis>rpc</emphasis>
 	methods. To enable the LDAPv3 StartTLS extended operation (RFC2830) for
 	<emphasis>ads</emphasis>, set
-	<smbconfoption name="ldap ssl">yes</smbconfoption>
+	<smbconfoption name="ldap ssl">start tls</smbconfoption>
 	<emphasis>and</emphasis>
 	<smbconfoption name="ldap ssl ads">yes</smbconfoption>.
 	See <refentrytitle>smb.conf</refentrytitle><manvolnum>5</manvolnum>
-- 
2.7.4