Re: plpgsql test layout

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: plpgsql test layout
Дата
Msg-id CAB7nPqQY6t6kc=yT8WrPN2SajB_3V=W2O86u3QDbD3BGrYXEBA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: plpgsql test layout  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: plpgsql test layout  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
Список pgsql-hackers
On Wed, Dec 13, 2017 at 6:36 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
>> On 12/11/17 19:29, Michael Paquier wrote:
>>> If I read vcregress.pl correctly, it seems to me that you need to do
>>> more with MSVC (see plcheck). The tests would kick if sql/ and
>>> expected/ are found, and the test list is fetched by looking at
>>> REGRESSION in the test files. However plpgsql code has an additional
>>> src/ folder which would cause the tests to not execute. If plpgsql
>>> code was moved on folder down then the tests would execute properly.
>
>> OK, I hacked something up for MSVC.  How about this?
>
> Looks ok to me, though I'm not in a position to actually test the
> msvc changes.

I have been able to check that the tests are working correctly, but
after changing the references from relative paths to absolute paths,
or my environment is complaining about ".." not being a command...

                my $status = $? >> 8;
                exit $status if $status;
                chdir "..";
+               chdir ".." if $lang eq 'plpgsql';
        }
This can be replaced with a single "$topdir/src/pl".

Attached is what I have some up with, based on Peter's v2.
-- 
Michael

Вложения

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

Предыдущее
От: Thomas Munro
Дата:
Сообщение: Re: Parallel Index Scan vs BTP_DELETED and BTP_HALF_DEAD
Следующее
От: Thomas Munro
Дата:
Сообщение: Re: pg_ctl on windows can't open postmaster.pid: Permission denied