Re: BUG #5339: Version of Perl detected incorrectly

On Mon, Feb 22, 2010 at 02:17:57PM -0700, Alex Hunsaker wrote:
> On Mon, Feb 22, 2010 at 13:07, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > Alex Hunsaker <badalex(at)gmail(dot)com> writes:
> >> !   perl_version_error=`$PERL -e 'use 5.00801;' 2>&1`
> >
> > This is not a path towards an acceptable solution, as it effectively
> > assumes what we are setting out to prove, namely that we have found
> > a reasonably modern version of perl.  Try it in perl 4...
>
> Well I would if I had it sitting around =). I did try to compile
> one... but chickened out as soon as I hit an error:
> $ make
> `sh cflags perly.o` perly.c
> CCCMD = cc -c -O
> In file included from perly.y:38:
> perl.h:279: error: conflicting types for ‘sys_errlist’
> /usr/include/bits/sys_errlist.h:28: note: previous declaration of
> ‘sys_errlist’ was here
> In file included from perl.h:540,
> from perly.y:38:
> arg.h:685:1: warning: "AF_LOCAL" redefined
> In file included from /usr/include/sys/socket.h:40,
> from /usr/include/netinet/in.h:25,
> from perl.h:224,
> from perly.y:38:
> /usr/include/bits/socket.h:116:1: warning: this is the location of the
> previous definition
>
>
> How about something like the below? Basically If we find "This is
> perl v4" we bail right then. Otherwise we use the version check I
> proposed up-thread? I may have inadvertently used some
> bash/gnu-isms... sorry about that.
>
> *** a/configure
> --- b/configure

Edit configure.in, not configure :)

Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate