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, const char *to); extern int pgunlink(const char *path); extern int pgsymlink(const char *oldpath, const char *newpath); ... #define rename(from, to) pgrename(from, to) #define unlink(path) pgunlink(path) #define symlink(oldpath, newpath) pgsymlink(oldpath, newpath)
so you should already be calling the junction code on Cygwin. --------------------------------------------------------------------------- Reini Urban wrote: > 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? > > > > What are the prospects of making the junction code work under cygwin? > > Somethink like the attached patch is easier. > Just replace symlink() for dirs with link() #ifdef __CYGWIN__ > > just wait a sec until the tests run through... > (completely fresh build) > -- > Reini Urban > http://xarch.tu-graz.ac.at/home/rurban/ > Index: tablespace.c > =================================================================== > RCS file: /projects/cvsroot/pgsql-server/src/backend/commands/tablespace.c,v > retrieving revision 1.11 > diff -u -b -r1.11 tablespace.c > --- tablespace.c 30 Aug 2004 02:54:38 -0000 1.11 > +++ tablespace.c 4 Oct 2004 18:37:13 -0000 > @@ -349,7 +349,11 @@ > linkloc = (char *) palloc(strlen(DataDir) + 11 + 10 + 1); > sprintf(linkloc, "%s/pg_tblspc/%u", DataDir, tablespaceoid); > > +#ifdef __CYGWIN__ > + if (link(location, linkloc) < 0) > +#else > if (symlink(location, linkloc) < 0) > +#endif > ereport(ERROR, > (errcode_for_file_access(), > errmsg("could not create symbolic link \"%s\": %m", > @@ -976,7 +980,11 @@ > linkloc = (char *) palloc(strlen(DataDir) + 11 + 10 + 1); > sprintf(linkloc, "%s/pg_tblspc/%u", DataDir, xlrec->ts_id); > > +#ifdef __CYGWIN__ > + if (link(location, linkloc) < 0) > +#else > if (symlink(location, linkloc) < 0) > +#endif > { > if (errno != EEXIST) > ereport(ERROR, > > ---------------------------(end of broadcast)--------------------------- > TIP 3: if posting/reading through Usenet, please send an appropriate > subscribe-nomail command to [EMAIL PROTECTED] so that your > message can get through to the mailing list cleanly -- Bruce Momjian | http://candle.pha.pa.us [EMAIL PROTECTED] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073 ---------------------------(end of broadcast)--------------------------- TIP 9: the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match