Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified)) by mollari.NetBSD.org (Postfix) with ESMTPS id 483A41A921F for ; Thu, 2 Dec 2021 09:44:05 +0000 (UTC) Received: by mail.netbsd.org (Postfix, from userid 605) id EBD0784E6F; Thu, 2 Dec 2021 09:44:04 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by mail.netbsd.org (Postfix) with ESMTP id 3456A84D7A for ; Thu, 2 Dec 2021 09:44:04 +0000 (UTC) X-Virus-Scanned: amavisd-new at netbsd.org Received: from mail.netbsd.org ([127.0.0.1]) by localhost (mail.netbsd.org [127.0.0.1]) (amavisd-new, port 10025) with ESMTP id FmwNFIJOBas6 for ; Thu, 2 Dec 2021 09:44:03 +0000 (UTC) Received: from cvs.NetBSD.org (ivanova.netbsd.org [199.233.217.197]) by mail.netbsd.org (Postfix) with ESMTP id 7F48F84D35 for ; Thu, 2 Dec 2021 09:44:03 +0000 (UTC) Received: by cvs.NetBSD.org (Postfix, from userid 500) id 2DEB0FAEC; Thu, 2 Dec 2021 09:44:03 +0000 (UTC) Content-Transfer-Encoding: 7bit Content-Type: multipart/mixed; boundary="_----------=_1638438243267840" MIME-Version: 1.0 Date: Thu, 2 Dec 2021 09:44:03 +0000 From: "Jonathan Perkin" Subject: CVS commit: pkgsrc/mk/platform To: pkgsrc-changes@NetBSD.org Reply-To: jperkin@netbsd.org X-Mailer: log_accum Message-Id: <20211202094403.2DEB0FAEC@cvs.NetBSD.org> Sender: pkgsrc-changes-owner@NetBSD.org List-Id: Precedence: bulk List-Unsubscribe: This is a multi-part message in MIME format. --_----------=_1638438243267840 Content-Disposition: inline Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII" 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. --_----------=_1638438243267840 Content-Disposition: inline Content-Length: 902 Content-Transfer-Encoding: binary Content-Type: text/x-diff; charset=us-ascii 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 --_----------=_1638438243267840--