Re: [CF2016-9] Allow spaces in working path on tap-tests

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: [CF2016-9] Allow spaces in working path on tap-tests
Дата
Msg-id CAB7nPqRSoXRVbQKXH0AH3M7wK_rp-ZFPi+qevmX2ajn7jQRGkQ@mail.gmail.com
обсуждение исходный текст
Ответ на [CF2016-9] Allow spaces in working path on tap-tests  (Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>)
Ответы Re: [CF2016-9] Allow spaces in working path on tap-tests  (Michael Paquier <michael.paquier@gmail.com>)
Список pgsql-hackers
On Mon, Jul 4, 2016 at 4:02 PM, Kyotaro HORIGUCHI
<horiguchi.kyotaro@lab.ntt.co.jp> wrote:
> Hello, this is just an entry mail for the next CF.
>
> The tap-test fails when the soruce directoy containing spaces. I
> accidentially hit this by a Jenkins project with the name "test
> project".
>
> The function system_log() is safe for such parameters but
> backup() uses it in wrong way. On the other hand,
> enable_restoring() and enable_archiving() forgets to quote such a
> path containing spaces as already done for Windows. I don't see
> the similar problem in other places.

Good catch, your fix looks good to me. I am noticing as well that the
invocations of pg_ctl, pg_dumpall, pg_upgrade and diff are likely
broken the same way in vcregress.pl.
-- 
Michael



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Renaming more clearly SHA functions in pgcrypto/
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: [CF2016-9] Allow spaces in working path on tap-tests