[pkgsrc/trunk]: pkgsrc/mk/platform mk: Remove use of /bin/ksh wrapper hack on...

[jperkin <jperkin%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/2b6c0dafc588
branches:  trunk
changeset: 770536:2b6c0dafc588
user:      jperkin <jperkin%pkgsrc.org@localhost>
date:      Thu Dec 02 09:44:03 2021 +0000

description:
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.

diffstat:

 mk/platform/Darwin.mk |  9 +--------
 1 files changed, 1 insertions(+), 8 deletions(-)

diffs (23 lines):

diff -r 3aeabf99b355 -r 2b6c0dafc588 mk/platform/Darwin.mk
--- a/mk/platform/Darwin.mk     Thu Dec 02 08:16:51 2021 +0000
+++ b/mk/platform/Darwin.mk     Thu Dec 02 09:44:03 2021 +0000
@@ -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 @@
 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