Re: Fwd: Idea: add --squash to cherry-pick

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

 



Thank Hannes and Junio for your suggestions!

Another issue with using "git diff A...B | git apply --3way", besides
it not showing the commit messages, is that I wouldn't be able to use
a graphical merge tool that shows the entire source file. When I use
"git cherry-pick" and there are conflicts, I see the change in the
context of the entire file.

(I'm using this modification I made of the meld diff tool, which shows
4 panes, and helps me understand the cause of the conflict:
https://github.com/noamraph/meld)

My main problem with using "rebase -i" is that it would require me to
fix merge conflicts one by one, on each commit in which they appear,
instead of fixing all conflicts at once, treating the change from A to
B as one. It also requires manual editing for every commit between A
and B.

I think that the best way to do what I want using the existing commands is:

git checkout A
git merge --squash B
git commit --no-edit
git checkout @{2}  # Go back to where we were at the beginning. This
is not exact, as you're in detached HEAD state.
git cherry-pick --edit @{1}  # cherry-pick the squashed commit A..B

This allows you to fix the merge conflicts in one go, shows the entire
files causing the conflicts, and allows you to edit the commit
message, starting with the descriptions of all the squashed commits.

I think this also gives a pretty good explanation of what "cherry-pick
--squash" will do: it really is the analog of the "merge --squash",
but for cherry-pick.

What do you think? Did I explain my need to "cherry-pick --squash" and
explain its behavior?

If I do an experiment and try to implement this, would this help?

Thanks,
Noam

On Thu, Aug 11, 2022 at 7:43 PM Junio C Hamano <gitster@xxxxxxxxx> wrote:
>
> Johannes Sixt <j6t@xxxxxxxx> writes:
>
> > That question on Stackoverflow asks "how to do X with Y". But Y (git
> > cherry-pick) is the wrong tool to do X (apply commits from a branch to
> > somewhere else in squashed form).
> >
> >    git diff A...B | git apply --3way
> >
> > would do what you want. You would have to come up with a new commit
> > message anyway, so cherry-pick would be of little use there.
>
> I do agree that cherry-pick is not the best solution for that XY
> problem, but in a typical undisciplined development work, it is not
> entirely implausible that pieces from original commit message may
> still be of some use, exactly the same way as people would want to
> use "squash" while "rebase -i".  Most commits may have completely
> unusable "oops fix", "worked more", etc. messages, but some of them
> may summarize what the past handful steps with such garbage messages
> have done, which may be worth salvaging.
>
> If I were solving the XY problem, I would probably use "rebase -i".
>
> To transplant the range A..B of the history on top of HEAD, for
> example, I'd start with (notice ^0 after B, as I do not trust myself
> so I'd leave the true branch B untouched as I may make mistakes
> while running rebase):
>
>     $ git checkout --detach HEAD        ;# this is only to use @{-1} later
>     $ git rebase -i --onto HEAD A B^0
>
> Then if my goal is to squash everything down into a single commit,
> then replace all 'pick', except for the first one, to 'squash'.
> That will give me one single chance to edit a single commit message,
> but the editor buffer starts with the log message from all of the
> original, so I can pick good bits from them while writing new stuf.
>
> I'll have the result on detached HEAD.  If I like the result, I may
> update the branch I was originally on with it.
>
>     $ git checkout -B @{-1}
>
> Or, if I don't, perhaps because I made mistakes, then I can just
> discard it and go back to the original branch.
>
>     $ git checkout @{-1}
>



[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]

  Powered by Linux