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

[Michael Niedermayer]

On Thu, Dec 15, 2022 at 10:14:40AM +0100, Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics wrote:
> 
> 
> 
> 
> -----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.

I think the entry in some form is fine but 
I would suggest to leave the entry out until the discussion with Lynne
reaches some consensus. And then after (we all mostly agree) add the entry
That way everyone can calmly discuss this

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20221215/7a41a232/attachment.sig>