From: | David Steele <david(at)pgmasters(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Stephen Frost <sfrost(at)snowman(dot)net>, Adam Brightwell <adam(dot)brightwell(at)crunchydata(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, "Tsunakawa, Takayuki" <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Configurable file mode mask |
Date: | 2018-03-05 23:50:52 |
Message-ID: | 2b9be06d-f79c-87da-975b-0482ef10f4a1@pgmasters.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 3/5/18 5:51 PM, Tom Lane wrote:
> David Steele <david(at)pgmasters(dot)net> writes:
>> On 3/5/18 5:11 PM, Tom Lane wrote:
>>> David Steele <david(at)pgmasters(dot)net> writes:
>>>> I'm not sure what the protocol for introducing a new Perl module is? I
>>>> couldn't find packages for the major OSes. Are we OK with using CPAN?
>
>>> I don't think that's cool. Anything that's not part of a standard Perl
>>> installation is a bit of a lift already, and if it's not packaged by
>>> major distros then it's really a problem for many people. (Yeah, they
>>> may know what CPAN is, but they might have local policy issues about
>>> installing directly from there.)
>
>> It's a little different here, because these packages are only used for
>> testing/development.
>
> True, but if we want this test to be part of either check-world or the
> buildfarm run, that's still a lot of people we're asking to install
> the extra module. If we're talking about saving just a few dozen
> lines of code, it ain't worth it.
+1.
--
-David
david(at)pgmasters(dot)net
From | Date | Subject | |
---|---|---|---|
Next Message | Nikita Glukhov | 2018-03-05 23:51:08 | Re: SQL/JSON: functions |
Previous Message | Nikita Glukhov | 2018-03-05 23:48:38 | Re: jsonpath |