Re: Patch: Remove all declarations from pg_attribute.h, consolidate BKI scripts

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers(at)postgresql(dot)org, John Naylor <jcnaylor(at)gmail(dot)com>
Subject: Re: Patch: Remove all declarations from pg_attribute.h, consolidate BKI scripts
Date: 2009-12-21 03:55:55
Message-ID: 603c8f070912201955p4c2f9f91gcc8b7b029f8fc654@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Dec 20, 2009 at 10:23 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
>> On Monday 21 December 2009 02:23:39 Robert Haas wrote:
>>> A more important point is whether we really need to make this
>>> dependent on Perl 5.6 or later.
>
>> I dont see a platform without perl 5.6 where a new enough flex/bison is
>> available...
>
> That argument only holds water if you are willing to follow the same
> rules as we use for flex/bison, ie, they are not needed to build from
> a source tarball.  Otherwise this *is* moving the goalposts on required
> tool support.

I believe that we have long had agreement on making the relevant files
distprep targets, so this will not be an issue. Anyway, the whole
thing is a silly argument anyway: we can certainly make this
compatible back even as far as Perl 5.0 if need be for very little
extra work.

What is worth a little bit of effort to establish is exactly what
version of Perl we're already depending on, so that we can document
that for the benefit of future tool writers. There's no reason why
this particular thing needs to be compatible further back than what is
already required otherwise.

...Robert

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2009-12-21 04:11:25 Re: Possible patch for better index name choosing
Previous Message John Naylor 2009-12-21 03:54:23 Re: Patch: Remove all declarations from pg_attribute.h, consolidate BKI scripts