Re: [PATCH] testsuite: Also exclude `*-*-kfreebsd*-gnu' from STB_GNU_UNIQUE tests
Jan Beulich <[email protected]> Tue, 21 Jul 2026 15:58:10 +0200
| Newsgroups | gmane.comp.gnu.binutils |
|---|---|
| Message-ID | <[email protected]> |
On 21.07.2026 15:42, Maciej W. Rozycki wrote:
> On Tue, 21 Jul 2026, Jan Beulich wrote:
>
>>> --- binutils-gdb.orig/binutils/testsuite/lib/binutils-common.exp
>>> +++ binutils-gdb/binutils/testsuite/lib/binutils-common.exp
>>> @@ -308,7 +308,7 @@ proc is_elf_unused_section_symbols {} {
>>> # This require ELFOSABI_GNU, and `bfd_elf_final_link'.
>>> #
>>> proc supports_gnu_unique {} {
>>> - if { [istarget *-*-freebsd*] } {
>>> + if { [istarget *-*-freebsd*] || [istarget *-*-kfreebsd*-gnu] } {
>>
>> Is the -gnu at the end really needed here? If yes, it's okay as-is, but
>> preferably it would be put in without that tail.
>
> See bfd/config.bfd for how the targets are referred there; we obviously
> need to stay consistent, because `foo-kfreebsd-bar' will choose different
> BFD vectors (whether it makes sense or not, but that's not the matter of
> this fix).
Afaict `foo-kfreebsd-bar' will simply not match anything, hitting the
"BFD does not support target" case. Therefore I don't see it as wrong to
have a wider match here than what bfd/config.bfd has. Yet no, I'm not
going to insist.
Jan