Re: [PATCH v3 6/8] rebase: add --update-refs option

From: Derrick Stolee <derrickstolee@github.com>
Date: Jun 29, 2022, 13:03 UTC
Message-ID: <8e55693e-dedf-fb42-d31a-33ebbdd6abd6@github.com>
In-Reply-To: <xmqqbkucijg3.fsf@gitster.g>

On 6/28/22 5:09 PM, Junio C Hamano wrote:

> "Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes:

...

Show 12 quoted lines

>> This change update the documentation and builtin to accept the
>> --update-refs option as well as updating the todo file with the
>> 'update-ref' commands. Tests are added to ensure that these todo
>> commands are added in the correct locations.
> 
> OK, so command line option is `--update-refs` because the range
> being rebased could contain more than one branch tips, and the
> option causes one `update-ref` insn per such ref left in the todo
> list?  I had to read the above twice to remind me what was going on.
> 
> The design looks OK here but has "rebase-merge/update-refs" already
> came into the picture, or will it be in future steps?  I lost track.

Sorry, we have the ability to read the file (and tests that read an artificial version of the file for the branch_checked_out() logic) but have not written it from within Git yet.

I can make it clearer what is _not_ in this patch and coming in a future change.

Thanks, -Stolee