Re: [PATCH v3 09/10] travis: Disable MIPS target

[Leif Lindholm]

On Tue, Jun 04, 2019 at 12:27:33PM +0200, Alexander Graf wrote:
> The MIPS target currently does not compile due to the following error
> message:
> 
>   configure: error: could not force big-endian
> 
> If anyone cares enough about MIPS to fix it up, be my guest and revert
> this commit afterwards. For now, we really need to move forward with
> a fully successful travis run to ensure that we catch regressions early
> on.

Having looked into this, I could possibly stitch something together.

As Alex and I found in parallel, the fault mentioned above is caused
by configure failing to identify the target (ending up with
mips-unknown-elf), and it then falling back to use the host cc for the
tests.

The way to work around that is to fully override the target
specification (--target=mips-linux, --target=mips64-linux). But a
further complication is that mips and mips64 use different
toolchains. And --target=mips*el-linux fails in the original way, so
we'd also need to override the toolchain name.

And then there are the different platforms: yeeloong, fuloong,
loongson, arc, qemu_mips.

So in order to enable CI for mips, we would need to:
- add the mips-linux toolchain from the same location as the others
- split up 64-bit and 32-bit targets into different jobs
- fix overriding toolchain name to support -el flavours with this
  prebuilt toolchain. (Or revamp toolchain support in some other way.)
and the killer
- figure out which cpu/platform combos actually make sense

But until someone steps up to clean up and look after mips, from my
point of view:
Reviewed-by: Leif Lindholm <address@hidden>

> Signed-off-by: Alexander Graf <address@hidden>
> ---
>  .travis.yml | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 5867efea5..b63a992aa 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -112,10 +112,13 @@ matrix:
>        env:
>          - GRUB_TARGETS="ia64-efi"
>          - CROSS_TARGETS="ia64-linux"
> -    - name: "mips"
> -      env:
> -        - GRUB_TARGETS="mips-arc mipsel-arc mipsel-qemu_mips-elf 
> mips-qemu_mips-flash"
> -        - CROSS_TARGETS="mips64-linux"
> +    # MIPS fails with the following error currently. If anyone cares about 
> MIPS
> +    # testing, please reenable it after resolving the problem:
> +    #    configure: error: could not force big-endian
> +    #- name: "mips"
> +    #  env:
> +    #    - GRUB_TARGETS="mips-arc mipsel-arc mipsel-qemu_mips-elf 
> mips-qemu_mips-flash"
> +    #    - CROSS_TARGETS="mips64-linux"
>      - name: "arm"
>        env:
>          - GRUB_TARGETS="arm-coreboot-vexpress arm-efi arm-uboot"
> -- 
> 2.16.4
>