Re: pg_regress breaks on msys

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pg_regress breaks on msys
Date: 2006-07-19 18:39:42
Message-ID: 44BE7C6E.5060907@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:

>Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>
>>Tom Lane wrote:
>>
>>
>>>This error message seems pretty thoroughly unhelpful though. Any ideas
>>>what it's unhappy about?
>>>
>>>
>
>
>
>>I think we need to change the pg_regress error messages so that it
>>includes the command string that failed, at least for now.
>>
>>
>
>Done, but I bet it doesn't tell us anything we don't know already.
>
>

Well, we have a result, courtesy of a special run from Stefan:
http://www.pgbuildfarm.org/cgi-bin/show_log.pl?nm=seahorse&dt=2006-07-19%2017:52:41
has:

Command was: ""C:/msys/1.0/home/pgbuild/pgfarmbuild/HEAD/pgsql.804/src/test/regress/./tmp_check/install/C:/msys/1.0/home/pgbuild/pgfarmbuild/HEAD/inst/bin/initdb" -D "C:/msys/1.0/home/pgbuild/pgfarmbuild/HEAD/pgsql.804/src/test/regress/./tmp_check/data" -L "C:/msys/1.0/home/pgbuild/pgfarmbuild/HEAD/pgsql.804/src/test/regress/./tmp_check/install/C:/msys/1.0/home/pgbuild/pgfarmbuild/HEAD/inst/share/postgresql" --noclean --no-locale >"./log/initdb.log" 2>&1"

The second "C:/msys/1.0/" should not be in the path to initdb.

Not sure how to fix.

cheers

andrew

>
>
>>It will be either quoting problem or a vitual path problem, I am pretty
>>sure. The old shell script ran in a bourne-shell-like manner. But
>>calling system() from a C program will call the Windows command shell,
>>where the quoting rules are quite different.
>>
>>
>
>In src/include/port.h we have
>
>/*
> * Win32 needs double quotes at the beginning and end of system()
> * strings. If not, it gets confused with multiple quoted strings.
> * It also requires double-quotes around the executable name and
> * any files used for redirection. Other args can use single-quotes.
> *
> * See the "Notes" section about quotes at:
> * http://home.earthlink.net/~rlively/MANUALS/COMMANDS/C/CMD.HTM
> */
>
>The referenced link seems to be dead :-( but AFAICS the pg_regress code
>is following the stated rules. Also, how is it getting past the "make
>install" step which is quoting things just the same? Puzzling.
>
> regards, tom lane
>
>
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2006-07-19 18:40:17 Re: pg_regress breaks on msys
Previous Message Tom Lane 2006-07-19 18:34:39 Re: pg_regress breaks on msys

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-07-19 18:40:17 Re: pg_regress breaks on msys
Previous Message Tom Lane 2006-07-19 18:34:39 Re: pg_regress breaks on msys