revisions: allow --glob and friends in parse_options-enabled commands

As v1.6.0-rc2~42 (2008-07-31) explains, even pseudo-options like --not
and --glob that need to be parsed in order with revisions should be
marked handled by handle_revision_opt to avoid an error when
parse_revision_opt callers like "git shortlog" encounter them.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/revision.c b/revision.c
index c9b1e3223464eaed0155f2d45b9d98a09bbf1a4d..238976466d28a26ffff5221ad985bf976b0a1007 100644 (file)
--- a/revision.c
+++ b/revision.c
@@ -1178,7 +1178,9 @@ static int handle_revision_opt(struct rev_info *revs, int argc, const char **arg
            !strcmp(arg, "--tags") || !strcmp(arg, "--remotes") ||
            !strcmp(arg, "--reflog") || !strcmp(arg, "--not") ||
            !strcmp(arg, "--no-walk") || !strcmp(arg, "--do-walk") ||
            !strcmp(arg, "--tags") || !strcmp(arg, "--remotes") ||
            !strcmp(arg, "--reflog") || !strcmp(arg, "--not") ||
            !strcmp(arg, "--no-walk") || !strcmp(arg, "--do-walk") ||

-           !strcmp(arg, "--bisect"))
+           !strcmp(arg, "--bisect") || !prefixcmp(arg, "--glob=") ||
+           !prefixcmp(arg, "--branches=") || !prefixcmp(arg, "--tags=") ||
+           !prefixcmp(arg, "--remotes="))

        {
                unkv[(*unkc)++] = arg;
                return 1;
        {
                unkv[(*unkc)++] = arg;
                return 1;


@@ -1534,6 +1536,16 @@ static int handle_revision_pseudo_opt(const char *submodule,
        const char *optarg;
        int argcount;
 
        const char *optarg;
        int argcount;
 

+       /*
+        * NOTE!
+        *
+        * Commands like "git shortlog" will not accept the options below
+        * unless parse_revision_opt queues them (as opposed to erroring
+        * out).
+        *
+        * When implementing your new pseudo-option, remember to
+        * register it in the list at the top of handle_revision_opt.
+        */

        if (!strcmp(arg, "--all")) {
                handle_refs(submodule, revs, *flags, for_each_ref_submodule);
                handle_refs(submodule, revs, *flags, head_ref_submodule);
        if (!strcmp(arg, "--all")) {
                handle_refs(submodule, revs, *flags, for_each_ref_submodule);
                handle_refs(submodule, revs, *flags, head_ref_submodule);

diff --git a/t/t6018-rev-list-glob.sh b/t/t6018-rev-list-glob.sh
index fb8291c812854608777dd8d35edfcda78df5069a..f00cebff3e42cf0a38d06a03b6f14ef08dd16372 100755 (executable)
--- a/t/t6018-rev-list-glob.sh
+++ b/t/t6018-rev-list-glob.sh
@@ -69,6 +69,18 @@ test_expect_success 'rev-parse --glob=heads/subspace' '
 
 '
 
 
 '
 

+test_expect_failure 'rev-parse accepts --glob as detached option' '
+
+       compare rev-parse "subspace/one subspace/two" "--glob heads/subspace"
+
+'
+
+test_expect_failure 'rev-parse is not confused by option-like glob' '
+
+       compare rev-parse "master" "--glob --symbolic master"
+
+'
+

 test_expect_success 'rev-parse --branches=subspace/*' '
 
        compare rev-parse "subspace/one subspace/two" "--branches=subspace/*"
 test_expect_success 'rev-parse --branches=subspace/*' '
 
        compare rev-parse "subspace/one subspace/two" "--branches=subspace/*"


@@ -129,6 +141,12 @@ test_expect_success 'rev-list --glob refs/heads/subspace/*' '
 
 '
 
 
 '
 

+test_expect_success 'rev-list not confused by option-like --glob arg' '
+
+       compare rev-list "master" "--glob -0 master"
+
+'
+

 test_expect_success 'rev-list --glob=heads/subspace/*' '
 
        compare rev-list "subspace/one subspace/two" "--glob=heads/subspace/*"
 test_expect_success 'rev-list --glob=heads/subspace/*' '
 
        compare rev-list "subspace/one subspace/two" "--glob=heads/subspace/*"


@@ -213,4 +231,36 @@ test_expect_success 'rev-list --remotes=foo' '
 
 '
 
 
 '
 

+test_expect_success 'shortlog accepts --glob/--tags/--remotes' '
+
+       compare shortlog "subspace/one subspace/two" --branches=subspace &&
+       compare shortlog \
+         "master subspace-x someref other/three subspace/one subspace/two" \
+         --branches &&
+       compare shortlog master "--glob=heads/someref/* master" &&
+       compare shortlog "subspace/one subspace/two other/three" \
+         "--glob=heads/subspace/* --glob=heads/other/*" &&
+       compare shortlog \
+         "master other/three someref subspace-x subspace/one subspace/two" \
+         "--glob=heads/*" &&
+       compare shortlog foo/bar --tags=foo &&
+       compare shortlog foo/bar --tags &&
+       compare shortlog foo/baz --remotes=foo
+
+'
+
+test_expect_failure 'shortlog accepts --glob as detached option' '
+
+       compare shortlog \
+         "master other/three someref subspace-x subspace/one subspace/two" \
+         "--glob heads/*"
+
+'
+
+test_expect_failure 'shortlog --glob is not confused by option-like argument' '
+
+       compare shortlog master "--glob -e master"
+
+'
+

 test_done
 test_done