Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes thereshould be per DATA() line - Mailing list pgsql-hackers

From Jeff Janes
Subject Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes thereshould be per DATA() line
Date
Msg-id CAMkU=1wuQW=xVfu-14A4VCvxO0ohkD3m9vk6HOj_dprQoKNAQw@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes thereshould be per DATA() line  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes thereshould be per DATA() line  (Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>)
Re: [HACKERS] [PATCH] Teach Catalog.pm how many attributes there should be per DATA() line  (ilmari@ilmari.org (Dagfinn Ilmari Mannsåker))
List pgsql-hackers
On Thu, Mar 9, 2017 at 3:20 PM, Robert Haas <robertmhaas@gmail.com> wrote:
On Mon, Mar 6, 2017 at 11:37 AM, Dagfinn Ilmari Mannsåker
<ilmari@ilmari.org> wrote:
> David Christensen <david@endpoint.com> writes:
>>> Hi David,
>>>
>>> Here's a review of your patch.
>>
>> Hi Ilmari, thanks for your time and review.  I’m fine with the revised version.
>
> Okay, I've marked the patch as Ready For Committer.

Committed.   Hopefully this doesn't contain any Perl bits that are
sufficiently new as to cause problems for our older BF members ... I
guess we'll see.


Bad luck there.  I'm getting this error on CentOS6.8, perl v5.10.1

Can't locate object method "input_line_number" via package "IO::Handle" at ../../../src/backend/catalog/Catalog.pm line 82, <INPUT_FILE> line 148.
make[3]: *** [fmgrtab.c] Error 25
make[2]: *** [utils/fmgroids.h] Error 2
make[2]: *** Waiting for unfinished jobs....
Can't locate object method "input_line_number" via package "IO::Handle" at ../../../src/backend/catalog/Catalog.pm line 82, <INPUT_FILE> line 148.
make[3]: *** [postgres.bki] Error 25
make[2]: *** [submake-schemapg] Error 2
make[1]: *** [all-backend-recurse] Error 2
make: *** [all-src-recurse] Error 2


I think we can just save $. and use that, as in the attached.


I as sabotaged a random line in src/include/catalog/pg_amop.h and it seems to report the error correctly.

Cheers,

Jeff




Attachment

pgsql-hackers by date:

Previous
From: Amit Langote
Date:
Subject: Re: [HACKERS] Partitioning vs ON CONFLICT
Next
From: Amit Langote
Date:
Subject: Re: [HACKERS] Partitioning vs ON CONFLICT