From: | Andy Colson <andy(at)squeakycode(dot)net> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgreSQL(dot)org, meskes(at)postgresql(dot)org, kris(at)shannon(dot)id(dot)au |
Subject: | Re: Perl 5.12 complains about ecpg parser-hacking scripts |
Date: | 2011-03-04 16:55:33 |
Message-ID: | 4D711985.4090409@squeakycode.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 3/3/2011 6:49 AM, Michael Meskes wrote:
> On Wed, Mar 02, 2011 at 01:33:35PM -0600, Andy Colson wrote:
>> I thought Kris was going to work on this, but saw no progress, and I
>> was bored the other day, so I started working on it.
>>
>> Here is a parse.pl, with some major refactoring.
>>
>> I named it with a 2 so I could run it beside the original and diff em:
>
> Thanks for all the work.
>
>> I am sure there are new bugs. I have not run it on anything but
>> 9.0.1. Are there other .y files you might feed it? (something other
>> than backend/parser/gram.y?)
>
> I ran it against several versions and it always gave the right output. So i
> decided to just commit it to the archive so we can see if it breaks anything.
> The old script is still in there so in case of a major problem that I cannot
> foresee we can simply change the Makefile back to using parse.pl.
>
> Michael
>
And here is check_rules. Much less change, but I did run it through
perl tidy, so I'll bet most of the lines changed.
-Andy
Attachment | Content-Type | Size |
---|---|---|
check_rules.pl | text/plain | 3.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2011-03-04 17:04:30 | Re: Quick Extensions Question |
Previous Message | hom | 2011-03-04 16:51:30 | Re: Open unmatch source file when step into parse_analyze() in Eclipse? |