Re: PATCH: Configurable file mode mask

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: David Steele <david(at)pgmasters(dot)net>
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 22:51:52
Message-ID: 24361.1520290312@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2018-03-05 22:55:51 Re: [HACKERS] user-defined numeric data types triggering ERROR: unsupported type
Previous Message Stephen Frost 2018-03-05 22:50:10 Re: 2018-03 CFM