Re: [PATCH] sequencer: update abort safety file more sparingly

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

 



On Sun, Sep 03, 2023 at 07:40:00PM +0100, Phillip Wood wrote:
On 03/09/2023 16:11, Oswald Buddenhagen wrote:
The only situation where the file's content matters is --continue'ing
(after a multi-cherry-pick merge conflict).

I don't think "cherry-pick --continue" consults the abort safety file,
duh, obvious blunder.

it only matters for "cherry-pick --skip"

that doesn't seem right. a --skip is just a --continue with a prior reset, more or less.

and "cherry-pick --abort".

that one, of course.

This means that it is
sufficient to write it in a single place, when we are prematurely
exiting the main workhorse.

I think this introduces a regression because the safety file will not get updated when "cherry-pick --continue" stops for the user to resolve conflicts.

true, there is indeed this second entry point.
i'll try to find a better "choke point".

which wasn't even reliable: a single pick executed during an interrupted sequence would bypass the safety.

An alternate view is that the abort safety file exists to prevent the user losing commits that have not been cherry-picked and it is desirable to be able to abort after cherry-picking a single pick in the middle of a sequence of cherry-picks.

if you did a fresh commit before or after the single pick, you'd lose it. also, the feature doesn't actually prevent aborting, only the automatic reset.

regards



[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