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@sss.pgh.pa.us> wrote:
> > Alex Hunsaker <badalex@gmail.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@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.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