Remove obsolete replacement system() on darwin
authorPeter Eisentraut <peter_e@gmx.net>
Thu, 18 Aug 2016 16:00:00 +0000 (12:00 -0400)
committerPeter Eisentraut <peter_e@gmx.net>
Thu, 18 Aug 2016 16:00:00 +0000 (12:00 -0400)
Per comment in the file, this was fixed around OS X 10.2.

src/backend/port/Makefile
src/backend/port/darwin/Makefile [deleted file]
src/backend/port/darwin/README [deleted file]
src/backend/port/darwin/system.c [deleted file]

index 89549d0d2b744e99305189ebcc850385c7915d36..aba1e92fe1fc4f743773072c80a973595d4f273b 100644 (file)
@@ -23,9 +23,6 @@ include $(top_builddir)/src/Makefile.global
 
 OBJS = atomics.o dynloader.o pg_sema.o pg_shmem.o $(TAS)
 
-ifeq ($(PORTNAME), darwin)
-SUBDIRS += darwin
-endif
 ifeq ($(PORTNAME), win32)
 SUBDIRS += win32
 endif
@@ -44,5 +41,4 @@ endif
 
 distclean clean:
        rm -f tas_cpp.s
-       $(MAKE) -C darwin clean
        $(MAKE) -C win32 clean
diff --git a/src/backend/port/darwin/Makefile b/src/backend/port/darwin/Makefile
deleted file mode 100644 (file)
index 9d463ff..0000000
+++ /dev/null
@@ -1,17 +0,0 @@
-#-------------------------------------------------------------------------
-#
-# Makefile--
-#    Makefile for port/darwin
-#
-# IDENTIFICATION
-#    src/backend/port/darwin/Makefile
-#
-#-------------------------------------------------------------------------
-
-subdir = src/backend/port/darwin
-top_builddir = ../../../..
-include $(top_builddir)/src/Makefile.global
-
-OBJS = system.o
-
-include $(top_srcdir)/src/backend/common.mk
diff --git a/src/backend/port/darwin/README b/src/backend/port/darwin/README
deleted file mode 100644 (file)
index 2d9df79..0000000
+++ /dev/null
@@ -1,36 +0,0 @@
-src/backend/port/darwin/README
-
-Darwin
-======
-
-The file system.c included herein is taken directly from Apple's Darwin
-open-source CVS archives, and is redistributed under the BSD copyright
-notice it bears.  (According to Apple's CVS logs, their version is
-identical to the FreeBSD original.)  It provides our own implementation of
-the system(3) function, which ought by all rights to be identical to the
-one provided in libc on Darwin machines.  Nonetheless, this version works,
-whereas the one that actually ships with Mac OS X 10.1 doesn't.  The
-shipped version appears to disconnect the calling process from any shared
-memory segments it is attached to.  (The symptom seen in PostgreSQL is
-that a backend attempting to execute CREATE DATABASE core-dumps.)  I would
-love to know why there is a discrepancy between the published source and
-the actual behavior --- tgl 7-Nov-2001.
-
-Appropriate bug reports have been filed with Apple --- see
-Radar Bug#s 2767956, 2683531, 2805147.  One hopes we can retire this
-kluge in the not too distant future.
-
-
-As of PostgreSQL 7.3 and Mac OS X 10.1, one should expect warnings
-like these while linking the backend:
-
-/usr/bin/ld: warning unused multiple definitions of symbol _system
-port/SUBSYS.o definition of _system in section (__TEXT,__text)
-/usr/lib/libm.dylib(system.o) unused definition of _system
-
-These are due to overriding system() per the above-described hack.
-
-
-The bug appears to be repaired in OS X 10.2.6 and later (possibly in
-earlier 10.2.* as well, but no systems handy to check).  We #ifdef out
-the substitute system() definition on 10.3 and later.
diff --git a/src/backend/port/darwin/system.c b/src/backend/port/darwin/system.c
deleted file mode 100644 (file)
index 1cd5266..0000000
+++ /dev/null
@@ -1,104 +0,0 @@
-/*
- * src/backend/port/darwin/system.c
- *
- * only needed in OS X 10.1 and possibly early 10.2 releases */
-#include <AvailabilityMacros.h> /* pgrminclude ignore */
-#if MAC_OS_X_VERSION_MAX_ALLOWED <= MAC_OS_X_VERSION_10_2 || !defined(MAC_OS_X_VERSION_10_2)
-
-/*
- * Copyright (c) 1988, 1993
- *     The Regents of the University of California.  All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in the
- *       documentation and/or other materials provided with the distribution.
- * 3. Neither the name of the University nor the names of its contributors
- *       may be used to endorse or promote products derived from this software
- *       without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- *
- * $FreeBSD: src/lib/libc/stdlib/system.c,v 1.6 2000/03/16 02:14:41 jasone Exp $
- */
-
-#if defined(LIBC_SCCS) && !defined(lint)
-static char sccsid[] = "@(#)system.c   8.1 (Berkeley) 6/4/93";
-#endif   /* LIBC_SCCS and not lint */
-
-#include <sys/wait.h>
-#include <signal.h>
-#include <unistd.h>
-#include <paths.h>
-#include <errno.h>
-
-int                    system(const char *command);
-
-int
-system(const char *command)
-{
-       pid_t           pid;
-       int                     pstat;
-       struct sigaction ign,
-                               intact,
-                               quitact;
-       sigset_t        newsigblock,
-                               oldsigblock;
-
-       if (!command)                           /* just checking... */
-               return (1);
-
-       /*
-        * Ignore SIGINT and SIGQUIT, block SIGCHLD. Remember to save existing
-        * signal dispositions.
-        */
-       ign.sa_handler = SIG_IGN;
-       (void) sigemptyset(&ign.sa_mask);
-       ign.sa_flags = 0;
-       (void) sigaction(SIGINT, &ign, &intact);
-       (void) sigaction(SIGQUIT, &ign, &quitact);
-       (void) sigemptyset(&newsigblock);
-       (void) sigaddset(&newsigblock, SIGCHLD);
-       (void) sigprocmask(SIG_BLOCK, &newsigblock, &oldsigblock);
-       switch (pid = fork())
-       {
-               case -1:                                /* error */
-                       break;
-               case 0:                 /* child */
-
-                       /*
-                        * Restore original signal dispositions and exec the command.
-                        */
-                       (void) sigaction(SIGINT, &intact, NULL);
-                       (void) sigaction(SIGQUIT, &quitact, NULL);
-                       (void) sigprocmask(SIG_SETMASK, &oldsigblock, NULL);
-                       execl(_PATH_BSHELL, "sh", "-c", command, (char *) NULL);
-                       _exit(127);
-               default:                                /* parent */
-                       do
-                       {
-                               pid = wait4(pid, &pstat, 0, (struct rusage *) 0);
-                       } while (pid == -1 && errno == EINTR);
-                       break;
-       }
-       (void) sigaction(SIGINT, &intact, NULL);
-       (void) sigaction(SIGQUIT, &quitact, NULL);
-       (void) sigprocmask(SIG_SETMASK, &oldsigblock, NULL);
-       return (pid == -1 ? -1 : pstat);
-}
-
-#endif   /* OS X < 10.3 */