send-pack: avoid redundant "pack-objects died with strange error"
commit64f003abd9fbb155a6331365c0783b926185882c
authorJonathan Nieder <jrnieder@gmail.com>
Sat, 16 Oct 2010 17:09:54 +0000 (16 12:09 -0500)
committerJunio C Hamano <gitster@pobox.com>
Mon, 18 Oct 2010 23:12:09 +0000 (18 16:12 -0700)
treee27e3e4adc4e90d869036c17f5e65dfb1f3ab328
parent7c6eafa35af805578990e76b691ff7f1a25a3c57
send-pack: avoid redundant "pack-objects died with strange error"

Saying "pack-objects died with strange error" after "pack-objects died
of signal 13" seems kind of redundant.  The latter message was
introduced when the run-command API changed to report abnormal exits
on behalf of the caller (v1.6.5-rc0~86^2~5, 2009-07-04).

Similarly, after a controlled pack-objects failure (detectable as a
normal exit with nonzero status), a "died with strange error" message
would be redundant next to the message from pack-objects itself.

So leave off the "strange error" messages.

The result should look something like this:

$ git push sf master
Counting objects: 21542, done.
Compressing objects: 100% (4179/4179), done.
fatal: Unable to create temporary file: Permission denied
error: pack-objects died of signal 13
error: failed to push some refs to 'ssh://sf.net/gitroot/project/project'
$

Or in the "controlled exit" case (contrived example):

[...]
fatal: delta size changed
error: failed to push some refs to 'ssh://example.com/foo/bar'
$

Improved-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/send-pack.c