Re: [PATCH v2 0/6] Fix checkout-dash to work with rebase

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ramkumar Ramachandra wrote:
> Thanks for the reorder and commit message tweaks.  I'm working on the
> series you put up on `pu` now.

That said, I do not agree with one of your commit message updates:

    checkout: respect GIT_REFLOG_ACTION

    GIT_REFLOG_ACTION is an environment variable specifying the reflog
    message to write after an action is completed.  Several other commands
    including merge, reset, and commit respect it.

    Fix the failing tests in t/checkout-last by making checkout respect it
    too.  You can now expect

      $ git checkout -

    to work as expected after any operation that internally uses "checkout"
    as its implementation detail, e.g. "rebase".

After the patch you _cannot_ expect checkout-dash to work after any
operation that internally uses checkout; it's limited to rebase.  Many
other scripts (eg. bisect) do _not_ set GIT_REFLOG_ACTION at all.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]