Re: CVE-2016-1238 fix breaks (at least) pg_rewind tests

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: CVE-2016-1238 fix breaks (at least) pg_rewind tests
Дата
Msg-id 20160909001705.rgll3bnooc457x56@alap3.anarazel.de
обсуждение исходный текст
Ответ на Re: CVE-2016-1238 fix breaks (at least) pg_rewind tests  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
Ответы Re: CVE-2016-1238 fix breaks (at least) pg_rewind tests  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On 2016-09-08 20:15:46 -0400, Peter Eisentraut wrote:
> On 9/8/16 6:04 PM, Alvaro Herrera wrote:
> > Andres Freund wrote:
> >> On 2016-09-08 18:13:06 -0300, Alvaro Herrera wrote:
> >>> I suppose -I$(srcdir) should be fine.  (Why the quotes?)
> >>
> >> Because quoting correctly seems like a good thing to do? Most people
> >> won't have whitespace in there, but it doesn't seem impossible?
> > 
> > Well, I think they are used without quotes in so many places that doing
> > it here seems rather pointless.
> 
> We don't support build directories with spaces in them, but we support
> installation directories with spaces in them.  So I guess that means
> your point is valid.

Even if not necessary in this specific case, I personally think it's
just a good to quote halfway sensibly. Especially when it's trivial to
do so.



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

Предыдущее
От: Peter Eisentraut
Дата:
Сообщение: Re: CVE-2016-1238 fix breaks (at least) pg_rewind tests
Следующее
От: Andres Freund
Дата:
Сообщение: Re: mdtruncate leaking fd.c handles?