Re: [PATCH] Make *-netbsd mean ELF rather than a.out for all CPU archs

[John Ericson]

Thanks Nick,

I've asked on the NetBSD "tech-toolchain" list where I fielded these questions before. I've also fixed up the typo on my local branch (https://github.com/Ericson2314/binutils-gdb/tree/netbsd-elf-by-default) so I don't forget. Hope to have a new version to submit soon.

Cheers,

John

On Fri, Jul 16, 2021, at 11:54 AM, Nick Clifton wrote:
> Hi John,
> 
> > As discussed previously, a.out support is now quite deprecated, and in
> > some cases removed, in both Binutils itself and NetBSD, so this legacy
> > default makes little sense.
> 
> One problem with that - the NS32K target.  As far as I can see this target
> only supports aout format NetBSD, not ELF...
> 
> Possibly NS32K support in NetBSD is entirely deprecated.  If so, then we
> can add the ns32k-netbsd target to the obsolete list.  But I would like
> to have you confirm this first.
> 
> 
> > diff --git a/binutils/testsuite/lib/binutils-common.exp b/binutils/testsuite/lib/binutils-common.exp
> > index 59e25df99b0..bfb8911737a 100644
> > --- a/binutils/testsuite/lib/binutils-common.exp
> > +++ b/binutils/testsuite/lib/binutils-common.exp
> > @@ -63,9 +63,7 @@ proc is_elf_format {} {
> >   return 0
> >       }
> >   
> > -    if { ![istarget *-*-netbsdelf*]
> > - && (   [istarget vax-*-netbsd*]
> > -      || [istarget ns32k-*-netbsd*]) } {
> > +    if { [istarget *-*-netbsdaout* } {
> >   return 0
> >       }
> >   
> 
> Typo.  The new line should be:
> 
>    +   if { [istarget *-*-netbsdaout*] } {
> 
> Ie with a closing square parenthesis.
> 
> A patch with the typo corrected and NS32K added to the obsolete list in config.bfd
> is approved, provided that you can confirm that the NS32K is no longer supported.
> 
> Cheers
>    Nick
> 
>