Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 8 Mar 2026 14:56:09 -0700
From:      Ahmad Khalifa <ahmadkhalifa570@gmail.com>
To:        Ed Maste <emaste@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org,  dev-commits-src-main@freebsd.org
Subject:   Re: git: 858f53dd43ec - main - Makefile.inc1: Don't force LLVM_BINUTILS off for cross-tools
Message-ID:  <CAMLT6uD-oACtr6oyynhh-cPM15D_wTz9Xrzn78amXnBq4j-Ddw@mail.gmail.com>
In-Reply-To: <69a88413.1be02.24089406@gitrepo.freebsd.org>

index | next in thread | previous in thread | raw e-mail

On Wed Mar 4, 2026 at 9:12 PM +0200, Ed Maste wrote:
> The branch main has been updated by emaste:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=858f53dd43ecb84cf2597229e9dbda2f242d9dd6
>
> commit 858f53dd43ecb84cf2597229e9dbda2f242d9dd6
> Author:     Ed Maste <emaste@FreeBSD.org>
> AuthorDate: 2026-03-04 15:06:26 +0000
> Commit:     Ed Maste <emaste@FreeBSD.org>
> CommitDate: 2026-03-04 19:10:48 +0000
>
>     Makefile.inc1: Don't force LLVM_BINUTILS off for cross-tools
>
>     Because of this setting we were still using ELF Tool Chain tools for
>     buildworld.  The sets of binary utilities are largely equivalent and
>     this went unnoticed after commit 1cae7121c667 ("Enable LLVM_BINUTILS
>     by default").
>
>     This was discovered recently because ELF Tool Chain objcopy produces
>     standalone debug files without phdrs and this caused an issue with a
>     3rd party ELF parser [1].  Remove the forced setting so that we use
>     LLVM's binutils to build the system.

llvm-objcopy doesn't get built during cross-tools, which results in
cross builds not having objcopy. Not sure if just specifying
llvm-objcopy under cross-tools would fix it, haven't had the time to
check.

See https://github.com/freebsd/freebsd-src/actions/runs/22685122750

Thanks.

>
>     [1] https://sourceware.org/bugzilla/show_bug.cgi?id=33876
>
>     Reviewed by:    imp, jhb
>     Sponsored by:   The FreeBSD Foundation
>     Differential Revision: https://reviews.freebsd.org/D55650
> ---
>  Makefile.inc1 | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/Makefile.inc1 b/Makefile.inc1
> index c4696abae8cd..8a1958902db5 100644
> --- a/Makefile.inc1
> +++ b/Makefile.inc1
> @@ -808,7 +808,6 @@ XMAKE=		${BMAKE} \
>  		TARGET=${TARGET} TARGET_ARCH=${TARGET_ARCH} \
>  		MK_CLANG=${MK_CLANG_BOOTSTRAP} \
>  		MK_LLDB=no \
> -		MK_LLVM_BINUTILS=no \
>  		MK_TESTS=no
>
>  # kernel-tools stage


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAMLT6uD-oACtr6oyynhh-cPM15D_wTz9Xrzn78amXnBq4j-Ddw>