Re: [COMMITTERS] pgsql: Fix breakage from earlier plperl fix.

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Alex Hunsaker <badalex(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: [COMMITTERS] pgsql: Fix breakage from earlier plperl fix.
Date: 2012-01-15 21:21:53
Message-ID: 4F134371.7070305@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On 01/13/2012 07:09 PM, Alex Hunsaker wrote:
> On Fri, Jan 13, 2012 at 16:07, Andrew Dunstan<andrew(at)dunslane(dot)net> wrote:
>>
>> On 01/12/2012 09:28 PM, Alex Hunsaker wrote:
>>> Util.c/o not depending on plperl_helpers.h was also throwing me for a loop
>>> so I fixed it and SPI.c... Thoughts?
>>
>> Basically looks good, but I'm confused by this:
>>
>> do language plperl $$ elog('NOTICE', ${^TAINT}); $$;
>>
>>
>>
>> Why is NOTICE quoted here? It's constant that we've been careful to set up.
> No reason.
>
> [ Err well It was just part of me flailing around while trying to
> figure out why elog was still crashing even thought I had the issue
> fixed. The real problem was Util.o was not being recompiled due to
> Util.c not depending on plperl_helpers.h) ]

Except that it doesn't work. The above produces no output in the
regression tests.

I've committed it with that changed and result file changed accordingly.

cheers

andrew

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Fujii Masao 2012-01-16 01:44:14 Re: pgsql: Correctly initialise shared recoveryLastRecPtr in recovery.
Previous Message Andrew Dunstan 2012-01-15 21:21:44 pgsql: Improve efficiency of recent changes to plperl's sv2cstr().

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Janes 2012-01-15 21:46:30 Re: Standalone synchronous master
Previous Message Heikki Linnakangas 2012-01-15 21:17:54 Re: Vacuum rate limit in KBps