| 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: | Whole Thread | Raw Message | 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? |