Re: Patch Acceptance Procedure

From: Crispin Cowan (crispinat_private)
Date: Tue Jul 24 2001 - 00:28:34 PDT

  • Next message: David Wagner: "Re: Patch Acceptance Procedure"

    Seth Arnold wrote:
    
    > On Mon, Jul 23, 2001 at 09:20:39PM -0400, jmjonesat_private wrote:
    > > HE suggested a one-day review period.  You seem to agree with me that it
    > > may be inadequate in some circumstances.  What circumstances?  This is a
    > > truely cosmic conundrum, but a first-order answer is probably better than
    > > none.
    >
    > I don't think Crispin intended for one day to be all the time allowed for
    > preparation of a competing proposal -- just that one day is enough time to
    > let someone look at a proposal and possibly complain if the idea seems
    > inadequate or misguided.
    >
    > The actual process of replacing the 'faulty' patch could possibly take weeks.
    
    Seth is correct, that's what I intended: a one-day waiting period for
    objections, followed by an arbitrary/appropriate period to develop
    alternatives.
    
    
    > Waiting more than one day is going to slow the project down too much. We want
    > to have something to take to 2.5.x for some small value of x. ;) But one day
    > should be enough for someone to at least make a rough guess if the patch
    > looks good or not.
    
    I really meant "one day" not "one business day." There are really helpful
    people working this project on their personal time, and it is not constructive
    to bias the rules in favor of commercially paid contributors.
    
    But I'm also quite convinced by Greg's arguments to keep it very informal.
    Basically, I think it would be polite for the people with write access to the
    tree to give a shout out that they're about to commit something. It might
    prevent a situation (which has already happened) in which a proposed patch that
    I expected to be rejected was committed.
    
    >From the functional point of view, the "About to commit" protocol helps busy
    people to prioritize reading stuff that is imminant ahead of stuff that's just
    being floated.
    
    Crispin
    
    --
    Crispin Cowan, Ph.D.
    Chief Scientist, WireX Communications, Inc. http://wirex.com
    Security Hardened Linux Distribution:       http://immunix.org
    Available for purchase: http://wirex.com/Products/Immunix/purchase.html
    
    
    
    
    _______________________________________________
    linux-security-module mailing list
    linux-security-moduleat_private
    http://mail.wirex.com/mailman/listinfo/linux-security-module
    



    This archive was generated by hypermail 2b30 : Tue Jul 24 2001 - 01:47:27 PDT