[vlc-devel] [PATCH 1/4] fifo: cleanup
remi at remlab.net
Mon Apr 13 13:59:41 CEST 2020
Le maanantaina 13. huhtikuuta 2020, 14.53.19 EEST Thomas Guillem a écrit :
> On Mon, Apr 13, 2020, at 13:47, Rémi Denis-Courmont wrote:
> > Le maanantaina 13. huhtikuuta 2020, 14.44.08 EEST Thomas Guillem a écrit :
> > > On Mon, Apr 13, 2020, at 13:36, Rémi Denis-Courmont wrote:
> > > > Le maanantaina 13. huhtikuuta 2020, 14.22.05 EEST Thomas Guillem a
> > > > > Hello,
> > > > >
> > > > > This set was pushed without giving me the chance to review it.
> > > >
> > > > Denis and I warned that the review deadlines were not going to work. I
> > > > specifically warned that it should be one week for everyone to have
> > > > fair
> > > > time.
> > > >
> > > > This was explicitly rejected.
> > >
> > > I think we decided 72hours of workday.
> > No that's clearly not the case. I've seen plenty of commits getting
> > through in less than that and mostly not from me.
> Yes, I already made this mistake, more than one time, and you told me, with
> a good reason, that it was not OK.
> > It's 24 hours (now) or it's one week (like before).
> 24 hours is impossible for me, specially with the new confinement situation.
You know, 24 hours is impossible for almost anyone, probably anyone in the
project, although it seems to be the working rule in Linux kernel somehow. The
point is that it's fair and it's reasonably fast to turn your patch series
around. As a committer, I find one week insanely long, and one day just about
As a reviewer, I think one week is the minimum, unless everybody is full time
pro and in the same continent, in which case one or two business days. But VLC
is not such a case, and it probably never will be.
Tapiolan uusi kaupunki, Uudenmaan tasavalta
More information about the vlc-devel