CVS commit: pkgsrc/mk/check

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

Module Name:    pkgsrc
Committed By:   jperkin
Date:           Wed Oct  2 10:49:01 UTC 2024

Modified Files:
        pkgsrc/mk/check: check-shlibs-macho.awk

Log Message:
mk: Fix previous change to check-shlibs-macho.awk.

Last November I made a change to improve the failure output from both
scripts, but while I correctly updated the function arguments in the ELF
version, I somehow missed doing so in the Mach-O version.  Unfortunately
this means that all checks since then have been passing due to the fact
that unset function arguments are permitted and default to "".

Found after comparing behaviour between this script and mktool.  Huge
apologies for the very long time that this has been left broken.


To generate a diff of this commit:
cvs rdiff -u -r1.11 -r1.12 pkgsrc/mk/check/check-shlibs-macho.awk

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

Modified files:

Index: pkgsrc/mk/check/check-shlibs-macho.awk
diff -u pkgsrc/mk/check/check-shlibs-macho.awk:1.11 pkgsrc/mk/check/check-shlibs-macho.awk:1.12
--- pkgsrc/mk/check/check-shlibs-macho.awk:1.11 Mon Nov 27 12:51:38 2023
+++ pkgsrc/mk/check/check-shlibs-macho.awk      Wed Oct  2 10:49:01 2024
@@ -1,4 +1,4 @@
-# $NetBSD: check-shlibs-macho.awk,v 1.11 2023/11/27 12:51:38 jperkin Exp $
+# $NetBSD: check-shlibs-macho.awk,v 1.12 2024/10/02 10:49:01 jperkin Exp $
 
 #
 # Read a list of potential Mach-O binaries from stdin.
@@ -110,7 +110,7 @@ function checkshlib(DSO,    needed, found) 
                                libcache[libfile] = system("test -f " \
                                                           shquote(libfile))
                        if (libcache[libfile] == 0) {
-                               check_pkg(lib)
+                               check_pkg(DSO, lib)
                                found = 1
                                break
                        }