From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | David Steele <david(at)pgmasters(dot)net> |
Cc: | pgsql-hackers(at)postgresql(dot)org, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Subject: | Re: perlcritic |
Date: | 2017-03-22 13:55:49 |
Message-ID: | 8E6316C5-5DDA-42A0-8D85-CB45E6CB2698@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 21 Mar 2017, at 19:20, David Steele <david(at)pgmasters(dot)net> wrote:
>
> On 3/6/17 12:02 PM, Dagfinn Ilmari Mannsåker wrote:
>> ilmari(at)ilmari(dot)org (Dagfinn Ilmari Mannsåker) writes:
>>
>>> Hi Peter,
>>>
>>> Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> writes:
>>>
>>>> I posted this about 18 months ago but then ran out of steam. [ ] Here
>>>> is an updated patch. The testing instructions below still apply.
>>>> Especially welcome would be ideas on how to address some of the places
>>>> I have marked with ## no critic.
>>>
>>> Attached is a patch on top of yours that addresses all the ## no critic
>>> annotations except RequireFilenameMatchesPackage, which can't be fixed
>>> without more drastic reworking of the plperl build process.
>>>
>>> Tested on perl 5.8.1 and 5.24.0 by configuring with --with-perl and
>>> --enable-tap-tests followed by make check-world, and running pgindent
>>> --build.
>>
>> Attached is an updated version of the patch, in which
>> src/tools/msvc/gendef.pl actually compiles. If someone on Windows could
>> test it, that would be great.
>
> You are signed up to review this patch. Do you know when you'll have a chance to do that?
I have on my TODO for today or tomorrow to wrap that up.
cheers ./daniel
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2017-03-22 13:59:23 | Re: Partition-wise join for join between (declaratively) partitioned tables |
Previous Message | Emil Iggland | 2017-03-22 13:47:19 | Fix for grammatical error in code-comment |