CVS commit: pkgsrc/mk/patch

["Thomas Klausner" <wiz%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   wiz
Date:           Tue Feb 18 15:26:11 UTC 2025

Modified Files:
        pkgsrc/mk/patch: bsd.patch-vars.mk

Log Message:
mk: Optimise patch checks.

Previously a shell was forked to see if there were patch-* files, and
due to logic inefficiencies this was actually called twice.  Instead,
just check for the existence of the patches directory.

>From Jonathan Perkin <jperkin%smartos.org@localhost>
via drecklypkg commit 083ee75b06d730d9379b96217a0bdafc3429e081


To generate a diff of this commit:
cvs rdiff -u -r1.8 -r1.9 pkgsrc/mk/patch/bsd.patch-vars.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/patch/bsd.patch-vars.mk
diff -u pkgsrc/mk/patch/bsd.patch-vars.mk:1.8 pkgsrc/mk/patch/bsd.patch-vars.mk:1.9
--- pkgsrc/mk/patch/bsd.patch-vars.mk:1.8       Fri Oct 11 12:53:13 2024
+++ pkgsrc/mk/patch/bsd.patch-vars.mk   Tue Feb 18 15:26:11 2025
@@ -1,4 +1,4 @@
-# $NetBSD: bsd.patch-vars.mk,v 1.8 2024/10/11 12:53:13 jperkin Exp $
+# $NetBSD: bsd.patch-vars.mk,v 1.9 2025/02/18 15:26:11 wiz Exp $
 #
 # This Makefile fragment is included separately by bsd.pkg.mk and
 # defines some variables which must be defined earlier than where
@@ -27,12 +27,11 @@
 
 PATCHFILES?=   # none
 
-pkgsrc_patches=        ${:!echo ${PATCHDIR}/patch-* ${PATCHDIR}/emul-*-patch-*!:N*\*}
 .if !empty(LOCALPATCHES)
 local_patches= ${:!echo ${LOCALPATCHES}/${PKGPATH}/*!:N*/CVS:N*/\*}
 .endif
 
-.if !empty(PATCHFILES) || !empty(pkgsrc_patches) || !empty(local_patches)
+.if !empty(PATCHFILES) || exists(${PATCHDIR}) || !empty(local_patches)
 USE_TOOLS+=    patch
 .endif
 .if (!empty(PATCHFILES) || !empty(pkgsrc_patches)) \