From: Andrew Dunstan <andrew@dunslane.net>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-hackers@postgresql.org
Subject: Re: [COMMITTERS] pgsql: Initialise perl library as documented in perl API.
Date: Thu, 04 Jun 2009 13:57:00 -0400
Message-ID: <4A280AEC.8090007@dunslane.net> (raw)
In-Reply-To: <1340.1244136781@sss.pgh.pa.us>
References: <20090604155955.545F775331E@cvs.postgresql.org>
<26709.1244133504@sss.pgh.pa.us>
<4A27FF9E.50305@dunslane.net>
<1340.1244136781@sss.pgh.pa.us>
Tom Lane wrote:
> Andrew Dunstan <andrew@dunslane.net> writes:
>
>> That's what we have a buildfarm for ;-) There's a failure on FBSD too
>> by the look of it. I'll dig some more to see what I can find.
>>
>
> I see this when building HEAD on Fedora 10:
>
> plperl.c: In function 'plperl_init_interp':
> plperl.c:450: warning: null argument where non-null required (argument 3)
>
> Seems like a good hint ...
>
>
>
Yeah. I didn't get that. But the odd thing is that on 5.8 especially it
shouldn't matter.
perl 5.8.8's perl.h has:
#ifndef PERL_SYS_INIT3
# define PERL_SYS_INIT3(argvp,argcp,envp) PERL_SYS_INIT(argvp,argcp)
#endif
and the only place it's defined elsewhere that I can see is for OS2 (for
anyone still running it!). These two Unixware machines have 5.8.8. and
the FBSD machine has 5.8.7. So surely it can't be that, unless I'm
missing something badly.
The unixish.h file has this on 5.8:
# define PERL_SYS_INIT(c,v) MALLOC_CHECK_TAINT2(*c,*v)
PERL_FPU_INIT MALLOC_INIT
I'm guessing the problem is actually somewhere in there.
I could construct a dummy environment to pass to perl to quiet that
warning, or I could even clone the environment - I'm mildly reluctant to
pass the real environment to this, as the perlembed man page blandly
tells us that this macro might mangle what is passed to it (even though
we know on 5.8 it doesn't).
cheers
andrew
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-committers@postgresql.org
Cc: andrew@dunslane.net, tgl@sss.pgh.pa.us, pgsql-hackers@postgresql.org
Subject: Re: [COMMITTERS] pgsql: Initialise perl library as documented in perl API.
In-Reply-To: <4A280AEC.8090007@dunslane.net>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox