Mailing-List: contact general-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: general@incubator.apache.org
Received-SPF: pass (nike.apache.org: local policy)
Date: Tue, 14 Sep 2010 04:22:28 +0200
From: Daniel Shahaf <d.s@daniel.shahaf.name>
To: general@incubator.apache.org
Subject: Re: Accepting patches in a podling
Message-ID: <20100914022228.GB17442@daniel3.local>
References: <1284425758.5045.58.camel@avon.watzmann.net>
 <AANLkTimEiofzbhLFDeWX99K_TBN9Jid=qoa7-T1RseH=@mail.gmail.com>
 <20100914014309.GA17294@daniel3.local>
 <415361.43155.qm@web54402.mail.re2.yahoo.com>
 <AANLkTi=9qx1erBx64Ows+iQ0AAp=m1k0rGLmpQo+ftkW@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <AANLkTi=9qx1erBx64Ows+iQ0AAp=m1k0rGLmpQo+ftkW@mail.gmail.com>
User-Agent: Mutt/1.5.18 (2008-05-17)

Benson Margulies wrote on Mon, Sep 13, 2010 at 22:03:12 -0400:
> However, I thank Joe for pointing out that I was careful to avoid stating a
> requirement when I didn't know that one existed.
> 

Thanks for clarifying.  To summarize, having all patches go via JIRA is
one possible way of working, but there is no requirement to use it ---
other ways are possible too.

> I've never hung out on an ASF project that (still) used email patches as the
> common practice, so I was unaware of it.
> 
> So, if you asked me, I'd say that using JIRA or BZ items is good
> organizational hygiene, giving more people visibility into the process and
> making it harder to drop a good contribution on the floor -- but if you have
> a working system involving an official mailing list, you have a working
> system.
> 

FWIW, what Subversion uses:

http://subversion.apache.org/docs/community-guide/community-guide.html#patches
http://subversion.apache.org/docs/community-guide/community-guide.html#patch-manager

Briefly, all patches go to dev@, and only the unlucky patches that
were neither applied nor rejected get stashed in the issue tracker
to avoid being dropped on the floor.

> On the other hand, I think that we all agree that a patch mailed by personal
> email to a committer who commits it is not a good thing.
> 

Yup.  The difference being whether or not the dev list is CCed.

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org