Message-ID: <4F15BBA1.5010701@2ndQuadrant.com>
Date: Tue, 17 Jan 2012 13:19:13 -0500
From: Greg Smith <greg@2ndQuadrant.com>
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US;
	rv:1.9.2.24) Gecko/20111120 Icedove/3.1.16
MIME-Version: 1.0
To: Peter Eisentraut <peter_e@gmx.net>
CC: pgsql-hackers@postgresql.org
Subject: Re: automating CF submissions (was xlog location arithmetic)
References: <4EDD97E5.8080201@timbira.com>	
	<CAHGQGwFBF5D22cpw5gqXqLUDJLoJOfi_rc7BDg0==ekX4887tg@mail.gmail.com>	
	<4F118D45.3060801@timbira.com> <4F122967.6070400@2ndQuadrant.com>	
	<CABwTF4WpYyMy2Uk2j=oTB8RB-cf2EmRJ=wfiW2r3Gj5SBb+fAA@mail.gmail.com>	
	<4F125998.5020204@2ndQuadrant.com> <4F148D29.7080504@agliodbs.com>	
	<4F14A3EE.8000402@2ndQuadrant.com>
	<1326819026.2820.1.camel@vanquo.pezone.net>
In-Reply-To: <1326819026.2820.1.camel@vanquo.pezone.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

On 01/17/2012 11:50 AM, Peter Eisentraut wrote:
> On mån, 2012-01-16 at 17:25 -0500, Greg Smith wrote:
>> The most reasonable answer to this is for people to publish a git repo
>> URL in addition to the "official" submission of changes to the list in
>> patch form.
> Note that the original complaint was that for the occasional reviewer,
> the current system takes at least 5 partially redundant steps in two
> different systems.  I doubt that adding a third system and more
> partially redundant steps it going to help that.

Publishing the submission via git is an extra step for the patch 
submitter.  If that happens, the reviewer can test just be cloning that, 
instead of first closing the PostgreSQL one then applying the patch.  It 
removes the "how do I fish the patch out of the archives?" problem from 
the reviewer's side of things.

-- 
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support www.2ndQuadrant.com