git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v5 6/7] replace: allow long option names

From
Christian Couder <chriscool@tuxfamily.org>
Date
Sep 6, 2013, 05:10 UTC
Message-ID
<20130906051100.6657.18553.chriscool@tuxfamily.org>
In-Reply-To
<20130906050702.6657.25651.chriscool@tuxfamily.org>

It is now standard practice in Git to have both short and long option names. So let's give a long option name to the git replace options too.

Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
---
 Documentation/git-replace.txt | 3 +++
 builtin/replace.c             | 6 +++---
 2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/Documentation/git-replace.txt b/Documentation/git-replace.txt
index a2bd2ee..414000e 100644
--- a/Documentation/git-replace.txt
+++ b/Documentation/git-replace.txt
@@ -54,13 +54,16 @@ achieve the same effect as the `--no-replace-objects` option.
 OPTIONS
 -------
 -f::
+--force::
 	If an existing replace ref for the same object exists, it will
 	be overwritten (instead of failing).
 
 -d::
+--delete::
 	Delete existing replace refs for the given objects.
 
 -l <pattern>::
+--list <pattern>::
 	List replace refs for objects that match the given pattern (or
 	all if no pattern is given).
 	Typing "git replace" without arguments, also lists all replace
diff --git a/builtin/replace.c b/builtin/replace.c
index 95736d9..d4d1b75 100644
--- a/builtin/replace.c
+++ b/builtin/replace.c
@@ -128,9 +128,9 @@ int cmd_replace(int argc, const char **argv, const char *prefix)
 {
 	int list = 0, delete = 0, force = 0;
 	struct option options[] = {
-		OPT_BOOLEAN('l', NULL, &list, N_("list replace refs")),
-		OPT_BOOLEAN('d', NULL, &delete, N_("delete replace refs")),
-		OPT_BOOLEAN('f', NULL, &force, N_("replace the ref if it exists")),
+		OPT_BOOLEAN('l', "list", &list, N_("list replace refs")),
+		OPT_BOOLEAN('d', "delete", &delete, N_("delete replace refs")),
+		OPT_BOOLEAN('f', "force", &force, N_("replace the ref if it exists")),
 		OPT_END()
 	};
 
-- 
1.8.4.rc1.28.ge2684af
Previous: Christian CouderNext: Christian Couder
Message 9 of 10 in “Check replacement object type and minor updates”
  1. 0/7 Check replacement object type and minor updatesChristian Couder, Sep 6, 2013
  2. 1/7 replace: forbid replacing an object with one of a different typeChristian Couder, Sep 6, 2013
  3. Junio C HamanoSep 6, 2013
  4. Junio C HamanoSep 6, 2013
  5. 2/7 Documentation/replace: state that objects must be of the same typeChristian Couder, Sep 6, 2013
  6. 3/7 t6050-replace: test that objects are of the same typeChristian Couder, Sep 6, 2013
  7. 4/7 t6050-replace: add test to clean up all the replace refsChristian Couder, Sep 6, 2013
  8. 5/7 Documentation/replace: add Creating Replacement Objects sectionChristian Couder, Sep 6, 2013
  9. 6/7 replace: allow long option namesChristian Couder, Sep 6, 2013
  10. 7/7 t6050-replace: use some long option namesChristian Couder, Sep 6, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.