[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#72225] [PATCH] _AM_PROG_CC_C_O: clean up after .dSYM directories pr
From: |
Zack Weinberg |
Subject: |
[bug#72225] [PATCH] _AM_PROG_CC_C_O: clean up after .dSYM directories properly |
Date: |
Sun, 21 Jul 2024 10:16:04 -0400 |
User-agent: |
Cyrus-JMAP/3.11.0-alpha0-568-g843fbadbe-fm-20240701.003-g843fbadb |
On Sun, Jul 21, 2024, at 5:20 AM, Eric Gallager via Patches for Automake wrote:
> On darwin, the compiler will generate .dSYM directories (for debug
> info) alongside executables when invoked, leading to messages like the
> following in configure output:
>
> checking whether /usr/local/bin/gcc understands -c and -o together...
> rm: conftest.dSYM: is a directory
> yes
>
> The attached patch ought to fix this by passing the -r flag to the rm
> invocation that's meant to clean up the conftests. One concern,
> though, is that in the same invocation, it also tries to remove
> anything called "core", so I worry what might happen with this change
> if "core" happens to be a directory... should I handle that
> separately?
Tests generated by autoconf proper use this formula for cleaning up after
link tests:
rm -f -r a.out a.exe b.out conftest.$ac_ext conftest.o conftest.obj
conftest.dSYM
I'd suggest that _AM_PROG_CC_C_O should be aligned with this - which means
*not* deleting anything named 'core'. You may or may not have $ac_ext
available;
even if you don't I'd suggest that you also make the 'conftest*' glob more
specific (without having looked at the rest of am-prog-cc-c-o.m4, something
like 'conftest.c conftest.o conftest.obj conftest.dSYM' is probably close).
zw