<pre style='margin:0'>
Christopher Nielsen (mascguy) pushed a commit to branch master
in repository macports-legacy-support.

</pre>
<p><a href="https://github.com/macports/macports-legacy-support/commit/4defe2871f0c9849b0acb33b41b00a4c55e569e3">https://github.com/macports/macports-legacy-support/commit/4defe2871f0c9849b0acb33b41b00a4c55e569e3</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 4defe2871f0c9849b0acb33b41b00a4c55e569e3
</span>Author: Fred Wright <fw@fwright.net>
AuthorDate: Fri Nov 15 22:02:40 2024 -0800

<span style='display:block; white-space:pre;color:#404040;'>    Makefile: Use C-based 'which' on Tiger.
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Unlike the 10.5+ 'which' command, which is a C program, the
</span><span style='display:block; white-space:pre;color:#404040;'>    OS-supplied 10.4 version is a csh script, with some deficiencies.
</span><span style='display:block; white-space:pre;color:#404040;'>    Since 2019, legacy-support has been supplying a patched version
</span><span style='display:block; white-space:pre;color:#404040;'>    of the csh script correcting some, but not all, deficiencies.  In
</span><span style='display:block; white-space:pre;color:#404040;'>    particular, the replacement version doesn't implement the '-a'
</span><span style='display:block; white-space:pre;color:#404040;'>    option, without providing an informative error message.
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    This change replaces the patched csh script with a build of the
</span><span style='display:block; white-space:pre;color:#404040;'>    10.5 version.  For consistency, it has the same universality as
</span><span style='display:block; white-space:pre;color:#404040;'>    the Apple-supplied programs, regardless of whether legacy-support
</span><span style='display:block; white-space:pre;color:#404040;'>    is being built universal in general.  Specifically, it uses the
</span><span style='display:block; white-space:pre;color:#404040;'>    new binarchs tool to copy the architectures from /usr/bin/true,
</span><span style='display:block; white-space:pre;color:#404040;'>    since full universal builds don't work on 10.4 ppc.
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    TESTED:
</span><span style='display:block; white-space:pre;color:#404040;'>    Built and tested with all option combinations.
</span><span style='display:block; white-space:pre;color:#404040;'>    Tested the updated install target.
</span>---
 Makefile | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/Makefile b/Makefile
</span><span style='display:block; white-space:pre;color:#808080;'>index 65a9ea2..62a3836 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/Makefile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/Makefile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -177,8 +177,12 @@ MANTESTPRGS      = $(MANTESTPRGS_C) $(MANTESTPRGS_CPP)
</span> MANTESTRUNS     := $(patsubst \
                      $(MANTESTPREFIX)%,$(MANRUNPREFIX)%,$(MANTESTPRGS))
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-TIGERBINDIR      = tiger_only/bin
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-TIGERBINS       := $(wildcard $(TIGERBINDIR)/*)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+TIGERSRCDIR      = tiger_only/src
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+TIGERSRCS       := $(wildcard $(TIGERSRCDIR)/*.c)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+TIGERPRGS       := $(patsubst %.c,%,$(TIGERSRCS))
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+TOOLDIR          = tools
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ARCHTOOL         = $(TOOLDIR)/binarchs.sh
</span> 
 define splitandfilterandmergemultiarch
        output='$(1)' && \
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -387,9 +391,10 @@ $(MANTESTOBJS_CPP): %.o: %.cpp $(ALLHEADERS)
</span> $(MANTESTPRGS_CPP): %: %.o $(BUILDDLIBPATH)
        $(CXX) $(TESTLDFLAGS) $< $(TESTLIBS) -o $@
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# Dummy target for building Tiger-only binaries, so Portfile can
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# reference it in case we need it in the future.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-tiger-bins:
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+$(TIGERPRGS): %: %.c
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+   $(CC) $$($(ARCHTOOL)) $< -o $@
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+tiger-bins: $(TIGERPRGS)
</span> 
 # Special clause for testing the cmath fix: Just need to verify that
 # building succeeds or fails, not that the executable runs or what it
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -491,8 +496,8 @@ install-syslib: $(BUILDSYSLIBPATH) | $(DESTDIR)$(LIBDIR)
</span> install-slib: $(BUILDSLIBPATH) | $(DESTDIR)$(LIBDIR)
        $(INSTALL_DATA) $(BUILDSLIBPATH) $(DESTDIR)$(LIBDIR)
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-install-tiger: $(TIGERBINS)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-   $(INSTALL_PROGRAM) $(TIGERBINS) $(DESTDIR)$(BINDIR)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+install-tiger: $(TIGERPRGS)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+   $(INSTALL_PROGRAM) $(TIGERPRGS) $(DESTDIR)$(BINDIR)
</span> 
 test check: $(TESTRUNS) $(XTESTRUNS) test_cmath test_faccessat_setuid_msg
 
</pre><pre style='margin:0'>

</pre>