diff options
| author | Kevin Ballard <kevin@sb.org> | 2010-03-29 19:46:38 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2010-03-29 21:22:37 -0700 |
| commit | 657ab61efa918f46f1b412613643d1a1bbed2194 (patch) | |
| tree | 5667119bfd878d0918ccdb5efdc6582241881c64 | |
| parent | e07665e524d0398a34059d605f674182bfeeaf39 (diff) | |
| download | git-657ab61efa918f46f1b412613643d1a1bbed2194.tar.gz git-657ab61efa918f46f1b412613643d1a1bbed2194.tar.xz | |
format-patch: Squelch 'fatal: Not a range." error
Don't output an error on `git format-patch --ignore-if-in-upstream HEAD`.
This matches the behavior of `git format-patch HEAD`.
Signed-off-by: Kevin Ballard <kevin@sb.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin-log.c | 9 | ||||
| -rwxr-xr-x | t/t4014-format-patch.sh | 4 |
2 files changed, 12 insertions, 1 deletions
diff --git a/builtin-log.c b/builtin-log.c index e0d5caa61..76962e1b0 100644 --- a/builtin-log.c +++ b/builtin-log.c @@ -1106,8 +1106,15 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix) return 0; } - if (ignore_if_in_upstream) + if (ignore_if_in_upstream) { + /* Don't say anything if head and upstream are the same. */ + if (rev.pending.nr == 2) { + struct object_array_entry *o = rev.pending.objects; + if (hashcmp(o[0].item->sha1, o[1].item->sha1) == 0) + return 0; + } get_patch_ids(&rev, &ids, prefix); + } if (!use_stdout) realstdout = xfdopen(xdup(1), "w"); diff --git a/t/t4014-format-patch.sh b/t/t4014-format-patch.sh index f2a2aaa2b..843ef7f88 100755 --- a/t/t4014-format-patch.sh +++ b/t/t4014-format-patch.sh @@ -557,4 +557,8 @@ test_expect_success 'format-patch -- <path>' ' ! grep "Use .--" error ' +test_expect_success 'format-patch --ignore-if-in-upstream HEAD' ' + git format-patch --ignore-if-in-upstream HEAD +' + test_done |