Re: [PATCH 3/4] travis: Remove mips builds

2020-09-22 Thread Petr Vorel
> On Wed, Sep 09, 2020 at 10:02:20PM +0200, Petr Vorel wrote:
> > Remove mips builds fix configure error:
> > configure: error: could not force big-endian)

> Could you try to fix mips and risc5 builds instead of removing them?
mips fails due using compiler [1] which lacks lacks big-endian support:
"configure: error: could not force big-endian"

Maybe later versions (9.3.0, 10.1.0) will have the support.
The other way (probably slower) would be to add foreign architecture and install
the compiler with apt.

And also IMHO job should stop on configure fail, not continue after that.
I'll probably send a patch fixing this.

> Daniel

[1] 
https://mirrors.kernel.org/pub/tools/crosstool/files/bin/x86_64/8.1.0/x86_64-gcc-8.1.0-nolibc-riscv32-linux.tar.gz

___
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel


Re: [PATCH 4/4] travis: Remove riscv32 build

2020-09-22 Thread Petr Vorel
Hi Daniel,

> To fix travis error:
> grub-mkimage: error: target 1036 not reachable from pc=ba.

https://travis-ci.org/github/pevik/grub/jobs/729143844
$ for target in $GRUB_TARGETS; do grub-mkimage -c grub.cfg -p / -O $target -o 
grub-$target echo reboot normal; done
grub-mkimage: error: target 1036 not reachable from pc=ba.

I found error messages in util/grub-mkimagexx.c, but I have no idea what is
missing in the test which causes grub-mkimage to fail. Any idea?
I doubt it's a real bug.

Kind regards,
Petr

___
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel


Re: [PATCH 2/4] travis: Fix sparc64 build

2020-09-22 Thread Petr Vorel
Hi Daniel,

...
> > ># Build all selected GRUB targets.
> > >- for target in $GRUB_TARGETS; do
> > > -  plat=${target#*-};
> > > -  arch=${target%-*};
> > > +  arch=$(echo $target | cut -d- -f1);
> > > +  plat=$(echo $target | cut -d- -f2);

> > I think it should be: plat=$(echo $target | cut -d- -f2-);
> > Note dash after "2"...
> No, this way it breaks sparc64 [1] (and does not help to fix mips [2]
> nor risc32 either).

> It looks like it's this error:
> configure: error: platform "ieee1275-raw" is not supported for target CPU 
> "sparc64"

Yes, the correct build without - at the end gets plat == "ieee1275", which is
correct.
https://travis-ci.org/github/pevik/grub/jobs/729143839

Kind regards,
Petr

> but I need to debug it more to see what's wrong.
Comment mips and risc32 under their patches.


Kind regards,
Petr

> > Though I would prefer this:
> >   arch=$(echo "$target" | cut -d - -f 1);
> >   plat=$(echo "$target" | cut -d - -f 2-);

> > Daniel

> Kind regards,
> Petr

> [1] https://travis-ci.org/github/pevik/grub/jobs/728077121
> [2] https://travis-ci.org/github/pevik/grub/jobs/728077123
> [3] https://travis-ci.org/github/pevik/grub/jobs/728077126

___
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel