Re: win32 build and test issues

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: pgsql-hackers-win32 <pgsql-hackers-win32(at)postgresql(dot)org>
Subject: Re: win32 build and test issues
Date: 2004-05-01 03:07:48
Message-ID: 200405010307.i4137mI28504@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-hackers-win32 pgsql-patches


I can apply the last part of this patch, but we can't patch configure,
only configure.in. Can you think of a way to patch that instead?

---------------------------------------------------------------------------

Andrew Dunstan wrote:
>
> Below are a couple of diffs.
>
> The first is the fix I made in configure - on my W2K machine with the
> current MinGW/MSys I was getting the previously reported symlink problem
> every time. With the looping patch (yes I *know* it's ugly, and we have
> to get to the root of the problem, but I wanted to get past it for now)
> it never happened, and builds worked. In any case we should use
> something like the last couple of lines at least, to *show* something
> went wrong. As we have it now we say we are doing something and not
> testing for failure.
>
> The second allows "make check" to proceed to almost the end - you still
> need to use the task manager to shut things down.
>
> The regression tests themselves seem to fail for the most part in every
> configuration I have tried. The problem seems to be mostly buffering
> issues - error messages appear out of order from what is expected. I
> have not made sure what passes if we ignore that issue.
>
> cheers
>
> andrew
>
>
>
> Index: configure
> ===================================================================
> RCS file: /projects/cvsroot/pgsql-server/configure,v
> retrieving revision 1.351
> diff -c -w -r1.351 configure
> *** configure 27 Apr 2004 20:09:27 -0000 1.351
> --- configure 29 Apr 2004 20:17:06 -0000
> ***************
> *** 19141,19151 ****
> --- 19141,19160 ----
> esac
>
> # Make a symlink if possible; otherwise try a hard link.
> + for linktry in 1 2 3 4 5; do
> ln -s $ac_rel_source $ac_dest 2>/dev/null ||
> ln $srcdir/$ac_source $ac_dest ||
> { { echo "$as_me:$LINENO: error: cannot link $ac_dest to $srcdir/$ac_source" >&5
> echo "$as_me: error: cannot link $ac_dest to $srcdir/$ac_source" >&2;}
> { (exit 1); exit 1; }; }
> + test -e $ac_dest && break
> + done
> + test -e $ac_dest ||
> + { { echo "$as_me:$LINENO: error: failed to link $ac_dest to $srcdir/$ac_source"
> + >&5
> + echo "$as_me: error: failed to link $ac_dest to $srcdir/$ac_source" >&2;}
> + { (exit 1); exit 1; }; }
> +
> done
> _ACEOF
>
> Index: src/test/regress/pg_regress.sh
> ===================================================================
> RCS file: /projects/cvsroot/pgsql-server/src/test/regress/pg_regress.sh,v
> retrieving revision 1.38
> diff -c -w -r1.38 pg_regress.sh
> *** src/test/regress/pg_regress.sh 8 Jan 2004 20:04:41 -0000 1.38
> --- src/test/regress/pg_regress.sh 29 Apr 2004 20:17:07 -0000
> ***************
> *** 294,305 ****
> if [ x"$temp_install" != x"" ]
> then
> if echo x"$temp_install" | grep -v '^x/' >/dev/null 2>&1; then
> temp_install="`pwd`/$temp_install"
> fi
>
> bindir=$temp_install/install/$bindir
> libdir=$temp_install/install/$libdir
> - pkglibdir=$temp_install/install/$pkglibdir
> datadir=$temp_install/install/$datadir
> PGDATA=$temp_install/data
>
> --- 294,313 ----
> if [ x"$temp_install" != x"" ]
> then
> if echo x"$temp_install" | grep -v '^x/' >/dev/null 2>&1; then
> + case $host_platform in
> + *-*-mingw32*)
> + pkglibdir="`pwd -W`/$temp_install/install/$pkglibdir"
> + temp_install="`pwd`/$temp_install"
> + ;;
> + *)
> temp_install="`pwd`/$temp_install"
> + pkglibdir=$temp_install/install/$pkglibdir
> + ;;
> + esac
> fi
>
> bindir=$temp_install/install/$bindir
> libdir=$temp_install/install/$libdir
> datadir=$temp_install/install/$datadir
> PGDATA=$temp_install/data
>
> ***************
> *** 336,342 ****
> # executables, not dlopen'ed ones)
> # ----------
> case $host_platform in
> ! *-*-cygwin*)
> PATH=$libdir:$PATH
> export PATH
> ;;
> --- 344,350 ----
> # executables, not dlopen'ed ones)
> # ----------
> case $host_platform in
> ! *-*-cygwin* | *-*-mingw32*)
> PATH=$libdir:$PATH
> export PATH
> ;;
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2004-05-01 03:09:15 Re: FW: Timezone library
Previous Message Marc G. Fournier 2004-05-01 02:34:12 Re: Plan for feature freeze?

Browse pgsql-hackers-win32 by date

  From Date Subject
Next Message Bruce Momjian 2004-05-01 03:09:48 Re: win32 build and test issues
Previous Message Andrew Dunstan 2004-05-01 03:01:51 Re: win32 build and test issues

Browse pgsql-patches by date

  From Date Subject
Next Message Andrew Dunstan 2004-05-01 03:08:11 Re: mingw regression test fixes
Previous Message Claudio Natoli 2004-05-01 03:07:35 Re: mingw regression test fixes