Improve dnssec-keymgr manual

Illustrate the syntax for the policy options, with semicolons.

Explicitly mention the "default" policy.

Fix a few typos and remove some redundant wording.

CHANGES

@@ -1,3 +1,6 @@
+5162.	[cleanup]	Improve dnssec-keymgr manual. Thanks to Tony Finch.
+			[GL !1518]
+
 5161.	[bug]		Do not require the SEP bit to be set for mirror zone
 			trust anchors. [GL #873]
 

bin/python/dnssec-keymgr.docbook

@@ -65,11 +65,12 @@
     </para>
     <para>
       DNSSEC policy can be read from a configuration file (default
-      <filename>/etc/dnssec-policy.conf</filename>), from which the key
+      <filename>/etc/dnssec-policy.conf</filename>), from which the
-      parameters, publication and rollover schedule, and desired
+      key parameters, publication and rollover schedule, and desired
-      coverage duration for any given zone can be determined.  This
+      coverage duration for any given zone can be determined. This
       file may be used to define individual DNSSEC policies on a
-      per-zone basis, or to set a default policy used for all zones.
+      per-zone basis, or to set a "<literal>default</literal>" policy
+      used for all zones.
     </para>
     <para>
       When <command>dnssec-keymgr</command> runs, it examines the DNSSEC
@@ -245,7 +246,7 @@
       </listitem>
       <listitem>
 	<para>
-	  Algorithm policies:
+	  <emphasis>Algorithm policies:</emphasis>
 	  (<option>algorithm-policy <replaceable>algorithm</replaceable> { ... };</option> )
 	  override default per-algorithm settings.  For example, by default,
 	  RSASHA256 keys use 2048-bit key sizes for both KSK and ZSK. This
@@ -255,11 +256,13 @@
       </listitem>
       <listitem>
 	<para>
-	  Zone policies:
+	  <emphasis>Zone policies:</emphasis>
 	  (<option>zone <replaceable>name</replaceable> { ... };</option> )
 	  set policy for a single zone by name. A zone policy can inherit
 	  a policy class by including a <option>policy</option> option.
 	  Zone names beginning with digits (i.e., 0-9) must be quoted.
+	  If a zone does not have its own policy then the
+	  "<literal>default</literal>" policy applies.
 	</para>
       </listitem>
     </itemizedlist>
@@ -268,7 +271,8 @@
     </para>
     <variablelist>
       <varlistentry>
-	<term><command>algorithm</command></term>
+	<term><command>algorithm</command>
+	  <replaceable>name</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    The key algorithm. If no policy is defined, the default is
@@ -277,13 +281,14 @@
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>coverage</command></term>
+	<term><command>coverage</command>
+	  <replaceable>duration</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    The length of time to ensure that keys will be correct; no action
 	    will be taken to create new keys to be activated after this time.
-	    This can be represented as a number of seconds, or as a duration using
+	    This can be represented as a number of seconds, or as a duration
-	    human-readable units (examples: "1y" or "6 months").
+	    using human-readable units (examples: "1y" or "6 months").
 	    A default value for this option can be set in algorithm policies
 	    as well as in policy classes or zone policies.
 	    If no policy is configured, the default is six months.
@@ -291,7 +296,8 @@
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>directory</command></term>
+	<term><command>directory</command>
+	  <replaceable>path</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    Specifies the directory in which keys should be stored.
@@ -299,11 +305,12 @@
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>key-size</command></term>
+	<term><command>key-size</command> <replaceable>keytype</replaceable>
+	  <replaceable>size</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    Specifies the number of bits to use in creating keys.
-	    Takes two arguments: keytype (eihter "zsk" or "ksk") and size.
+	    The keytype is either "zsk" or "ksk".
 	    A default value for this option can be set in algorithm policies
 	    as well as in policy classes or zone policies. If no policy is
 	    configured, the default is 2048 bits for RSA keys.
@@ -311,7 +318,8 @@
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>keyttl</command></term>
+	<term><command>keyttl</command>
+	  <replaceable>duration</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    The key TTL. If no policy is defined, the default is one hour.
@@ -319,46 +327,50 @@
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>post-publish</command></term>
+	<term><command>post-publish</command> <replaceable>keytype</replaceable>
+	  <replaceable>duration</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    How long after inactivation a key should be deleted from the zone.
 	    Note: If <option>roll-period</option> is not set, this value is
-	    ignored. Takes two arguments: keytype (eihter "zsk" or "ksk") and a
+	    ignored. The keytype is either "zsk" or "ksk".
-	    duration. A default value for this option can be set in algorithm
+	    A default duration for this option can be set in algorithm
 	    policies as well as in policy classes or zone policies. The default
 	    is one month.
 	  </para>
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>pre-publish</command></term>
+	<term><command>pre-publish</command> <replaceable>keytype</replaceable>
+	  <replaceable>duration</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    How long before activation a key should be published.  Note: If
 	    <option>roll-period</option> is not set, this value is ignored.
-	    Takes two arguments: keytype (either "zsk" or "ksk") and a duration.
+	    The keytype is either "zsk" or "ksk".
-	    A default value for this option can be set in algorithm policies
+	    A default duration for this option can be set in algorithm policies
 	    as well as in policy classes or zone policies.  The default is
 	    one month.
 	  </para>
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>roll-period</command></term>
+	<term><command>roll-period</command> <replaceable>keytype</replaceable>
+	  <replaceable>duration</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    How frequently keys should be rolled over.
-	    Takes two arguments: keytype (eihter "zsk" or "ksk") and a duration.
+	    The keytype is either "zsk" or "ksk".
-	    A default value for this option can be set in algorithm policies
+	    A default duration for this option can be set in algorithm policies
 	    as well as in policy classes or zone policies.  If no policy is
-	    configured, the default is one year for ZSK's. KSK's do not
+	    configured, the default is one year for ZSKs. KSKs do not
 	    roll over by default.
 	  </para>
 	</listitem>
       </varlistentry>
       <varlistentry>
-	<term><command>standby</command></term>
+	<term><command>standby</command> <replaceable>keytype</replaceable>
+	  <replaceable>number</replaceable><literal>;</literal></term>
 	<listitem>
 	  <para>
 	    Not yet implemented.