From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Murray Cumming <murrayc(at)murrayc(dot)com>, pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: BUG #7514: postgres -k no longer works with spaces in the path |
Date: | 2012-09-06 18:52:36 |
Message-ID: | CA+TgmobTznOfXb_6faRvBA+GZRJ0yeuNyvZMwvCRk0E9hrRXug@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
On Wed, Sep 5, 2012 at 8:55 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>> Maybe it would be easier if multiple -k options accumulated.
>
> After further thought I'm not very enamored of that concept. We've made
> considerable compromises to ensure that every postmaster command-line
> option corresponds exactly to some GUC parameter; it does not seem to me
> that -k is important enough to deserve an exception.
>
> However, it occurs to me that Murray's original complaint about spaces
> in -k pathnames could be dealt with via very narrow changes in
> SplitDirectoriesString. That function was modeled on
> SplitIdentifierString, with perhaps not enough thought about the
> differences between identifiers and pathnames. In particular, I suggest
> that there's no need to reject embedded spaces in pathnames, and plenty
> of systems on which it's important not to. (Think "Program Files" on
> Windows, for instance.) It probably still makes sense to trim leading
> and trailing whitespace, but let's allow embedded spaces.
>
> If we did that, the new code would be incompatible with the old only for
> paths including commas, leading double quotes, or leading/trailing
> whitespace. I submit that all of those cases are pretty uncommon,
> especially compared to embedded space.
That seems reasonable.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2012-09-06 19:39:04 | Re: BUG #7521: Cannot disable WAL log while using pg_dump |
Previous Message | Robert Haas | 2012-09-06 18:50:05 | Re: Minor inheritance/check bug: Inconsistent behavior |