grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v3 10/10] travis: Disable IA64 target


From: Leif Lindholm
Subject: Re: [PATCH v3 10/10] travis: Disable IA64 target
Date: Tue, 4 Jun 2019 14:34:09 +0100
User-agent: NeoMutt/20170113 (1.7.2)

On Tue, Jun 04, 2019 at 12:27:34PM +0200, Alexander Graf wrote:
> The IA64 target currently does not compile due to the following error
> message:
> 
>   grub-mkimage: error: undefined symbol grub_arch_sync_dma_caches.
>
> If anyone cares enough about IA64 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.

I am reasonably convinced this was simply neglected when d8901e3ba1
("cache: Fix compilation for ppc, sparc and arm64") was pushed.
(And I don't believe ia64 has been buildable since.)

Adding (__ia64__) to that conditional makes the mkimage step work.
Unless someone convinces me differently, I'll submit that as a patch.

/
    Leif

> 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 b63a992aa..5367ca15b 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -108,10 +108,13 @@ matrix:
>        env:
>          - GRUB_TARGETS="sparc64-ieee1275-aout"
>          - CROSS_TARGETS="sparc64-linux"
> -    - name: "ia64"
> -      env:
> -        - GRUB_TARGETS="ia64-efi"
> -        - CROSS_TARGETS="ia64-linux"
> +    # IA fails with the following error currently. If anyone cares about IA64
> +    # testing, please reenable it after resolving the problem:
> +    #    grub-mkimage: error: undefined symbol grub_arch_sync_dma_caches.
> +    #- name: "ia64"
> +    #  env:
> +    #    - GRUB_TARGETS="ia64-efi"
> +    #    - CROSS_TARGETS="ia64-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
> -- 
> 2.16.4
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]