GNU bug report logs - #7622
[PATCH] lib-src/emacsclient.c: Change -H to -h. Order options alphabetically

Previous Next

Package: emacs;

Reported by: Jari Aalto <jari.aalto <at> cante.net>

Date: Sun, 12 Dec 2010 14:43:02 UTC

Severity: wishlist

Tags: patch

Found in version 23.2+1-5.1

Done: Lars Magne Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Lars Magne Ingebrigtsen <larsi <at> gnus.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#7622: closed ([PATCH] lib-src/emacsclient.c: Change -H to -h.
 Order options alphabetically)
Date: Tue, 10 Apr 2012 23:50:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Wed, 11 Apr 2012 01:48:05 +0200
with message-id <m3y5q3p2ru.fsf <at> stories.gnus.org>
and subject line Re: bug#7622: [PATCH] lib-src/emacsclient.c: Change -H to -h. Order options alphabetically
has caused the debbugs.gnu.org bug report #7622,
regarding [PATCH] lib-src/emacsclient.c: Change -H to -h. Order options alphabetically
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
7622: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=7622
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Jari Aalto <jari.aalto <at> cante.net>
To: submit <at> debbugs.gnu.org
Subject: [PATCH] lib-src/emacsclient.c: Change -H to -h. Order options
	alphabetically
Date: Sun, 12 Dec 2010 16:48:37 +0200
[Message part 3 (text/plain, inline)]
Package: emacs
Version: 23.2+1-5.1
Severity: wishlist
Tags: patch

The patch changes help option to lowercase -h like found e.g. in GNU
cp(1). The option help is also presented alphabetically.

The patch is against Git:

    74bd01f 2010-12-12  Merge branch 'master' of git://git.sv.gnu.org/emacs

[ lib-src/ChangeLog ]

2010-12-12  Jari Aalto  <jari.aalto <at> cante.net>

        * emacsclient.c (decode_options): Change help option from -H to -h.
        (print_help_and_exit): Order options alphabetically.

[0001-emacsclient.c-decode_options-Change-help-option-from.patch (text/x-diff, inline)]
From f9c51ef32df48c3a01cb32e9b67aa5694794ccef Mon Sep 17 00:00:00 2001
From: Jari Aalto <jari.aalto <at> cante.net>
Date: Sun, 12 Dec 2010 16:41:47 +0200
Subject: [PATCH] * emacsclient.c (decode_options): Change help option from -H to -h.
 (print_help_and_exit): Order options alphabetically.
Organization: Private
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit


Signed-off-by: Jari Aalto <jari.aalto <at> cante.net>
---
 lib-src/emacsclient.c |   36 +++++++++++++++++++-----------------
 1 files changed, 19 insertions(+), 17 deletions(-)

diff --git a/lib-src/emacsclient.c b/lib-src/emacsclient.c
index 48ea3d2..9759898 100644
--- a/lib-src/emacsclient.c
+++ b/lib-src/emacsclient.c
@@ -516,9 +516,9 @@ decode_options (int argc, char **argv)
     {
       int opt = getopt_long_only (argc, argv,
 #ifndef NO_SOCKETS_IN_FILE_SYSTEM
-			     "VHnea:s:f:d:tc",
+			     "Vhnea:s:f:d:tc",
 #else
-			     "VHnea:f:d:tc",
+			     "Vhnea:f:d:tc",
 #endif
 			     longopts, 0);
 
@@ -581,7 +581,7 @@ decode_options (int argc, char **argv)
           current_frame = 0;
 	  break;
 
-	case 'H':
+	case 'h':
 	  print_help_and_exit ();
 	  break;
 
@@ -646,29 +646,31 @@ Tell the Emacs server to visit the specified files.\n\
 Every FILE can be either just a FILENAME or [+LINE[:COLUMN]] FILENAME.\n\
 \n\
 The following OPTIONS are accepted:\n\
--V, --version		Just print version info and return\n\
--H, --help    		Print this usage information message\n\
--nw, -t, --tty 		Open a new Emacs frame on the current terminal\n\
+-a EDITOR, --alternate-editor=EDITOR\n\
+			Editor to fallback to if the server is not running\n"
+#ifndef WINDOWSNT
+"			If EDITOR is the empty string, start Emacs in daemon\n\
+			mode and try connecting again\n"
+#endif /* not WINDOWSNT */
+"\
 -c, --create-frame    	Create a new frame instead of trying to\n\
 			use the current Emacs frame\n\
--e, --eval    		Evaluate the FILE arguments as ELisp expressions\n\
--n, --no-wait		Don't wait for the server to return\n\
 -d DISPLAY, --display=DISPLAY\n\
 			Visit the file in the given display\n\
+-e, --eval    		Evaluate the FILE arguments as ELisp expressions\n\
+-f SERVER, --server-file=SERVER\n\
+			Set filename of the TCP authentication file\n\
+-n, --no-wait		Don't wait for the server to return\n\
+-nw, -t, --tty 		Open a new Emacs frame on the current terminal\n\
 --parent-id=ID          Open in parent window ID, via XEmbed\n"
 #ifndef NO_SOCKETS_IN_FILE_SYSTEM
 "-s SOCKET, --socket-name=SOCKET\n\
 			Set filename of the UNIX socket for communication\n"
 #endif
-"-f SERVER, --server-file=SERVER\n\
-			Set filename of the TCP authentication file\n\
--a EDITOR, --alternate-editor=EDITOR\n\
-			Editor to fallback to if the server is not running\n"
-#ifndef WINDOWSNT
-"			If EDITOR is the empty string, start Emacs in daemon\n\
-			mode and try connecting again\n"
-#endif /* not WINDOWSNT */
-"\n\
+"\
+-h, --help    		Print this usage information message\n\
+-V, --version		Just print version info and return\n\
+\n\
 Report bugs with M-x report-emacs-bug.\n", progname);
   exit (EXIT_SUCCESS);
 }
-- 
1.7.2.3

[Message part 5 (text/plain, inline)]
-- System Information
Debian Release: squeeze/sid
  APT Prefers testing
  APT policy: (990, testing) (500, unstable) (1, experimental)
Architecture: amd64
Kernel: Linux picasso 2.6.32-5-amd64 #1 SMP Fri Sep 17 21:50:19 UTC 2010 x86_64 GNU/Linux
Locale: LANG=en_DK.UTF-8

-- Versions of packages `emacs depends on'.
Depends:
emacs23         23.2+1-5.1      GNU Emacs is the extensible self-documenting 
emacs23-lucid   23.2+1-5.1      GNU Emacs is the extensible self-documenting 
emacs23-nox     23.2+1-5.1      GNU Emacs is the extensible self-documenting 
[Message part 6 (message/rfc822, inline)]
From: Lars Magne Ingebrigtsen <larsi <at> gnus.org>
To: Jari Aalto <jari.aalto <at> cante.net>
Cc: 7622-close <at> debbugs.gnu.org
Subject: Re: bug#7622: [PATCH] lib-src/emacsclient.c: Change -H to -h. Order
	options alphabetically
Date: Wed, 11 Apr 2012 01:48:05 +0200
Jari Aalto <jari.aalto <at> cante.net> writes:

> The patch changes help option to lowercase -h like found e.g. in GNU
> cp(1). The option help is also presented alphabetically.

[larsi <at> stories ~/src/emacs/trunk]$ cp -h
cp: invalid option -- 'h'

So I don't think this change would be correct.

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/


This bug report was last modified 13 years and 137 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.