Subject: Re: [PATCH v2 3/3] Use die_errno() instead of die() when checking syscalls

From: Thomas Rast <[email-masked]>
Date: Sun, 7 Jun 2009 00:27:41 +0200

Johannes Sixt wrote:
> > @@ -2262,7 +2262,6 @@ int cmd_blame(int argc, const char **argv, const char
> > *prefix)
> >
> >  	if (revs_file && read_ancestry(revs_file))
> >  		die_errno("reading graft file '%s' failed", revs_file);
> > -
> >  	if (cmd_is_annotate) {
> >  		output_option |= OUTPUT_ANNOTATE_COMPAT;
> >  		blame_date_mode = DATE_ISO8601;
> 
> Unrelated and not an improvement.

I used an Emacs macro to turn the die(..., strerror(errno)) [that I
had from v1] into die_errno, and obviously something went terribly
wrong _and_ I missed it.  Sorry :-(

I'll make a new version tomorrow.

-- 
Thomas Rast
trast@{inf,student}.ethz.ch