Re: [CYGWIN] [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-18 Thread Leeuw van der, Tim
ROTECTED] Subject: Re: [CYGWIN] [HACKERS] open item: tablespace handing in pg_dump/pg_restore Bruce Momjian <[EMAIL PROTECTED]> writes: > OK, I have applied the following patch that uses Cygwin native symlink() > instead of the Win32 junctions. The reason for this is that Cygwin >

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-13 Thread Magnus Hagander
>> > > > OK, I have applied the following patch that uses Cygwin native >> > > > symlink() instead of the Win32 junctions. The reason >for this is >> > > > that Cygwin symlinks work on Win95/98/ME where >junction points do >> > > > not >> > > >> > > Is this really a Win95/98/ME vs NT distinc

Re: [CYGWIN] [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-13 Thread Reini Urban
to other machines. -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Behalf Of Tom Lane Sent: Tuesday, October 12, 2004 1:02 AM To: Bruce Momjian Cc: Reini Urban; PostgreSQL Developers; [EMAIL PROTECTED] Subject: Re: [CYGWIN] [HACKERS] open item: tablespace handing in pg_dum

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-12 Thread Bruce Momjian
Magnus Hagander wrote: > > > > OK, I have applied the following patch that uses Cygwin native > > > > symlink() instead of the Win32 junctions. The reason for this is > > > > that Cygwin symlinks work on Win95/98/ME where junction points do > > > > not > > > > > > Is this really a Win95/98/ME

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-12 Thread Reini Urban
Bruce Momjian schrieb: Greg Stark wrote: Bruce Momjian <[EMAIL PROTECTED]> writes: OK, I have applied the following patch that uses Cygwin native symlink() instead of the Win32 junctions. The reason for this is that Cygwin symlinks work on Win95/98/ME where junction points do not Is this really a

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-12 Thread Greg Stark
"Magnus Hagander" <[EMAIL PROTECTED]> writes: > No, it's file system. > Junctions will not work on NT OS:es with FAT32. > Directory junctions require NTFSv5, which is only available on Windows > 2000 and newer. So then there really has to be a run-time check for this. Either at initdb time, or a

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-12 Thread Magnus Hagander
> > > OK, I have applied the following patch that uses Cygwin native > > > symlink() instead of the Win32 junctions. The reason for this is > > > that Cygwin symlinks work on Win95/98/ME where junction points do > > > not > > > > Is this really a Win95/98/ME vs NT distinction or a FAT32 > vs

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-11 Thread Bruce Momjian
Greg Stark wrote: > Bruce Momjian <[EMAIL PROTECTED]> writes: > > > OK, I have applied the following patch that uses Cygwin native symlink() > > instead of the Win32 junctions. The reason for this is that Cygwin > > symlinks work on Win95/98/ME where junction points do not > > Is this really a

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-11 Thread Greg Stark
Bruce Momjian <[EMAIL PROTECTED]> writes: > OK, I have applied the following patch that uses Cygwin native symlink() > instead of the Win32 junctions. The reason for this is that Cygwin > symlinks work on Win95/98/ME where junction points do not Is this really a Win95/98/ME vs NT distinction or

Re: [CYGWIN] [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-11 Thread Reini Urban
Bruce Momjian schrieb: Tom Lane wrote: Bruce Momjian <[EMAIL PROTECTED]> writes: OK, I have applied the following patch that uses Cygwin native symlink() instead of the Win32 junctions. The reason for this is that Cygwin symlinks work on Win95/98/ME where junction points do not and we have no way

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-11 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian <[EMAIL PROTECTED]> writes: > > OK, I have applied the following patch that uses Cygwin native symlink() > > instead of the Win32 junctions. The reason for this is that Cygwin > > symlinks work on Win95/98/ME where junction points do not and we have no > > way to kn

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-11 Thread Tom Lane
Bruce Momjian <[EMAIL PROTECTED]> writes: > OK, I have applied the following patch that uses Cygwin native symlink() > instead of the Win32 junctions. The reason for this is that Cygwin > symlinks work on Win95/98/ME where junction points do not and we have no > way to know what system will be run

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-11 Thread Bruce Momjian
OK, I have applied the following patch that uses Cygwin native symlink() instead of the Win32 junctions. The reason for this is that Cygwin symlinks work on Win95/98/ME where junction points do not and we have no way to know what system will be running the Cygwin binaries so the safest bet is to

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-07 Thread Reini Urban
Reini Urban schrieb: Tom Lane schrieb: Bruce Momjian <[EMAIL PROTECTED]> writes: I am confused. CVS has in port.h: so you should already be calling the junction code on Cygwin. true. didn't thought of that. very strange. Yeah, I'm sure he is, but it looks from the regression results like it doesn'

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Reini Urban
Tom Lane schrieb: Bruce Momjian <[EMAIL PROTECTED]> writes: I am confused. CVS has in port.h: so you should already be calling the junction code on Cygwin. true. didn't thought of that. very strange. > Yeah, I'm sure he is, but it looks from the regression results like it doesn't quite work on Cy

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Tom Lane
Bruce Momjian <[EMAIL PROTECTED]> writes: > I am confused. CVS has in port.h: > so you should already be calling the junction code on Cygwin. Yeah, I'm sure he is, but it looks from the regression results like it doesn't quite work on Cygwin. Is that fixable? If so, we'd have a choice of whethe

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Tom Lane
Reini Urban <[EMAIL PROTECTED]> writes: > Somethink like the attached patch is easier. > Just replace symlink() for dirs with link() #ifdef __CYGWIN__ Wouldn't it be cleaner to #define symlink as link? regards, tom lane ---(end of broadcast)--

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Reini Urban
Reini Urban schrieb: Tom Lane schrieb: Gavin Sherry <[EMAIL PROTECTED]> writes: I though this may have been the problem. configure.in defines HAVE_SYMLINK to 1 if we are win32. It seems that for Reini's case we are setting our template (and PORTNAME) to win32 when I suspect it should be cygwin. An

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Bruce Momjian
I am confused. CVS has in port.h: #if defined(WIN32) || defined(__CYGWIN__) /* * Win32 doesn't have reliable rename/unlink during concurrent access, * and we need special code to do symlinks. */ extern int pgrename(const char *from, c

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Reini Urban
Tom Lane schrieb: Gavin Sherry <[EMAIL PROTECTED]> writes: I though this may have been the problem. configure.in defines HAVE_SYMLINK to 1 if we are win32. It seems that for Reini's case we are setting our template (and PORTNAME) to win32 when I suspect it should be cygwin. Anyone got any ideas? Wh

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Tom Lane
Gavin Sherry <[EMAIL PROTECTED]> writes: > I though this may have been the problem. configure.in defines HAVE_SYMLINK > to 1 if we are win32. It seems that for Reini's case we are setting our > template (and PORTNAME) to win32 when I suspect it should be cygwin. > Anyone got any ideas? What are th

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Gavin Sherry
On Mon, 4 Oct 2004, Reini Urban wrote: > Reini Urban schrieb: > > no HAVE_SYMLINK defined, though CYGWIN should added -DHAVE_SYMLINK. > > oops, sorry for the noise. of course CYGWIN has it defined in pg_config.h. > CYGWIN can only do hardlinks (junctions) on directories of course. > > maybe HAVE_S

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Gavin Sherry
On Mon, 4 Oct 2004, Reini Urban wrote: > Gavin Sherry schrieb: > > On Mon, 4 Oct 2004, Reini Urban wrote: > >>>I cannot recreate on Linux. What platform, etc, are you on? > >> > >>hmm, I'll investigate then. > >> > >>postgresql latest CVS with 2 minor shlib building patches left > >> (added -lpg

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Reini Urban
Reini Urban schrieb: no HAVE_SYMLINK defined, though CYGWIN should added -DHAVE_SYMLINK. oops, sorry for the noise. of course CYGWIN has it defined in pg_config.h. CYGWIN can only do hardlinks (junctions) on directories of course. maybe HAVE_SYMLINKS should be extended to HAVE_DIR_SYMLINKS when you

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Reini Urban
Gavin Sherry schrieb: On Mon, 4 Oct 2004, Reini Urban wrote: I cannot recreate on Linux. What platform, etc, are you on? hmm, I'll investigate then. postgresql latest CVS with 2 minor shlib building patches left (added -lpgport) cygwin-1.5.11 gcc-3.4.1 Hmm.. sounds like we're trying to support ta

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Magnus Hagander
> > >>But the regression test fails: (the only failing test > against cvs > > HEAD) > > >>This is not only a pg_dump/pg_restore issue, or? > > >> > > >>-- Will fail with bad path > > >>CREATE TABLESPACE badspace LOCATION '/no/such/location'; > > >>ERROR: could not set permissions on directory

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Gavin Sherry
On Mon, 4 Oct 2004, Reini Urban wrote: > Gavin Sherry schrieb: > > On Mon, 4 Oct 2004, Reini Urban wrote: > >>But the regression test fails: (the only failing test against cvs HEAD) > >>This is not only a pg_dump/pg_restore issue, or? > >> > >>-- Will fail with bad path > >>CREATE TABLESPACE bad

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Reini Urban
Gavin Sherry schrieb: On Mon, 4 Oct 2004, Reini Urban wrote: >>But the regression test fails: (the only failing test against cvs HEAD) This is not only a pg_dump/pg_restore issue, or? -- Will fail with bad path CREATE TABLESPACE badspace LOCATION '/no/such/location'; ERROR: could not set permissi

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Gavin Sherry
On Mon, 4 Oct 2004, Reini Urban wrote: > But the regression test fails: (the only failing test against cvs HEAD) > This is not only a pg_dump/pg_restore issue, or? > > -- Will fail with bad path > CREATE TABLESPACE badspace LOCATION '/no/such/location'; > ERROR: could not set permissions on direc

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-10-04 Thread Reini Urban
Bruce Momjian schrieb: Fabien COELHO wrote: Dear hackers, I'm happy to do the pg_dump changes, assuming Tom gets the SET stuff sorted out. ISTM that the tablespace handling or ignoring in pg_dump/pg_restore is still an open issue in current CVS head... waiting for a proper implementation after the

Re: [HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-09-01 Thread Bruce Momjian
Fabien COELHO wrote: > > Dear hackers, > > > I'm happy to do the pg_dump changes, assuming Tom gets the SET stuff sorted > > out. > > ISTM that the tablespace handling or ignoring in pg_dump/pg_restore is > still an open issue in current CVS head... waiting for a proper > implementation after t

[HACKERS] open item: tablespace handing in pg_dump/pg_restore

2004-09-01 Thread Fabien COELHO
Dear hackers, > I'm happy to do the pg_dump changes, assuming Tom gets the SET stuff sorted > out. ISTM that the tablespace handling or ignoring in pg_dump/pg_restore is still an open issue in current CVS head... waiting for a proper implementation after the brain-storming on what seemed to be