lxterminal icon indicating copy to clipboard operation
lxterminal copied to clipboard

[Patch] man/lxterminal.xml: description of -t and --title is wrong. Add description for help and version.

Open ZjYwMj opened this issue 4 years ago • 3 comments

For

$ lxterminal -v
lxterminal 0.4.0

The following patch

  1. Fixes descriptions of -t and --title in the manual page. Compare it with lxterminal -h.
  2. Add entries for the help and version options.
--- a/man/lxterminal.xml	2021-06-13 17:27:50.173982416 +0000
+++ b/man/lxterminal.xml	2021-06-13 17:27:08.264651478 +0000
@@ -55,17 +55,24 @@
 	<listitem>	  <para>Set the terminal's size in characters and lines.</para>
 	</listitem>
       </varlistentry>
+      <varlistentry>	<term>	  <option>-h</option>
+	  <option>--help</option>
+	</term>fixes
+	<listitem>	  <para>Prints a short help message, and exit.</para>
+	</listitem>
+      </varlistentry>
       <varlistentry>	<term>	  <option>-l</option>
 	  <option>--loginshell</option>
 	</term>
 	<listitem>	  <para>Executes login shell.</para>
 	</listitem>
       </varlistentry>
-      <varlistentry>	<term>	  <option>-t <replaceable>NAME[,NAME[,NAME[...]]]</replaceable></option>
-	  <option>--title=<replaceable>NAME[,NAME[,NAME[...]]]</replaceable></option>
+      <varlistentry>	<term>	  <option>-T <replaceable>NAME</replaceable></option>
+	  <option>-t <replaceable>NAME</replaceable></option>
+	  <option>--title=<replaceable>NAME</replaceable></option>
+ 	  <option>--tabs=<replaceable>NAME,NAME[,NAME[...]]</replaceable></option>
 	</term>
-	<listitem>	  <para>Set the terminal's title. Use comma for multiple tabs.</para>
+	<listitem>	  <para>Set the terminal's title. Comma separated multiple titles create multiple tabs only with the <option>--tabs=</option> form.</para>
 	</listitem>
       </varlistentry>
       <varlistentry>        <term>          <option>--working-directory=<replaceable>DIRECTORY</replaceable></option>
@@ -73,6 +80,12 @@
         <listitem>          <para>Set the terminal's working directory.</para>
         </listitem>
       </varlistentry>
+      <varlistentry>	<term>	  <option>-v</option>
+	  <option>--version</option>
+	</term>
+	<listitem>	  <para>Prints version information, and exit.</para>
+	</listitem>
+      </varlistentry>
     </variablelist>
   </refsect1>
 

ZjYwMj avatar Jun 13 '21 17:06 ZjYwMj

Thank you. Would you like to add this as a pull request to maintain credit?

FinboySlick avatar Jun 14 '21 16:06 FinboySlick

As far as I can tell, I have right now added it as a pull request.

ZjYwMj avatar Jun 15 '21 00:06 ZjYwMj

I have modified the patch some few minutes ago. Actually, It is probably better if I will update my lxterminal repository. I will get to that now. I will explain what I did at the comment in my lxterminal repository.

ZjYwMj avatar Jun 17 '21 18:06 ZjYwMj