diff mbox series

[FFmpeg-devel] MAINTAINERS: remove myself from FFmpeg

Message ID CAPYw7P65jHPN1e40-=C58Vg6gP4NidtGw9KnzR7q__-RRJP_Tw@mail.gmail.com
State New
Headers show
Series [FFmpeg-devel] MAINTAINERS: remove myself from FFmpeg | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Paul B Mahol Dec. 7, 2023, 7:29 a.m. UTC
Attached.

Comments

Jean-Baptiste Kempf Dec. 7, 2023, 8:54 a.m. UTC | #1
Why?

On Thu, 7 Dec 2023, at 08:29, Paul B Mahol wrote:
> Attached.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
>
> Attachments:
> * 0001-MAINTAINERS-remove-myself-from-FFmpeg.patch
Nicolas George Dec. 7, 2023, 8:59 a.m. UTC | #2
Jean-Baptiste Kempf (12023-12-07):
> Why?

Because after twelve years libav has finally managed to take control and
FFmpeg is now essentially dead.

Thank you for your contribution in this.
Jean-Baptiste Kempf Dec. 7, 2023, 9:06 a.m. UTC | #3
On Thu, 7 Dec 2023, at 09:59, Nicolas George wrote:
> Jean-Baptiste Kempf (12023-12-07):
>> Why?
>
> Because after twelve years libav has finally managed to take control and
> FFmpeg is now essentially dead.

You are delusional. libav is dead since a long time.
The fact that people are pushing against your patches does not mean they are wrong or an enemy.

> Thank you for your contribution in this.

You are welcome.
Nicolas George Dec. 7, 2023, 9:10 a.m. UTC | #4
Jean-Baptiste Kempf (12023-12-07):
> You are delusional.

Oh, this is absolutely appropriate, especially from somebody who just
got elected to apply the code of conduct.
Jean-Baptiste Kempf Dec. 7, 2023, 9:12 a.m. UTC | #5
On Thu, 7 Dec 2023, at 10:10, Nicolas George wrote:
> Jean-Baptiste Kempf (12023-12-07):
>> You are delusional.
>
> Oh, this is absolutely appropriate, especially from somebody who just
> got elected to apply the code of conduct.

Keep antagonizing more people in the community, it's a great idea.
Nicolas George Dec. 7, 2023, 9:14 a.m. UTC | #6
Jean-Baptiste Kempf (12023-12-07):
> Keep antagonizing more people in the community, it's a great idea.

The community is dead. And you never belonged to it anyway, with your
superior attitude.
Jean-Baptiste Kempf Dec. 7, 2023, 9:16 a.m. UTC | #7
On Thu, 7 Dec 2023, at 10:14, Nicolas George wrote:
> The community is dead. And you never belonged to it anyway, with your
> superior attitude.

You calling someone "superior attitude" is amazing.
Michael Niedermayer Dec. 7, 2023, 12:32 p.m. UTC | #8
On Thu, Dec 07, 2023 at 08:29:14AM +0100, Paul B Mahol wrote:
> Attached.

>  MAINTAINERS |   55 -------------------------------------------------------
>  1 file changed, 55 deletions(-)
> 57486b662b3460fe6704e2ec236b0cd65cfe75f9  0001-MAINTAINERS-remove-myself-from-FFmpeg.patch
> From b249499fccb49705ade14362875ebf4d22628fa4 Mon Sep 17 00:00:00 2001
> From: Paul B Mahol <onemda@gmail.com>
> Date: Thu, 7 Dec 2023 08:27:14 +0100
> Subject: [PATCH] MAINTAINERS: remove myself from FFmpeg
> 
> Signed-off-by: Paul B Mahol <onemda@gmail.com>
> ---
>  MAINTAINERS | 55 -----------------------------------------------------
>  1 file changed, 55 deletions(-)

Is there anything i can do for you to reconsider ?

I dont think i understand the reason behind this

Is there something i can help with ?

thx

[...]
Ronald S. Bultje Dec. 7, 2023, 2:29 p.m. UTC | #9
Hi,

On Thu, Dec 7, 2023 at 7:33 AM Michael Niedermayer <michael@niedermayer.cc>
wrote:

> On Thu, Dec 07, 2023 at 08:29:14AM +0100, Paul B Mahol wrote:
> > Attached.
>
> >  MAINTAINERS |   55
> -------------------------------------------------------
> >  1 file changed, 55 deletions(-)
> > 57486b662b3460fe6704e2ec236b0cd65cfe75f9
> 0001-MAINTAINERS-remove-myself-from-FFmpeg.patch
> > From b249499fccb49705ade14362875ebf4d22628fa4 Mon Sep 17 00:00:00 2001
> > From: Paul B Mahol <onemda@gmail.com>
> > Date: Thu, 7 Dec 2023 08:27:14 +0100
> > Subject: [PATCH] MAINTAINERS: remove myself from FFmpeg
> >
> > Signed-off-by: Paul B Mahol <onemda@gmail.com>
> > ---
> >  MAINTAINERS | 55 -----------------------------------------------------
> >  1 file changed, 55 deletions(-)
>
> Is there anything i can do for you to reconsider ?
>
> I dont think i understand the reason behind this
>

I think many of us don't.

Paul, want to elaborate or point to a log of what happened?

Ronald
Leo Izen Dec. 7, 2023, 2:59 p.m. UTC | #10
On 12/7/23 03:59, Nicolas George wrote:
> Jean-Baptiste Kempf (12023-12-07):
>> Why?
> 
> Because after twelve years libav has finally managed to take control and
> FFmpeg is now essentially dead.
> 
> Thank you for your contribution in this.
> 

I believe the question was directed at Paul, who has not answered yet 
(as of writing this).

- Leo Izen (Traneptora)
Nicolas George Dec. 7, 2023, 4:32 p.m. UTC | #11
Leo Izen (12023-12-07):
> I believe the question was directed at Paul

That does not make my answer any less relevant.
Rémi Denis-Courmont Dec. 7, 2023, 4:39 p.m. UTC | #12
Le torstaina 7. joulukuuta 2023, 10.59.06 EET Nicolas George a écrit :
> Jean-Baptiste Kempf (12023-12-07):
> > Why?
> 
> Because after twelve years libav has finally managed to take control and
> FFmpeg is now essentially dead.

The question was for Paul. Even if you take Anton's knee-jerk threats of 
reverts as LibAV-think, they were but the last straw (Paul wrote as much).

You have had heated arguments against Paul in recent times too. You have also 
argued a lot of exercising your review privileges, which sounds like a very 
libavish notion to me - a LibAV notion that made into written down FFmpeg 
project rules. As a matter of fact, regardless of who was right or wrong, and 
whence, I can only _observe_ that Paul did complain specifically about you on 
the IRC channel. To be fair, he also abundantly abused JB there, although I do 
not know how much of it was sarcasm vs actual attack.

But in any case, by that same logic, you could also be "thank[ed] for your 
contribution in this". In my opinion, this would be unfair to you, and 
accordingly, you are being unfair to whomever you designate by "libav".

(For the sake of utmost clarity, I am not so vain as to consider myself a part 
of the former project known as libav with however few contributions I made 
thereto.)

> Thank you for your contribution in this.

@CC: Yes, that second sentence can be construed as an ad hominem against 
Anton. Feel free to ban me.
Leo Izen Dec. 7, 2023, 4:48 p.m. UTC | #13
On 12/7/23 11:32, Nicolas George wrote:
> Leo Izen (12023-12-07):
>> I believe the question was directed at Paul
> 
> That does not make my answer any less relevant.
> 
> 

I'm not going to have an argument with you about why a question to Paul 
about a decision Paul made should be answered by Paul, considering that 
he is the only one who can read his own thoughts.

When you reply to this email with some off-topic paragraph about 
something something Libav something something, I'm not going to respond.

- Leo Izen (Traneptora)
Nicolas George Dec. 19, 2023, 1:11 p.m. UTC | #14
Rémi Denis-Courmont (12023-12-07):
> You have had heated arguments against Paul in recent times too. You have also 
> argued a lot of exercising your review privileges, which sounds like a very 
> libavish notion to me

Only because you were not there at the time to get a first-hand
impression. That patches should be reviewed if possible was the policy
way before libav. That came with a set of implicit rules: waiting a few
days, then pinging, then waiting a few days and only then pushing
without review. The role of maintainer would affect the reasonable value
for “a few” days.

Paul insistence on pushing after barely 24 hours on code with a
maintainer that is not him always contradicting the way of doing things.

Furthermore, his refusal to give more time to the maintainer when asked
to is not just that: it is a level of rudeness and incivility
incompatible with working together with other people.

But Paul's attitude was annoying but never a real problem: resist his
eagerness a little and soon he finds something else to do and forgets
about pushing immediately for weeks or months.

For reference, libav turned the practice that patches should be reviewed
into a hard rule that patches must be reviewed. At the same time, since
they had kicked out or disgusted a significant part of the projects'
maintainers, they had nobody capable of actually reviewing the code. As
a result, when a patch was proposed by a major libav contributor, after
the ping somebody else who did not know the code would post a clueless
“LGTM”.

(The online archives of libav-devel seem to have disappeared, so I
cannot link to the example I bookmarked.)
Vittorio Giovara Dec. 19, 2023, 8:34 p.m. UTC | #15
On Tue, Dec 19, 2023 at 8:12 AM Nicolas George <george@nsup.org> wrote:

> Rémi Denis-Courmont (12023-12-07):
> > You have had heated arguments against Paul in recent times too. You have
> also
> > argued a lot of exercising your review privileges, which sounds like a
> very
> > libavish notion to me
>
> Only because you were not there at the time to get a first-hand
> impression. That patches should be reviewed if possible was the policy
> way before libav. That came with a set of implicit rules: waiting a few
> days, then pinging, then waiting a few days and only then pushing
> without review. The role of maintainer would affect the reasonable value
> for “a few” days.
>
> Paul insistence on pushing after barely 24 hours on code with a
> maintainer that is not him always contradicting the way of doing things.
>
> Furthermore, his refusal to give more time to the maintainer when asked
> to is not just that: it is a level of rudeness and incivility
> incompatible with working together with other people.
>
> But Paul's attitude was annoying but never a real problem: resist his
> eagerness a little and soon he finds something else to do and forgets
> about pushing immediately for weeks or months.
>
> For reference, libav turned the practice that patches should be reviewed
> into a hard rule that patches must be reviewed. At the same time, since
> they had kicked out or disgusted a significant part of the projects'
> maintainers, they had nobody capable of actually reviewing the code. As
> a result, when a patch was proposed by a major libav contributor, after
> the ping somebody else who did not know the code would post a clueless
> “LGTM”.
>
> (The online archives of libav-devel seem to have disappeared, so I
> cannot link to the example I bookmarked.)
>

I am not too sure that bringing up a topic from 12 days ago with arguments
from 12 years ago is bringing any value to the conversation. Just as a
note, remember that a clueless LGTM is a better review than NO review, and
in fact it's the system that it's employed in any modern software house:
the master branch is usually protected and any PR/MR needs both CI pass and
at least a read from a developer.

Oh and for the sake of your (and our readers') time, don't bother replying,
I'm not interested in discussing 12 years ago affairs or modern development
practices here. I do invite you to evaluate whether your vision of ffmpeg
is still the one shared by the community as a whole though.

Regards
Nicolas George Dec. 20, 2023, 9:17 a.m. UTC | #16
Vittorio Giovara (12023-12-19):
> I am not too sure that bringing up a topic from 12 days ago with arguments
> from 12 years ago is bringing any value to the conversation.

Those who do not learn from history are bound to repeat it.

>								Just as a
> note, remember that a clueless LGTM is a better review than NO review, and

Uh. No, a clueless LGTM is absolutely no better than no review at all,
that is what clueless means.

> in fact it's the system that it's employed in any modern software house:
> the master branch is usually protected and any PR/MR needs both CI pass and
> at least a read from a developer.

I have a scoop for you: Libre Software projects are not “modern software
house”s where most effort is spent covering one's ass and trying to
blame somebody else for unavoidable problems.

> Oh and for the sake of your (and our readers') time, don't bother replying,
> I'm not interested in discussing 12 years ago affairs or modern development
> practices here.

You know, YOU have the power to stop wasting your time and ours: just
stop posting wrong and dishonest messages, and I will not have to
correct them.
diff mbox series

Patch

From b249499fccb49705ade14362875ebf4d22628fa4 Mon Sep 17 00:00:00 2001
From: Paul B Mahol <onemda@gmail.com>
Date: Thu, 7 Dec 2023 08:27:14 +0100
Subject: [PATCH] MAINTAINERS: remove myself from FFmpeg

Signed-off-by: Paul B Mahol <onemda@gmail.com>
---
 MAINTAINERS | 55 -----------------------------------------------------
 1 file changed, 55 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 57137e1d6d..b517ed8342 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -144,7 +144,6 @@  Codecs:
   bgmc.c, bgmc.h                        Thilo Borgmann
   binkaudio.c                           Peter Ross
   cavs*                                 Stefan Gehrer
-  cdxl.c                                Paul B Mahol
   celp_filters.*                        Vitor Sessak
   cinepak.c                             Roberto Togni
   cinepakenc.c                          Rl / Aetey G.T. AB
@@ -163,7 +162,6 @@  Codecs:
   dv.c                                  Roman Shaposhnik
   dvbsubdec.c                           Anshul Maheshwari
   eacmv*, eaidct*, eat*                 Peter Ross
-  evrc*                                 Paul B Mahol
   exif.c, exif.h                        Thilo Borgmann
   ffv1*                                 Michael Niedermayer
   ffwavesynth.c                         Nicolas George
@@ -217,7 +215,6 @@  Codecs:
   nvdec*, nvenc*                        Timo Rothenpieler
   omx.c                                 Martin Storsjo, Aman Gupta
   opus*                                 Rostislav Pehlivanov
-  paf.*                                 Paul B Mahol
   pcx.c                                 Ivo van Poorten
   pgssubdec.c                           Reimar Doeffinger
   ptx.c                                 Ivo van Poorten
@@ -239,16 +236,13 @@  Codecs:
   srt*                                  Aurelien Jacobs
   sunrast.c                             Ivo van Poorten
   svq3.c                                Michael Niedermayer
-  tak*                                  Paul B Mahol
   truemotion1*                          Mike Melanson
   tta.c                                 Alex Beregszaszi, Jaikrishnan Menon
-  ttaenc.c                              Paul B Mahol
   txd.c                                 Ivo van Poorten
   v4l2_*                                Jorge Ramirez-Ortiz
   vc2*                                  Rostislav Pehlivanov
   vcr1.c                                Michael Niedermayer
   videotoolboxenc.c                     Rick Kern, Aman Gupta
-  vima.c                                Paul B Mahol
   vorbisdec.c                           Denes Balatoni, David Conrad
   vorbisenc.c                           Oded Shimon
   vp3*                                  Mike Melanson
@@ -262,9 +256,7 @@  Codecs:
   wmavoice.c                            Ronald S. Bultje
   wmv2.c                                Michael Niedermayer
   xan.c                                 Mike Melanson
-  xbm*                                  Paul B Mahol
   xface                                 Stefano Sabatini
-  xwd*                                  Paul B Mahol
 
 Hardware acceleration:
   dxva2*                                Hendrik Leppkes, Laurent Aimar, Steve Lhomme
@@ -308,64 +300,33 @@  Generic parts:
   motion_estimation.c                   Davinder Singh
 
 Filters:
-  f_drawgraph.c                         Paul B Mahol
-  af_adelay.c                           Paul B Mahol
-  af_aecho.c                            Paul B Mahol
-  af_afade.c                            Paul B Mahol
   af_amerge.c                           Nicolas George
-  af_aphaser.c                          Paul B Mahol
   af_aresample.c                        Michael Niedermayer
-  af_astats.c                           Paul B Mahol
   af_atempo.c                           Pavel Koshevoy
-  af_biquads.c                          Paul B Mahol
-  af_chorus.c                           Paul B Mahol
-  af_compand.c                          Paul B Mahol
   af_firequalizer.c                     Muhammad Faiz
   af_hdcd.c                             Burt P.
-  af_ladspa.c                           Paul B Mahol
   af_loudnorm.c                         Kyle Swanson
   af_pan.c                              Nicolas George
-  af_sidechaincompress.c                Paul B Mahol
-  af_silenceremove.c                    Paul B Mahol
-  avf_aphasemeter.c                     Paul B Mahol
-  avf_avectorscope.c                    Paul B Mahol
   avf_showcqt.c                         Muhammad Faiz
-  vf_blend.c                            Paul B Mahol
   vf_bwdif                              Thomas Mundt (CC <thomas.mundt@hr.de>)
   vf_chromakey.c                        Timo Rothenpieler
-  vf_colorchannelmixer.c                Paul B Mahol
   vf_colorconstancy.c                   Mina Sami    (CC <minas.gorgy@gmail.com>)
-  vf_colorbalance.c                     Paul B Mahol
   vf_colorkey.c                         Timo Rothenpieler
-  vf_colorlevels.c                      Paul B Mahol
   vf_coreimage.m                        Thilo Borgmann
-  vf_deband.c                           Paul B Mahol
   vf_dejudder.c                         Nicholas Robbins
   vf_delogo.c                           Jean Delvare (CC <jdelvare@suse.com>)
   vf_drawbox.c/drawgrid                 Andrey Utkin
-  vf_extractplanes.c                    Paul B Mahol
-  vf_histogram.c                        Paul B Mahol
   vf_hqx.c                              Clément Bœsch
   vf_idet.c                             Pascal Massimino
-  vf_il.c                               Paul B Mahol
   vf_(t)interlace                       Thomas Mundt (CC <thomas.mundt@hr.de>)
   vf_lenscorrection.c                   Daniel Oberhoff
   vf_libplacebo.c                       Niklas Haas
-  vf_mergeplanes.c                      Paul B Mahol
   vf_mestimate.c                        Davinder Singh
   vf_minterpolate.c                     Davinder Singh
-  vf_neighbor.c                         Paul B Mahol
-  vf_psnr.c                             Paul B Mahol
-  vf_random.c                           Paul B Mahol
   vf_readvitc.c                         Tobias Rapp (CC t.rapp at noa-archive dot com)
   vf_scale.c                            Michael Niedermayer
-  vf_separatefields.c                   Paul B Mahol
-  vf_ssim.c                             Paul B Mahol
-  vf_stereo3d.c                         Paul B Mahol
-  vf_telecine.c                         Paul B Mahol
   vf_tonemap_opencl.c                   Ruiling Song
   vf_yadif.c                            Michael Niedermayer
-  vf_zoompan.c                          Paul B Mahol
 
 Sources:
   vsrc_mandelbrot.c                     Michael Niedermayer
@@ -387,7 +348,6 @@  Muxers/Demuxers:
   4xm.c                                 Mike Melanson
   aadec.c                               Vesselin Bontchev (vesselin.bontchev at yandex dot com)
   adtsenc.c                             Robert Swain
-  afc.c                                 Paul B Mahol
   aiffdec.c                             Baptiste Coudurier, Matthieu Bouron
   aiffenc.c                             Baptiste Coudurier, Matthieu Bouron
   alp.c                                 Zane van Iperen
@@ -398,16 +358,12 @@  Muxers/Demuxers:
   argo_brp.c                            Zane van Iperen
   argo_cvg.c                            Zane van Iperen
   ass*                                  Aurelien Jacobs
-  astdec.c                              Paul B Mahol
   astenc.c                              James Almer
   avi*                                  Michael Niedermayer
   avisynth.c                            Stephen Hutchinson
-  avr.c                                 Paul B Mahol
   bink.c                                Peter Ross
   boadec.c                              Michael Niedermayer
-  brstm.c                               Paul B Mahol
   caf*                                  Peter Ross
-  cdxl.c                                Paul B Mahol
   codec2.c                              Tomas Härdin
   crc.c                                 Michael Niedermayer
   dashdec.c                             Steven Liu
@@ -416,10 +372,8 @@  Muxers/Demuxers:
   dfpwmdec.c                            Jack Bruienne
   dss.c                                 Oleksij Rempel
   dtsdec.c                              foo86
-  dtshddec.c                            Paul B Mahol
   dv.c                                  Roman Shaposhnik
   electronicarts.c                      Peter Ross
-  epafdec.c                             Paul B Mahol
   evc*                                  Samsung (Dawid Kozinski)
   ffm*                                  Baptiste Coudurier
   flic.c                                Mike Melanson
@@ -434,7 +388,6 @@  Muxers/Demuxers:
   imf*                                  Pierre-Anthony Lemieux
   img2*.c                               Michael Niedermayer
   ipmovie.c                             Mike Melanson
-  ircam*                                Paul B Mahol
   iss.c                                 Stefan Gehrer
   jpegxl*                               Leo Izen
   jvdec.c                               Peter Ross
@@ -442,14 +395,12 @@  Muxers/Demuxers:
   libmodplug.c                          Clément Bœsch
   libopenmpt.c                          Josh de Kock
   lmlm4.c                               Ivo van Poorten
-  lvfdec.c                              Paul B Mahol
   lxfdec.c                              Tomas Härdin
   matroska.c                            Aurelien Jacobs, Andreas Rheinhardt
   matroskadec.c                         Aurelien Jacobs, Andreas Rheinhardt
   matroskaenc.c                         David Conrad, Andreas Rheinhardt
   matroska subtitles (matroskaenc.c)    John Peebles
   metadata*                             Aurelien Jacobs
-  mgsts.c                               Paul B Mahol
   microdvd*                             Aurelien Jacobs
   mm.c                                  Peter Ross
   mov.c                                 Baptiste Coudurier
@@ -462,7 +413,6 @@  Muxers/Demuxers:
   msnwc_tcp.c                           Ramiro Polla
   mtv.c                                 Reynaldo H. Verdejo Pinochet
   mxf*                                  Baptiste Coudurier, Tomas Härdin
-  nistspheredec.c                       Paul B Mahol
   nsvdec.c                              Francois Revol
   nut*                                  Michael Niedermayer
   nuv.c                                 Reimar Doeffinger
@@ -470,11 +420,9 @@  Muxers/Demuxers:
   oggenc.c                              Baptiste Coudurier
   oggparse*.c                           David Conrad
   oma.c                                 Maxim Poliakovski
-  paf.c                                 Paul B Mahol
   pp_bnk.c                              Zane van Iperen
   psxstr.c                              Mike Melanson
   pva.c                                 Ivo van Poorten
-  pvfdec.c                              Paul B Mahol
   r3d.c                                 Baptiste Coudurier
   raw.c                                 Michael Niedermayer
   rdt.c                                 Ronald S. Bultje
@@ -495,11 +443,9 @@  Muxers/Demuxers:
   sdp.c                                 Martin Storsjo
   segafilm.c                            Mike Melanson
   segment.c                             Stefano Sabatini
-  smjpeg*                               Paul B Mahol
   spdif*                                Anssi Hannula
   srtdec.c                              Aurelien Jacobs
   swf.c                                 Baptiste Coudurier
-  takdec.c                              Paul B Mahol
   tta.c                                 Alex Beregszaszi
   txd.c                                 Ivo van Poorten
   voc.c                                 Aurelien Jacobs
@@ -509,7 +455,6 @@  Muxers/Demuxers:
   webvtt*                               Matthew J Heaney
   westwood.c                            Mike Melanson
   wtv.c                                 Peter Ross
-  wvenc.c                               Paul B Mahol
 
 Protocols:
   async.c                               Zhang Rui
-- 
2.42.1