Thread: Re: [pgsql-hackers-win32] Testing needed for recent tablespace

Re: [pgsql-hackers-win32] Testing needed for recent tablespace

From
Bruce Momjian
Date:
Yep, related.  There was a typo in win32.h for lstat that was exposed by
the new tablespace changes.  I also added the proper parameters rather
than just redefining the symbol.

Attached is the patch.  Do we need to repackage beta2 for this?  Tom did
ask for Win32 testers two days ago and it is just being tested now.

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

markir@coretech.co.nz wrote:
> Hmmm... not entirely sure if this is related - but I get a compile failure on
> win2000 pro :
>
> gcc -O2 -fno-strict-aliasing -Wall -Wmissing-prototypes -Wmissing-declarations
> -L../../src/port -L/usr/local/lib  -o postgres.exe
> -Wl,--base-file,postgres.base postgres.exp access/SUBSYS.o bootstrap/SUBSYS.o
> catalog/SUBSYS.o parser/SUBSYS.o commands/SUBSYS.o executor/SUBSYS.o
> lib/SUBSYS.o libpq/SUBSYS.o main/SUBSYS.o nodes/SUBSYS.o optimizer/SUBSYS.o
> port/SUBSYS.o postmaster/SUBSYS.o regex/SUBSYS.o rewrite/SUBSYS.o
> storage/SUBSYS.o tcop/SUBSYS.o utils/SUBSYS.o ../../src/timezone/SUBSYS.o
> -lpgport -lz -lwsock32 -lm  -lws2_32
> commands/SUBSYS.o(.text+0x2a827):tablespace.c: undefined reference to `slat'
> make[2]: *** [postgres] Error 1
> make[2]: Leaving directory
> `/home/Administrator/develop/c/postgresql-8.0.0beta1/src/backend'
> make[1]: *** [all] Error 2
> make[1]: Leaving directory
> `/home/Administrator/develop/c/postgresql-8.0.0beta1/src'
> make: *** [all] Error 2
>
> regards
>
> Mark
>
> Quoting Tom Lane <tgl@sss.pgh.pa.us>:
>
> > Would someone check that I didn't break the Windows port with this
> > recent commit:
> >
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
>
>                http://www.postgresql.org/docs/faqs/FAQ.html
>

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
Index: win32.h
===================================================================
RCS file: /cvsroot/pgsql-server/src/include/port/win32.h,v
retrieving revision 1.29
retrieving revision 1.31
diff -c -c -r1.29 -r1.31
*** win32.h    29 Aug 2004 05:06:57 -0000    1.29
--- win32.h    31 Aug 2004 11:29:56 -0000    1.31
***************
*** 206,212 ****
  /*
   * Supplement to <sys/stat.h>.
   */
! #define lstat slat

  /*
   * Supplement to <errno.h>.
--- 206,212 ----
  /*
   * Supplement to <sys/stat.h>.
   */
! #define lstat(path, sb)    stat((path), (sb))

  /*
   * Supplement to <errno.h>.

Re: [pgsql-hackers-win32] Testing needed for recent

From
markir@coretech.co.nz
Date:
Quoting Bruce Momjian <pgman@candle.pha.pa.us>:

>
> Yep, related.  There was a typo in win32.h for lstat that was exposed by
> the new tablespace changes.  I also added the proper parameters rather
> than just redefining the symbol.
>
> Attached is the patch.  Do we need to repackage beta2 for this?  Tom did
> ask for Win32 testers two days ago and it is just being tested now.
>
Apologies for the awful timing - cpu on my windows box died on Saturday, and
only got a replacement going yesterday (I know it sounds a bit like 'the dog
ate my homework'...)

regards

Mark