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

[Lynne]

Oct 24, 2022, 18:29 by jamrial at gmail.com:

> On 10/24/2022 12:56 PM, Lynne wrote:
>
>> Oct 24, 2022, 09:42 by d.kozinski at samsung.com:
>>
>>> - Changelog update
>>> - MAINTAINERS update
>>>
>>> Signed-off-by: Dawid Kozinski <d.kozinski at samsung.com>
>>> ---
>>>  Changelog   | 3 ++-
>>>  MAINTAINERS | 5 +++++
>>>  2 files changed, 7 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/Changelog b/Changelog
>>> index ec9de1bd85..19e9ae3b1f 100644
>>> --- a/Changelog
>>> +++ b/Changelog
>>> @@ -45,6 +45,8 @@ version 5.1:
>>>  - remap_opencl filter
>>>  - added chromakey_cuda filter
>>>  - added bilateral_cuda filter
>>> +- eXtra-fast Essential Video Encoder (XEVE)
>>> +- eXtra-fast Essential Video Decoder (XEVD)
>>>  version 5.0:
>>> @@ -92,7 +94,6 @@ version 5.0:
>>>  - anlmf audio filter
>>>  - IMF demuxer (experimental)
>>>  -
>>>  version 4.4:
>>>  - AudioToolbox output device
>>>  - MacCaption demuxer
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index eebfa5cfb7..df8d8eca73 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -200,6 +200,8 @@ Codecs:
>>>  libvpx*                               James Zern
>>>  libxavs.c                             Stefan Gehrer
>>>  libxavs2.c                            Huiwen Ren
>>> +  libxevd.c                             Dawid Kozinski
>>> +  libxeve.c,                            Dawid Kozinski
>>>  libzvbi-teletextdec.c                 Marton Balint
>>>  lzo.h, lzo.c                          Reimar Doeffinger
>>>  mdec.c                                Michael Niedermayer
>>> @@ -420,6 +422,9 @@ Muxers/Demuxers:
>>>  dv.c                                  Roman Shaposhnik
>>>  electronicarts.c                      Peter Ross
>>>  epafdec.c                             Paul B Mahol
>>> +  evc.c, evc.h                          Dawid Kozinski
>>> +  evcdec.c                              Dawid Kozinski
>>> +  evc_parser.c                          Dawid Kozinski
>>>  ffm*                                  Baptiste Coudurier
>>>  flic.c                                Mike Melanson
>>>  flvdec.c                              Michael Niedermayer
>>>
>>
>> Nak, that list is only for those with push access, and no
>> other changes may be made in the same patch.
>>
>
> No, it's the other way around. Those in this list may be eligible for push access.
> Being listed here gives them the right to NAK a patch made for a module they maintain, as well as their approval being (ideally) a requirement before making changes to it.
>

Nope. Michael will give anyone on the list push access.
That's why I insisted on a rule to separate out such patches to avoid sneaking in.
The whole problem would be solved by having 2 lists, but instead, users
have to explicitly ask not to get push access if they want to get added to maintainers.