From 21e3edd6cac14f121867f4f9578db34eba7b2492 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 10 Nov 2009 18:53:38 +0000 Subject: [PATCH] Revert the temporary patch to work around Snow Leopard readdir() bug. Apple has fixed that bug in 10.6.2, and we should encourage users to update to that version rather than trusting this cosmetic patch. As was recently noted by Stephen Tyler, this patch was only masking the problem in the context of DROP TABLESPACE, but the failure could occur in other places such as pg_xlog cleanup. --- src/backend/commands/tablespace.c | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/src/backend/commands/tablespace.c b/src/backend/commands/tablespace.c index eef2eb61e9..595fb330b6 100644 --- a/src/backend/commands/tablespace.c +++ b/src/backend/commands/tablespace.c @@ -37,7 +37,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/tablespace.c,v 1.62 2009/09/12 15:51:52 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/commands/tablespace.c,v 1.63 2009/11/10 18:53:38 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -612,17 +612,6 @@ remove_tablespace_directories(Oid tablespaceoid, bool redo) subfile))); pfree(subfile); - - /* - * OS X 10.6 and 10.6.1 have a bug in readdir() that causes the - * next call to fail after deleting the current element. Hopefully - * that will be fixed real soon, but for the moment we have this - * ugly kluge to restart the directory scan. - */ -#ifdef __darwin__ - FreeDir(dirdesc); - dirdesc = AllocateDir(location); -#endif } FreeDir(dirdesc); -- 2.40.0