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

[PATCH 5/5] log: parse separate option for --glob

From
Matthieu Moy <matthieu.moy@imag.fr>
Date
Aug 5, 2010, 08:22 UTC
Message-ID
<1280996576-1165-5-git-send-email-Matthieu.Moy@imag.fr>
In-Reply-To
<vpqmxt14gjt.fsf@bauges.imag.fr>
Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
---
 revision.c               |    7 +++++--
 t/t6018-rev-list-glob.sh |    6 ++++++
 2 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/revision.c b/revision.c
index 489a3c2..f241f34 100644
--- a/revision.c
+++ b/revision.c
@@ -1484,6 +1484,8 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, struct s
 {
 	int i, flags, left, seen_dashdash, read_from_stdin, got_rev_arg = 0;
 	const char **prune_data = NULL;
+	const char *optarg;
+	int argcount;
 
 	/* First, search for "--" */
 	seen_dashdash = 0;
@@ -1530,10 +1532,11 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, struct s
 				handle_refs(revs, flags, for_each_remote_ref);
 				continue;
 			}
-			if (!prefixcmp(arg, "--glob=")) {
+			if ((argcount = parse_long_opt("glob", argv + i, &optarg))) {
 				struct all_refs_cb cb;
+				i += argcount - 1;
 				init_all_refs_cb(&cb, revs, flags);
-				for_each_glob_ref(handle_one_ref, arg + 7, &cb);
+				for_each_glob_ref(handle_one_ref, optarg, &cb);
 				continue;
 			}
 			if (!prefixcmp(arg, "--branches=")) {
diff --git a/t/t6018-rev-list-glob.sh b/t/t6018-rev-list-glob.sh
index 58428d9..fb8291c 100755
--- a/t/t6018-rev-list-glob.sh
+++ b/t/t6018-rev-list-glob.sh
@@ -123,6 +123,12 @@ test_expect_success 'rev-list --glob=refs/heads/subspace/*' '
 
 '
 
+test_expect_success 'rev-list --glob refs/heads/subspace/*' '
+
+	compare rev-list "subspace/one subspace/two" "--glob refs/heads/subspace/*"
+
+'
+
 test_expect_success 'rev-list --glob=heads/subspace/*' '
 
 	compare rev-list "subspace/one subspace/two" "--glob=heads/subspace/*"
-- 
1.7.2.1.30.g18195
Previous: Matthieu MoyNext: Jonathan Nieder
Message 25 of 27 in “What's cooking in git.git (Aug 2010, #01; Wed, 4)”
  1. Junio C HamanoAug 4, 2010
  2. jk/tag-contains: stalledTed Ts'o, Aug 5, 2010
  3. Junio C HamanoAug 5, 2010
  4. Junio C HamanoAug 5, 2010
  5. Jeff KingAug 5, 2010
  6. Junio C HamanoAug 5, 2010
  7. Jeff KingAug 5, 2010
  8. Jay SoffianAug 5, 2010
  9. Jeff KingAug 5, 2010
  10. Jay SoffianAug 5, 2010
  11. Ted Ts'oAug 5, 2010
  12. Junio C HamanoAug 5, 2010
  13. Thomas RastAug 5, 2010
  14. Junio C HamanoAug 5, 2010
  15. Junio C HamanoAug 6, 2010
  16. tc/checkout-BJonathan Nieder, Aug 5, 2010
  17. Tay Ray ChuanAug 5, 2010
  18. Matthieu MoyAug 5, 2010
  19. 1/5 diff: parse separate options like -S fooMatthieu Moy, Aug 5, 2010
  20. Jakub NarebskiAug 5, 2010
  21. Matthieu MoyAug 5, 2010
  22. 2/5 diff: split off a function for --stat-* option parsingMatthieu Moy, Aug 5, 2010
  23. 3/5 diff: parse separate options --stat-width n, --stat-name-width nMatthieu Moy, Aug 5, 2010
  24. 4/5 log: parse separate options like git log --grep fooMatthieu Moy, Aug 5, 2010
  25. 5/5 log: parse separate option for --globMatthieu Moy, Aug 5, 2010
  26. mm/shortopt-detachedJonathan Nieder, Aug 5, 2010
  27. Dmitry V. LevinAug 5, 2010

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.