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

[Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics]

-----Original Message-----
From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Michael
Niedermayer
Sent: środa, 14 grudnia 2022 22:36
To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v14 9/9] avcodec/evc: Changes in
Changelog and MAINTAINERS files

On Tue, Dec 13, 2022 at 08:33:29AM -0500, Ronald S. Bultje wrote:
> Hi David,
> 
> On Tue, Dec 13, 2022 at 7:22 AM Dawid Kozinski/Multimedia (PLT) 
> /SRPOL/Staff Engineer/Samsung Electronics <d.kozinski at samsung.com> wrote:
> 
> > 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?
> >
> 
> Here's a question for you, and the answer probably becomes 
> self-evident from that. If you, Dawid, stop working for Samsung, for 
> example because you're starting your own business or Samsung fires you 
> or Google hires you, or if Samsung stops sponsoring this new codec 
> called "EVC" or stops contributing to this new library "libxeve". Will 
> you, Dawid, still maintain these files?
> 
> If the answer is yes, then you can shorten these lines ("evc*.[ch]: Dawid"
> & "livxev[ed].c: Dawid") and keep them.
> 
> If the answer is no, then I think you should remove (or adjust) these 
> lines, since they are (in their current form) inaccurate: you are not 
> maintaining these files, your company is.
> 

I think for code maintained by a company we still should list a person
because persons can be contacted while large companies are sometimes very
difficult to contact.
maybe
Dawid Kozinski (Samsung) or Samsung (Dawid Kozinski) or something like that
would specify this better

thx

Hi,
To be clear. We are not fighting for the right to push. The write access is
not our goal at the moment.
We just want to do our contribution to the FFMpeg project and provide
support for the EVC codec and the only reason we've added entries to the
MAINTENANCE file is to provide the information so others know who to contact
about the codec.

Yesterday I submitted new patches to the FFMepg patchwork and following what
Lynne said I removed our entries from the MAINTENANCE file.

However, If I understand you correctly, I shouldn't remove it, I should
though leave the information in the file.
So now I should restore what I removed. 
Correct me if I'm wrong cause it's a bit confusing.

Anyway, we will restore previously removed information from the MAINTENANCE
file, information that points to the person who can handle EVC-related
queries, but please take a look at what we have uploaded to the patchwork.
Check if it is OK or do we need still to change something else.

BR
Dawid

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

"I am not trying to be anyone's saviour, I'm trying to think about the
future and not be sad" - Elon Musk