[FFmpeg-devel] [PATCH] MAINTAINERS: add a separate list for those with push access

[Lynne]

Dec 15, 2022, 20:34 by michael at niedermayer.cc:

> On Thu, Dec 15, 2022 at 02:13:49AM +0100, Lynne wrote:
>
>> This list is incomplete, and just contains those I could see
>> while looking at the recent git log. If it looks like I've forgotten you, I definitely haven't!
>> We may complete the list at a later date.
>>
>> This makes it such that those who add themselves to MAINTAINERS do not
>> get push access by default, but rather, they have to request it
>> explicitly in a different commit. This used to be the situation
>> before it was changed at the start of this year and is pretty much what
>> everyone expects.
>>
>> Patch attached.
>>
>> MAINTAINERS |   15 +++++++++++++++
>>  1 file changed, 15 insertions(+)
>> 6a083061d75f6655771bde377f96aadad19b21c6  0001-MAINTAINERS-add-a-separate-list-for-those-with-push-.patch
>> From 5c353412a25fd46c5077e5cf92ddfd6532eb46cb Mon Sep 17 00:00:00 2001
>> From: Lynne <dev at lynne.ee>
>> Date: Thu, 15 Dec 2022 02:05:00 +0100
>> Subject: [PATCH] MAINTAINERS: add a separate list for those with push access
>>
>> This list is incomplete, and just contains those I could remember
>> while looking at the recent git log.
>> We may complete the list at a later date.
>>
>> This makes it such that those who add themselves to MAINTAINERS do not
>> get push access by default, but rather, they have to request it
>> explicitly in a different commit.
>>
>> This used to be the situation
>> before it was changed at the start of this year.
>>
>
> I remember no such change.
> What i do remember is really long ago trying to push people toward pushing in
> their own repository and sending pull requests similar to the kernel. But this
> was not popular so i droped the idea.
>
> Whereever code is maintained teh maintainer should have write access to that
> place other things become inconvenient quickly.
>
> maintainers who cannot change the code they maintain should stay an exception
>

This is exactly what changed. Before, maintainers who didn't get push
access was the norm, not the standard.

Regardless, if you agree with the patch, I see no reason to continue discussing this.