From: Brian Norris <computersforpeace@gmail.com>
To: <linux-kernel@vger.kernel.org>
Cc: Brian Norris <computersforpeace@gmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
Joe Perches <joe@perches.com>
Subject: [PATCH v3 3/4] get_maintainer: add --no-foo options to --help
Date: Wed, 4 Nov 2015 12:58:27 -0800 [thread overview]
Message-ID: <1446670708-122507-3-git-send-email-computersforpeace@gmail.com> (raw)
In-Reply-To: <1446670708-122507-1-git-send-email-computersforpeace@gmail.com>
Many flag options are boolean and support both a positive and a negative
invocation from the command line. Some of these are even mentioned by
example (e.g., --nogit is mentioned as a default option), but they
aren't explicitly mentioned in the list of options. It happens that some
of these are pretty important, as they are default-on, and to turn them
off, you have to know about the --no-foo version.
Rather than clutter the whole help text with bracketed '--[no]foo',
let's just mention the general rule, a la 'man gcc'.
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Joe Perches <joe@perches.com>
---
v2 -> v3:
* move to Notes section
* use <brackets>
* add --no-<foo> (Perl supports more than one form of negation)
v1 -> v2:
https://lkml.org/lkml/2015/11/4/479
* don't clutter with --[no]foo bracketing; just include this note up front;
suggested by Joe
v1:
https://lkml.org/lkml/2015/8/19/441
scripts/get_maintainer.pl | 3 +++
1 file changed, 3 insertions(+)
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 6c307276f3d6..145f1bf6472e 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -845,6 +845,9 @@ Notes:
Entries in this file can be any command line argument.
This file is prepended to any additional command line arguments.
Multiple lines and # comments are allowed.
+ Most options have both positive and negative forms.
+ The negative forms for --<foo> are --no<foo> and --no-<foo>.
+
EOT
}
--
2.6.0.rc2.230.g3dd15c0
next prev parent reply other threads:[~2015-11-04 20:59 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-04 20:58 [PATCH v3 1/4] get_maintainer: add missing documentation for --git-blame-signatures Brian Norris
2015-11-04 20:58 ` [PATCH v3 2/4] get_maintainer: it's '--pattern-depth', not '-pattern-depth' Brian Norris
2015-11-04 20:58 ` Brian Norris [this message]
2015-11-04 20:58 ` [PATCH v3 4/4] get_maintainer: --r (list reviewer) is on by default Brian Norris
2015-11-04 21:13 ` [PATCH v3 1/4] get_maintainer: add missing documentation for --git-blame-signatures Joe Perches
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1446670708-122507-3-git-send-email-computersforpeace@gmail.com \
--to=computersforpeace@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=joe@perches.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox