Skip to content

Commit

Permalink
revision.c: support --notes command-line option
Browse files Browse the repository at this point in the history
We already have --show-notes, but it has a few shortcomings:

  1. Using --show-notes=<ref> implies that we should also
     show the default notes. Which means you also need to
     use --no-standard-notes if you want to suppress them.

  2. It is negated by --no-notes, which doesn't match.

  3. It's too long to type. :)

This patch introduces --notes, which behaves exactly like
--show-notes, except that using "--notes=<ref>" does not
imply showing the default notes.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Jeff King authored and Junio C Hamano committed Mar 29, 2011
1 parent 3a03cf6 commit 7249e91
Show file tree
Hide file tree
Showing 2 changed files with 32 additions and 5 deletions.
15 changes: 10 additions & 5 deletions revision.c
Original file line number Diff line number Diff line change
Expand Up @@ -1367,17 +1367,22 @@ static int handle_revision_opt(struct rev_info *revs, int argc, const char **arg
revs->verbose_header = 1;
revs->pretty_given = 1;
get_commit_format(arg+9, revs);
} else if (!strcmp(arg, "--show-notes")) {
} else if (!strcmp(arg, "--show-notes") || !strcmp(arg, "--notes")) {
revs->show_notes = 1;
revs->show_notes_given = 1;
revs->notes_opt.use_default_notes = 1;
} else if (!prefixcmp(arg, "--show-notes=")) {
} else if (!prefixcmp(arg, "--show-notes=") ||
!prefixcmp(arg, "--notes=")) {
struct strbuf buf = STRBUF_INIT;
revs->show_notes = 1;
revs->show_notes_given = 1;
if (revs->notes_opt.use_default_notes < 0)
revs->notes_opt.use_default_notes = 1;
strbuf_addstr(&buf, arg+13);
if (!prefixcmp(arg, "--show-notes")) {
if (revs->notes_opt.use_default_notes < 0)
revs->notes_opt.use_default_notes = 1;
strbuf_addstr(&buf, arg+13);
}
else
strbuf_addstr(&buf, arg+8);
expand_notes_ref(&buf);
string_list_append(&revs->notes_opt.extra_notes_refs,
strbuf_detach(&buf, NULL));
Expand Down
22 changes: 22 additions & 0 deletions t/t3301-notes.sh
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,28 @@ do
'
done

test_expect_success 'setup alternate notes ref' '
git notes --ref=alternate add -m alternate
'

test_expect_success 'git log --notes shows default notes' '
git log -1 --notes >output &&
grep xyzzy output &&
! grep alternate output
'

test_expect_success 'git log --notes=X shows only X' '
git log -1 --notes=alternate >output &&
! grep xyzzy output &&
grep alternate output
'

test_expect_success 'git log --notes --notes=X shows both' '
git log -1 --notes --notes=alternate >output &&
grep xyzzy output &&
grep alternate output
'

test_expect_success 'create -m notes (setup)' '
: > a5 &&
git add a5 &&
Expand Down

0 comments on commit 7249e91

Please sign in to comment.