Re: What's cooking in git.git (Aug 2021, #06; Mon, 16)

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 17, 2021, 16:47 UTC
Message-ID: <xmqqh7fonhlo.fsf@gitster.g>
In-Reply-To: <92511bbe-5ef3-cfef-0a0b-30fbda2df201@gmail.com>

Phillip Wood <phillip.wood123@gmail.com> writes:

Show 12 quoted lines

>> * pw/rebase-skip-final-fix (2021-08-13) 3 commits
>>   - rebase --continue: remove .git/MERGE_MSG
>>   - rebase --apply: restore some tests
>>   - t3403: fix commit authorship
>>   Checking out all the paths from HEAD during the last conflicted
>>   step in "git rebase" and continuing would cause the step to be
>>   skipped (which is expected), but leaves MERGE_HEAD file behind in
>>   $GIT_DIR and confuses the next "git commit", which has been
>>   corrected.
>>   Will merge to 'next'.
>
> Thanks

Thank -you- for the series. You really do not need to comment on your topic only to say that. But if you need to _stop_ me from merging a topic to 'next' (because you want to replace it one more time, or you see something wrong with it), please don't hesitate to.

Show 11 quoted lines

>> * lh/systemd-timers (2021-07-02) 3 commits
>>   - maintenance: add support for systemd timers on Linux
>>   - maintenance: `git maintenance run` learned `--scheduler=<scheduler>`
>>   - cache.h: Introduce a generic "xdg_config_home_for(…)" function
>>   "git maintenance" scheduler learned to use systemd timers as a
>>   possible backend.
>>   Waiting for reviews.
>
> I'd really like to see this merged, there was some discussion a couple
> of weeks ago (cf <4aed0293-6a48-d370-3b72-496b7c631cb5@gmail.com>), it 
> petered out but no one seemed to be objecting to merging it.

Thanks for paying attention to this item.

My impression was that everybody likes the ability to use systemd timers as an alternative to cron, but as Derrick summarised in the message you pointed at, I think the implementation is not quite there.

So "Waiting for reviews" is a mislabel; we are waiting for a reroll.

Show 8 quoted lines

>> * zh/cherry-pick-advice (2021-08-14) 1 commit
>>   - cherry-pick: use better advice message
>>   The advice message that "git cherry-pick" gives when it asks
>>   conflicted replay of a commit to be resolved by the end user has
>>   been updated.
>
> I think this is getting there now, I need to look at the tests in v5
> but the changes to sequencer.c looked good to me.

Yes.

Show 8 quoted lines

>> * jk/commit-edit-fixup-fix (2021-08-15) 1 commit
>>   - commit: restore --edit when combined with --fixup
>>   "git commit --fixup" now works with "--edit" again, after it was
>>   broken in v2.32.
>>   Will merge to 'next'?
>
> Yes please, there was a new version posted at the weekend but I've
> just checked and you've already picked it up in seen.

Yes, this looked good to me, too.

Thanks.