This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: is live!

On 05/26/2014 09:52 AM, Siddhesh Poyarekar wrote:
> On 26 May 2014 13:07, Andreas Jaeger <> wrote:
>> What happens if one person says "fine" while another one disagrees? Is
>> Accepted then the right state for this?
> The first reviewer sets 'Accepted' (assuming she doesn't need another
> reviewer to validate) and if another reviewer disagrees then he can
> set the status back to 'Under Review'.  if he agrees, then there's no
> need to change any status.
>> Could you write up a workflow: Live of a patch, something like:
>> 1. Patch gets submitted, system puts patch into state NEW
>> 2. Reviewer comments on it:
>> a) Disagrees: Sets it to
>> b) agrees: Sets it to
>> c) Not a patch for glibc, set it to Not Applicable
>> 3. Patch gets committed: Set it to Commited/Accepted
> That's a good idea.  I have modified the current document[1] and also
> added the Committed status to the document.  Once we have consensus on
> adding the new 'Committed' status, I'll also add it to patchwork.
> Siddhesh
> [1]

thanks, that answers my current ;) questions nicely,

 Andreas Jaeger aj@{,} Twitter/Identica: jaegerandi
  SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 NÃrnberg, Germany
   GF: Jeff Hawn,Jennifer Guild,Felix ImendÃrffer,HRB16746 (AG NÃrnberg)
    GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]