CVS commit: pkgsrc/mk/platform

["Jonathan Perkin" <jperkin%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   jperkin
Date:           Thu Dec  2 09:44:03 UTC 2021

Modified Files:
        pkgsrc/mk/platform: Darwin.mk

Log Message:
mk: Remove use of /bin/ksh wrapper hack on Darwin.

There are only a few releases where this appears to work, and the commit logs
show this to be a complete pain to get working.  I've now confirmed it's also
broken on Big Sur where ksh simply segfaults on both x86_64 and arm64.

This may have been marginally useful in the past, but we now have cwrappers to
provide a much bigger performance improvement, and so this hack was only used
during bootstrap to build a small number of packages anyway.


To generate a diff of this commit:
cvs rdiff -u -r1.110 -r1.111 pkgsrc/mk/platform/Darwin.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/mk/platform/Darwin.mk
diff -u pkgsrc/mk/platform/Darwin.mk:1.110 pkgsrc/mk/platform/Darwin.mk:1.111
--- pkgsrc/mk/platform/Darwin.mk:1.110  Mon Nov 29 16:14:23 2021
+++ pkgsrc/mk/platform/Darwin.mk        Thu Dec  2 09:44:03 2021
@@ -1,4 +1,4 @@
-# $NetBSD: Darwin.mk,v 1.110 2021/11/29 16:14:23 jperkin Exp $
+# $NetBSD: Darwin.mk,v 1.111 2021/12/02 09:44:03 jperkin Exp $
 #
 # Variable definitions for the Darwin operating system.
 
@@ -253,13 +253,6 @@ MAKE_ENV+= MACOSX_DEPLOYMENT_TARGET="10.
 CONFIGURE_ENV+=                gl_cv_func_getcwd_abort_bug=no
 .endif
 
-# Use ksh to improve wrapper script performance, except on buggy Mavericks.
-.if exists(/bin/ksh)
-.  if ${OPSYS_VERSION} < 100900 || ${OPSYS_VERSION} >= 101000
-WRAPPER_BIN_SH?=       /bin/ksh
-.  endif
-.endif
-
 # strnlen(3) is available from Lion onwards
 .if ${OPSYS_VERSION} < 100700
 _OPSYS_MISSING_FEATURES+=      strnlen