diff options
author | Junio C Hamano <gitster@pobox.com> | 2014-09-29 21:36:10 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-09-29 21:36:10 +0200 |
commit | 507fe835ed0753ff266fdb998853cc5dea3758c6 (patch) | |
tree | 06a7d355d50bea90bf2a3859c11e9d9a5b633b6f /refs.c | |
parent | Merge branch 'hj/pretty-naked-decoration' (diff) | |
parent | stash: prefer --quiet over shell redirection of the standard error stream (diff) | |
download | git-507fe835ed0753ff266fdb998853cc5dea3758c6.tar.xz git-507fe835ed0753ff266fdb998853cc5dea3758c6.zip |
Merge branch 'da/rev-parse-verify-quiet'
"rev-parse --verify --quiet $name" is meant to quietly exit with a
non-zero status when $name is not a valid object name, but still
gave error messages in some cases.
* da/rev-parse-verify-quiet:
stash: prefer --quiet over shell redirection of the standard error stream
refs: make rev-parse --quiet actually quiet
t1503: use test_must_be_empty
Documentation: a note about stdout for git rev-parse --verify --quiet
Diffstat (limited to 'refs.c')
-rw-r--r-- | refs.c | 10 |
1 files changed, 7 insertions, 3 deletions
@@ -3159,7 +3159,7 @@ static int read_ref_at_ent_oldest(unsigned char *osha1, unsigned char *nsha1, return 1; } -int read_ref_at(const char *refname, unsigned long at_time, int cnt, +int read_ref_at(const char *refname, unsigned int flags, unsigned long at_time, int cnt, unsigned char *sha1, char **msg, unsigned long *cutoff_time, int *cutoff_tz, int *cutoff_cnt) { @@ -3177,8 +3177,12 @@ int read_ref_at(const char *refname, unsigned long at_time, int cnt, for_each_reflog_ent_reverse(refname, read_ref_at_ent, &cb); - if (!cb.reccnt) - die("Log for %s is empty.", refname); + if (!cb.reccnt) { + if (flags & GET_SHA1_QUIETLY) + exit(128); + else + die("Log for %s is empty.", refname); + } if (cb.found_it) return 0; |