[FFmpeg-devel] 5.0 release

Michael Niedermayer michael at niedermayer.cc
Wed Dec 15 17:15:35 EET 2021


On Wed, Dec 15, 2021 at 04:08:43PM +0100, Diederick C. Niehorster wrote:
> Hi Michael,
> 
> On Wed, Dec 15, 2021 at 3:52 PM Michael Niedermayer
> <michael at niedermayer.cc> wrote:
> >
> > On Tue, Dec 14, 2021 at 05:42:01PM +0100, Diederick C. Niehorster wrote:
> > > Hi Michael,
> > >
> > > On Tue, Dec 14, 2021 at 5:19 PM Michael Niedermayer
> > > <michael at niedermayer.cc> wrote:
> > > >
> > > >
> > > > The patches do not say in their commit messages that they have been reviewed
> > > > maybe you want to repost them with that changed.
> > >
> > > Thanks for having a look!
> > > I have mentioned this here:
> > > https://ffmpeg.org/pipermail/ffmpeg-devel/2021-December/289461.html
> > >
> > > Or do you mean that for each specific patch, if there were previous
> > > review comments, i should mention that it was updated in response to
> > > the review? For some that may be hard (e.g. i was asked to change the
> > > order of some commits).
> >
> > Patches which where approved already and not changed since then could
> > contain that information in the commit message. We tend to use some
> > Reviewed-by: ...
> > for that
> > that may speed up future review and commit
> >
> > I dont know dshow and dont have a proper setup to test it so i wont be
> > applying or reviewing these patches
> 
> Ah, i see. I have made changes here in response to other review
> comment since i got that LGTM for the series. I'll contact the dshow
> maintainer again for that (I assume he would be the only one who
> should be committing my patchset?).

anyone with commit rights could apply after a review by someone who knows
the code. Of course could != will

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- 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/20211215/f730c429/attachment.sig>


More information about the ffmpeg-devel mailing list