Re: [PATCH v3 1/2] am: handle stray $dotest directory

2013-06-18 Thread Ramkumar Ramachandra
Junio C Hamano wrote:
>> Where did the error occur?
>
> I think you can compare what is in 'pu' and what you sent out
> without asking.

Ah, I sent out the series before I taught Emacs to indent properly.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 1/2] am: handle stray $dotest directory

2013-06-18 Thread Junio C Hamano
Ramkumar Ramachandra  writes:

> Junio C Hamano wrote:
>>> Will replace what has been queued on 'pu'.
>>
>> ... after fixing an indentation error, that is.
>
> Where did the error occur?

I think you can compare what is in 'pu' and what you sent out
without asking.

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 1/2] am: handle stray $dotest directory

2013-06-18 Thread Ramkumar Ramachandra
Junio C Hamano wrote:
>> Will replace what has been queued on 'pu'.
>
> ... after fixing an indentation error, that is.

Where did the error occur?  I thought I taught my Emacs to always
indent properly.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 1/2] am: handle stray $dotest directory

2013-06-17 Thread Junio C Hamano
Junio C Hamano  writes:

> Will replace what has been queued on 'pu'.

... after fixing an indentation error, that is.

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 1/2] am: handle stray $dotest directory

2013-06-16 Thread Junio C Hamano
Ramkumar Ramachandra  writes:

>  else
> + # Possible stray $dotest directory in the independent-run
> + # case; in the --rebasing case, it is upto the caller
> + # (git-rebase--am) to take care of stray directories.
> + if test -d "$dotest" && test -z "$rebasing"

The $rebasing variable is set only when the command line of "git am"
has --rebasing, i.e. only when it is driven by "git rebase".  So
this will not affect what happens to "git am --abort" that is run by
the end user.  That sounds like a safe thing to do as far as "am" is
concerned.

Will replace what has been queued on 'pu'.  Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html