[FFmpeg-devel] [PATCH 2/2] doc/developer: Code pushed without patches on ffmpeg-devel must be announced on the ML
Michael Niedermayer
michael at niedermayer.cc
Fri Aug 25 18:34:39 EEST 2023
On Thu, Aug 24, 2023 at 10:04:16PM +0200, Andreas Rheinhardt wrote:
> Michael Niedermayer:
> > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > ---
> > doc/developer.texi | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/doc/developer.texi b/doc/developer.texi
> > index 383120daaa..1c0091fc74 100644
> > --- a/doc/developer.texi
> > +++ b/doc/developer.texi
> > @@ -856,6 +856,15 @@ way to get everyone's patches reviewed sooner.
> > Reviews must be constructive and when rejecting a patch the reviewer must explain
> > their reasons and ideally suggest an alternative approach.
> >
> > +If a change is pushed without being sent to ffmpeg-devel, the developer
> > +pushing it must annouce doing so on the ffmpeg-devel mailing list immedeatly.
> > + at example
> > +forgot a semicolon in this patch, pushed a seperate fix
> > +pushed my new autograd engine and stable diffusion filter. Didnt want to
> > +go through the bikeshed if that belongs in FFmpeg, go to the GA if you want
> > +it removed. Otherwise Just tell me what i should improve and ill look into it.
> > + at end example
>
> This encourages pushing patches (even completely new filters) without
> sending them to the ML.
That was not the intend but if you look at "cvslog" and ffmpeg-devel, you will
notice that there are things being pushed that have not been seen on the
ffmpeg-devel mailing list.
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Old school: Use the lowest level language in which you can solve the problem
conveniently.
New school: Use the highest level language in which the latest supercomputer
can solve the problem without the user falling asleep waiting.
-------------- 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/20230825/d42eb089/attachment.sig>
More information about the ffmpeg-devel
mailing list