Re: [PATCH 0/3] Maintenance IV: Platform-specific background maintenance

From: Derrick Stolee <stolee@gmail.com>
Date: Nov 4, 2020, 13:02 UTC
Message-ID: <af2befd7-7d93-5981-42c8-700e49f3999a@gmail.com>
In-Reply-To: <xmqqo8keoyy8.fsf@gitster.c.googlers.com>

On 11/3/2020 5:30 PM, Junio C Hamano wrote:

Show 16 quoted lines

> Derrick Stolee <stolee@gmail.com> writes:
>> I'm hoping that parts 3 and 4 can make it in time for 2.30,
>> so the feature is universally available for all platforms.
>> I realize that's not entirely up to just me.
> 
> After writing this entry in the What's cooking report:
> 
>     * ds/maintenance-part-4 (2020-11-03) 3 commits
>      - maintenance: use Windows scheduled tasks
>      - maintenance: use launchctl on macOS
>      - maintenance: extract platform-specific scheduling
>      (this branch uses ds/maintenance-part-3.)
> 
>      Follow-up on the "maintenance part-3" which introduced scheduled
>      maintenance tasks to support platforms whose native scheduling
>      methods are not 'cron'.

This is a good summary of this series.

> I was wondering if I should propose making these two parts into one,
> so we may be pretty much on the same page.

I'm happy to have the two topics be merged into one series, but I'd prefer to only re-roll these three patches during review. Of course, by keeping them together we have the ability to re- roll all of the patches, but part-3 has a decent length and has been stable for a while.

I defer to what is easiest for you.

Thanks, -Stolee