| From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
|---|---|
| To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Geoghegan <pg(at)heroku(dot)com>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: [COMMITTERS] pgsql: Replace duplicate_oids with Perl implementation |
| Date: | 2013-11-13 13:56:02 |
| Message-ID: | 528384F2.2040903@dunslane.net |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
On 11/13/2013 08:36 AM, Peter Eisentraut wrote:
>> What exactly is your argument, here?
> If we change unused_oids to a Perl implementation, we will add
> additional inconvenience to users who don't have /usr/bin/perl in that
> exact location. Versus allowing use by users who don't have /bin/sh. I
> don't know what the ratio between those two camps is, among potential
> users of unused_oids.
According to my count we currently have 30 instances of scripts with
this shebang line. That includes things not called from makefiles, such
as pgindent.
I don't recall this causing an issue.
For any small group of people that might exists who don't have
/usr/bin/perl, they have a simple workaround to call "perl unused_oids".
Can we please get on with dealing with real problems? I don't believe
this is one.
cheers
andrew
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Kevin Grittner | 2013-11-13 15:10:04 | pgsql: Free ignorelist after each regression test schedule. |
| Previous Message | Peter Eisentraut | 2013-11-13 13:36:38 | Re: [COMMITTERS] pgsql: Replace duplicate_oids with Perl implementation |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2013-11-13 14:07:17 | Re: UTF8 national character data type support WIP patch and list of open issues. |
| Previous Message | Amit Kapila | 2013-11-13 13:53:34 | Re: ALTER SYSTEM SET command to change postgresql.conf parameters (RE: Proposal for Allow postgresql.conf values to be changed via SQL [review]) |