This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Patch for readelf testsuite


"H . J . Lu" <hjl@lucon.org> writes:
> On Fri, Mar 30, 2001 at 10:09:58AM -0800, Chris G. Demetriou wrote:
> > Was this run through the testsuite on any particular architectures?
> 
> Yes/No. Yes, I ran it on Linux/ia32. No, it just skipped Linux/ia32.
> Here is a patch.

Heh.  That's almost funny.  8-)

> -if ![istarget "*-*elf"] then {
> +if { ![istarget i?86-*-sysv4*] \
> +     && ![istarget i?86-*-unixware] \
> +     && ![istarget i?86-*-elf*] \
> +     && ![istarget i?86-*-linux*] \
> +     && ![istarget ia64-*-elf*] \
> +     && ![istarget ia64-*-linux*] \
> +     && ![istarget m68k-*-linux*] \
> +     && ![istarget mips*-*-irix5*] \
> +     && ![istarget powerpc-*-elf*] \
> +     && ![istarget powerpc-*-linux*] \
> +     && ![istarget powerpc-*-sysv4*] \
> +     && ![istarget sparc*-*-elf] \
> +     && ![istarget sparc*-*-solaris2*] \
> +     && ![istarget sparc*-*-linux*] \
> +     && ![istarget arm*-*-linux*] \
> +     && ![istarget alpha*-*-linux*] } {
>      verbose "$READELF is only intended for ELF targets" 2
>      return
>  }

perhaps add '"*-*elf"' to the list of supported targets, and don't
bother with specifically mentioning ELF later?

In particular, I note that you omit mips*-*-elf* which is the target I
had the problem on, i i could have sworn that there were a bunch of
other -elf targets which could reasonably be tested (m68k-elf,
arm-elf, etc.?)


thanks for looking into it...


chris


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]