Ronnie Sahlberg wrote:
> On Wed, May 14, 2014 at 5:04 PM, Jonathan Nieder <> wrote:

>> If it were ever triggered, the message
>>         error: some bad thing
>>         fatal: failed transaction create for refs/heads/master
>> looks overly verbose and unclear.  Something like
>>         fatal: cannot create ref refs/heads/master: some bad thing
> I changed it to :
>    die("cannot create ref '%s'", refname);
> But it would still mean you would have
>          error: some bad thing
>          fatal: cannot create 'refs/heads/master'
> To make it better we have to wait until the end of the second patch
> series, ref-transactions-next
> where we will have an err argument to _update/_create/_delete too and
> thus we can do this from update-ref.c :
>    if (transaction_create_sha1(transaction, refname, new_sha1,
>        update_flags, msg, &err))
>    die("%s", err.buf);

Thanks.  Sounds good.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to