At present, the Book has this argument to configure

            --with-passprompt="[sudo] password for %p: "

but doesn't explain the "%p", merely saying

--with-passprompt: This switch sets the password prompt.

which might leave the user wondering what the %p stands for.

The attached patch adds an explanation, taken from the sudo man page,
as follows:

--with-passprompt: This switch sets the password prompt. The %p will
be expanded to the name of the user whose password is being requested.

Having added that extra explanation though, I found that the text
looked a bit "unwieldy",
in that the text flows under the argument, and so version 2 of the
patch replaces the
"standard paragraph text", used for the command explanations, with a
<variablelist">
that makes things look a lot tidier, and has the advantage, at least
in my rendering, of
having the argument in the same font as it appears in the command
instructions above
the explanation.

Just my the'pen'th though,
Kevin
Index: postlfs/security/sudo.xml
===================================================================
--- postlfs/security/sudo.xml	(revision 22992)
+++ postlfs/security/sudo.xml	(working copy)
@@ -163,6 +170,7 @@
 
     <para>
       <parameter>--with-passprompt</parameter>: This switch sets the password prompt.
+      The <parameter>%p</parameter> will be expanded to the name of the user whose password is being requested.
     </para>
 
     <para>
Index: postlfs/security/sudo.xml
===================================================================
--- postlfs/security/sudo.xml	(revision 22992)
+++ postlfs/security/sudo.xml	(working copy)
@@ -137,36 +137,71 @@
   <sect2 role="commands">
     <title>Command Explanations</title>
 
-    <para>
-     <parameter>--libexecdir=/usr/lib</parameter>: This switch controls where
-     private programs are installed.  Everything in that directory is a library, so
-     they belong under <filename class="directory">/usr/lib</filename> instead of
-     <filename class="directory">/usr/libexec</filename>.
-    </para>
+  <variablelist>
+    <varlistentry>
+      <term><parameter>--libexecdir=/usr/lib</parameter></term>
+      <listitem>
+        <para>
+          This switch controls where private programs are installed.
+        </para>
+        <para>
+          Everything in that directory is a library, so they belong under
+          <filename class="directory">/usr/lib</filename> instead of
+          <filename class="directory">/usr/libexec</filename>.
+        </para>
+      </listitem>
+    </varlistentry>
 
-    <para>
-      <parameter>--with-secure-path</parameter>: This switch transparently adds
-      <filename class="directory">/sbin</filename> and <filename
-      class="directory">/usr/sbin</filename> directories to the
-      <envar>PATH</envar> environment variable.
-    </para>
+    <varlistentry>
+      <term><parameter>--with-secure-path</parameter></term>
+      <listitem>
+      <para>
+        This switch transparently adds
+        <filename class="directory">/sbin</filename> and <filename
+        class="directory">/usr/sbin</filename> directories to the
+        <envar>PATH</envar> environment variable.
+      </para>
+      </listitem>
+    </varlistentry>
 
-    <para>
-      <parameter>--with-all-insults</parameter>: This switch includes all the
-      <application>sudo</application> insult sets.
-    </para>
+    <varlistentry>
+      <term><parameter>--with-all-insults</parameter></term>
+      <listitem>
+      <para>
+        This switch includes all the
+        <application>sudo</application> insult sets.
+      </para>
+      </listitem>
+    </varlistentry>
 
-    <para>
-      <parameter>--with-env-editor</parameter>: This switch enables use of the
-      environment variable EDITOR for <command>visudo</command>.
-    </para>
+    <varlistentry>
+      <term><parameter>--with-env-editor</parameter></term>
+      <listitem>
+      <para>
+        This switch enables use of the
+        environment variable EDITOR for <command>visudo</command>.
+      </para>
+      </listitem>
+    </varlistentry>
 
-    <para>
-      <parameter>--with-passprompt</parameter>: This switch sets the password prompt.
-    </para>
+    <varlistentry>
+      <term><parameter>--with-passprompt</parameter></term>
+      <listitem>
+      <para>
+        This switch sets the password prompt.
+      </para>
+      <para>
+        The <parameter>%p</parameter> will be expanded to the name of the
+        user whose password is being requested.
+      </para>
+      </listitem>
+    </varlistentry>
 
+    <varlistentry>
+      <term><option>--without-pam</option></term>
+      <listitem>
     <para>
-      <option>--without-pam</option>: This switch avoids building
+       This switch avoids building
       <application>Linux-PAM</application> support when
       <application>Linux-PAM</application> is installed on the system.
     </para>
@@ -173,7 +208,11 @@
 <!-- See the developer note above before the configure command
     <xi:include xmlns:xi="http://www.w3.org/2001/XInclude";
       href="../../xincludes/static-libraries.xml"/>-->
+      </listitem>
+    </varlistentry>
 
+  </variablelist>
+
     <note>
       <para>
         There are many options to <application>sudo</application>'s
-- 
http://lists.linuxfromscratch.org/listinfo/blfs-dev
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Reply via email to