[FFmpeg-devel] [PATCH v14 9/9] avcodec/evc: Changes in Changelog and MAINTAINERS files

[Lynne]

Dec 14, 2022, 22:45 by michael at niedermayer.cc:

> On Tue, Dec 13, 2022 at 07:42:23PM +0100, Lynne wrote:
>
>> Dec 13, 2022, 13:22 by d.kozinski at samsung.com:
>>
>> > We made some changes in our EVC wrapper implementation and would like to
>> > submit new patches to patchwork, but it's still unclear to me how to deal
>> > with the MAINTAINERS file. 
>> >
>> > Should I leave the following lines:
>> > +  libxevd.c                             Dawid Kozinski
>> > +  libxeve.c,                            Dawid Kozinski
>> > +  evc.c, evc.h                        Dawid Kozinski
>> > +  evcdec.c                             Dawid Kozinski
>> > +  evc_parser.c                      Dawid Kozinski
>> >
>> > or should I remove them?
>> >
>> > We are expecting a clear and consistent standpoint on this matter.
>> >
>>
>> Get rid of them. Michael has made it clear anyone on the list
>> must have commit rights. I'm not for blocking anyone from having
>> commit rights, but you've made zero contributions to the project
>> so far, and maintaining requires some level of dedication.
>>
>
> You surely have the right to object to Dawid having commit rights.
> And i understand your argument why, but that has nothing to do with
> me or the list. There are people on the MAINTAINERS list who do not
> have commit rights.
> for example the developers who i presume are paid by loongson to maintain 
> the mips/loongson code do not currently have commit right.
> Seems a similar case to me
>

You're making assumptions. You've said that anyone who's on maintainers
needs push rights. Then you said that unless someone explicitly asks not to
get push rights, they will if they're added to maintainers. Right now, the
person hasn't said anything, yet you're assuming he doesn't want push rights.

I've sent a patch to list maintainers with push access. Those who'd like
push access will have to explicitly request it in a different commit.
If that patch is accepted, I have no problem with this patch.