Re: branching for 9.2devel

From: Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Greg Stark <gsstark(at)mit(dot)edu>, Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: branching for 9.2devel
Date: 2011-05-02 06:01:47
Message-ID: BANLkTin=8zFUgAYkH0-AnNE6wMcvGCyP2g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Apr 26, 2011 at 2:25 AM, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:

>
>
> On 04/25/2011 04:28 PM, Tom Lane wrote:
>
>> Andrew Dunstan<andrew(at)dunslane(dot)net> writes:
>>
>>> On 04/25/2011 03:30 PM, Tom Lane wrote:
>>>
>>>> *Ouch*. Really? It's hard to believe that anyone would consider it
>>>> remotely usable for more than toy-sized projects, if you have to list
>>>> all the typedef names on the command line.
>>>>
>>> Looks like BSD does the same. It's just that we hide it in pgindent:
>>>
>> Oh wow, I never noticed that. That's going to be a severe problem for
>> the "run it anywhere" goal. The typedefs list is already close to 32K,
>> and is not going anywhere but up. There are already platforms on which
>> a shell command line that long will fail, and I think once we break past
>> 32K we might find it failing on even pretty popular ones.
>>
>>
>>
>
>
> Well, my solution would be to replace pgindent with a perl script (among
> other advantages, it would then run everywhere we build, including Windows),
> and filter the typedefs list so that we only use the ones that appear in
> each file with that file, instead of passing the whole list to each file.
>
>
Can we not setup a automatic mechanism where a submitter can send a patch to
some email id, the patch gets applied on the current HEAD, pgindent is run
and the new patch is sent back to the submitter who can then submit it to
the hackers for review. If the patch does not apply cleanly, the same can
also be emailed back to the submitter.

Thanks,
Pavan

--
Pavan Deolasee
EnterpriseDB http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Joseph Conway 2011-05-02 06:26:05 clog_redo causing very long recovery time
Previous Message Tom Lane 2011-05-02 05:07:19 Re: Select For Update and Left Outer Join