[FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states
michael at niedermayer.cc
Sat Oct 22 22:20:20 EEST 2016
On Sat, Oct 22, 2016 at 10:41:29AM -0800, Lou Logan wrote:
> On Sat, Oct 22, 2016, at 09:49 AM, Bodecs Bela wrote:
> > under review: someone marked it because he/she investigate this patch.
> > So the patch submitter and other potential reviewers may feel/be_sure
> > that this patch is already handled by someone else.
> > I suggest to use it. This is psychologic aspect that the patch submitter
> > may feel more patienty toward review process opposite to mere "new"
> > state.
> In my opinion that seems like unnecessary extra work. All patches in
> theory are to be "under review". Reviewing takes enough time,
> initiative, and motivation as is, and adding another, potentially
> superfluous step just over complicates it.
> Perhaps "Under Review" can just be removed, disabled, hidden, or
> documented as "this option is ignored". Same with "Deferred" and
> "Awaiting Upstream".
Removed "Under Review" and "Awaiting Upstream"
Theres a patch with "Deferred" state, this would need to be changed
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Those who would give up essential Liberty, to purchase a little
temporary Safety, deserve neither Liberty nor Safety -- Benjamin Franklin
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel