* [PATCH 0/4] Fix Travis CI
@ 2020-09-09 20:02 Petr Vorel
2020-09-09 20:02 ` [PATCH 1/4] travis: Run bootstrap to fix build Petr Vorel
` (3 more replies)
0 siblings, 4 replies; 17+ messages in thread
From: Petr Vorel @ 2020-09-09 20:02 UTC (permalink / raw)
To: grub-devel; +Cc: Petr Vorel
Hi,
although grub does not use Travis CI (or at least not publicly) some
developers might welcome fixed .travis.yml for testing their patches.
I haven't figured out how to fix riscv32 and mips.
Kind regards,
Petr
Petr Vorel (4):
travis: Run bootstrap to fix build
travis: Fix sparc64 build
travis: Remove mips builds
travis: Remove riscv32 build
.travis.yml | 17 +++++------------
1 file changed, 5 insertions(+), 12 deletions(-)
--
2.27.0.rc0
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH 1/4] travis: Run bootstrap to fix build
2020-09-09 20:02 [PATCH 0/4] Fix Travis CI Petr Vorel
@ 2020-09-09 20:02 ` Petr Vorel
2020-09-17 13:41 ` Daniel Kiper
2020-09-09 20:02 ` [PATCH 2/4] travis: Fix sparc64 build Petr Vorel
` (2 subsequent siblings)
3 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2020-09-09 20:02 UTC (permalink / raw)
To: grub-devel; +Cc: Petr Vorel
autogen.sh isn't enough:
$ ./autogen.sh
Gnulib not yet bootstrapped; run ./bootstrap instead.
The command "./autogen.sh" exited with 1.
Using bootstrap requires to install autopoint package.
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
.travis.yml | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index 81de20fa3..4bd05a30a 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -12,6 +12,7 @@ language: c
addons:
apt:
packages:
+ - autopoint
- libsdl1.2-dev
- lzop
- ovmf
@@ -35,7 +36,7 @@ before_script:
script:
# Comments must be outside the command strings below, or the Travis parser
# will get confused.
- - ./autogen.sh
+ - ./bootstrap
# Build all selected GRUB targets.
- for target in $GRUB_TARGETS; do
--
2.27.0.rc0
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 2/4] travis: Fix sparc64 build
2020-09-09 20:02 [PATCH 0/4] Fix Travis CI Petr Vorel
2020-09-09 20:02 ` [PATCH 1/4] travis: Run bootstrap to fix build Petr Vorel
@ 2020-09-09 20:02 ` Petr Vorel
2020-09-17 13:48 ` Daniel Kiper
2020-09-09 20:02 ` [PATCH 3/4] travis: Remove mips builds Petr Vorel
2020-09-09 20:02 ` [PATCH 4/4] travis: Remove riscv32 build Petr Vorel
3 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2020-09-09 20:02 UTC (permalink / raw)
To: grub-devel; +Cc: Petr Vorel
Instead non-existing sparc64-ieee1275 format use all 3 sparc64 formats:
sparc64-ieee1275-{aout,cdcore,raw}.
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
.travis.yml | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 4bd05a30a..d019a0172 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -40,8 +40,8 @@ script:
# 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);
[ "$arch" = "arm64" ] && arch=aarch64-linux;
[ "$arch" = "arm" ] && arch=arm-linux-gnueabi;
[ "$arch" = "ia64" ] && arch=ia64-linux;
@@ -81,7 +81,7 @@ matrix:
- CROSS_TARGETS="powerpc64-linux"
- name: "sparc64"
env:
- - GRUB_TARGETS="sparc64-ieee1275"
+ - GRUB_TARGETS="sparc64-ieee1275-aout sparc64-ieee1275-cdcore sparc64-ieee1275-raw"
- CROSS_TARGETS="sparc64-linux"
- name: "ia64"
env:
--
2.27.0.rc0
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [PATCH 2/4] travis: Fix sparc64 build
2020-09-09 20:02 ` [PATCH 2/4] travis: Fix sparc64 build Petr Vorel
@ 2020-09-17 13:48 ` Daniel Kiper
2020-09-21 22:19 ` Petr Vorel
0 siblings, 1 reply; 17+ messages in thread
From: Daniel Kiper @ 2020-09-17 13:48 UTC (permalink / raw)
To: Petr Vorel; +Cc: grub-devel
On Wed, Sep 09, 2020 at 10:02:19PM +0200, Petr Vorel wrote:
> Instead non-existing sparc64-ieee1275 format use all 3 sparc64 formats:
> sparc64-ieee1275-{aout,cdcore,raw}.
>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> .travis.yml | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index 4bd05a30a..d019a0172 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -40,8 +40,8 @@ script:
>
> # 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"...
Though I would prefer this:
arch=$(echo "$target" | cut -d - -f 1);
plat=$(echo "$target" | cut -d - -f 2-);
Daniel
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 2/4] travis: Fix sparc64 build
2020-09-17 13:48 ` Daniel Kiper
@ 2020-09-21 22:19 ` Petr Vorel
2020-09-22 9:05 ` Petr Vorel
0 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2020-09-21 22:19 UTC (permalink / raw)
To: Daniel Kiper; +Cc: grub-devel
Hi Daniel,
thanks for merging first commit.
> On Wed, Sep 09, 2020 at 10:02:19PM +0200, Petr Vorel wrote:
> > Instead non-existing sparc64-ieee1275 format use all 3 sparc64 formats:
> > sparc64-ieee1275-{aout,cdcore,raw}.
> > Signed-off-by: Petr Vorel <pvorel@suse.cz>
> > ---
> > .travis.yml | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> > diff --git a/.travis.yml b/.travis.yml
> > index 4bd05a30a..d019a0172 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -40,8 +40,8 @@ script:
> > # 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"
but I need to debug it more to see what's wrong.
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
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/4] travis: Remove mips builds
2020-09-09 20:02 [PATCH 0/4] Fix Travis CI Petr Vorel
2020-09-09 20:02 ` [PATCH 1/4] travis: Run bootstrap to fix build Petr Vorel
2020-09-09 20:02 ` [PATCH 2/4] travis: Fix sparc64 build Petr Vorel
@ 2020-09-09 20:02 ` Petr Vorel
2020-09-17 13:50 ` Daniel Kiper
2020-09-09 20:02 ` [PATCH 4/4] travis: Remove riscv32 build Petr Vorel
3 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2020-09-09 20:02 UTC (permalink / raw)
To: grub-devel; +Cc: Petr Vorel
Remove mips builds fix configure error:
configure: error: could not force big-endian)
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
.travis.yml | 4 ----
1 file changed, 4 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index d019a0172..ca2a1fa5a 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -87,10 +87,6 @@ matrix:
env:
- GRUB_TARGETS="ia64-efi"
- CROSS_TARGETS="ia64-linux"
- - name: "mips"
- env:
- - GRUB_TARGETS="mips-arc mipsel-arc mipsel-qemu_mips mips-qemu_mips"
- - CROSS_TARGETS="mips64-linux"
- name: "arm"
env:
- GRUB_TARGETS="arm-coreboot arm-efi arm-uboot"
--
2.27.0.rc0
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [PATCH 3/4] travis: Remove mips builds
2020-09-09 20:02 ` [PATCH 3/4] travis: Remove mips builds Petr Vorel
@ 2020-09-17 13:50 ` Daniel Kiper
2020-09-17 14:17 ` Petr Vorel
2020-09-22 9:14 ` Petr Vorel
0 siblings, 2 replies; 17+ messages in thread
From: Daniel Kiper @ 2020-09-17 13:50 UTC (permalink / raw)
To: Petr Vorel; +Cc: grub-devel
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?
Daniel
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/4] travis: Remove mips builds
2020-09-17 13:50 ` Daniel Kiper
@ 2020-09-17 14:17 ` Petr Vorel
2020-09-22 9:14 ` Petr Vorel
1 sibling, 0 replies; 17+ messages in thread
From: Petr Vorel @ 2020-09-17 14:17 UTC (permalink / raw)
To: Daniel Kiper; +Cc: grub-devel
Hi Daniel,
> 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?
Right, I'll try and send v2.
Kind regards,
Petr
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/4] travis: Remove mips builds
2020-09-17 13:50 ` Daniel Kiper
2020-09-17 14:17 ` Petr Vorel
@ 2020-09-22 9:14 ` Petr Vorel
2020-10-27 19:53 ` Daniel Kiper
1 sibling, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2020-09-22 9:14 UTC (permalink / raw)
To: Daniel Kiper; +Cc: grub-devel
> 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
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/4] travis: Remove mips builds
2020-09-22 9:14 ` Petr Vorel
@ 2020-10-27 19:53 ` Daniel Kiper
0 siblings, 0 replies; 17+ messages in thread
From: Daniel Kiper @ 2020-10-27 19:53 UTC (permalink / raw)
To: Petr Vorel; +Cc: grub-devel
Hi Petr,
First of all, sorry for late reply...
On Tue, Sep 22, 2020 at 11:14:48AM +0200, Petr Vorel wrote:
> > 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"
Well, it seems to me that it tries to use RISC-V compiler to compile
MIPS target... The MIPS version downloaded from the same site builds
big-endian without any issue.
> Maybe later versions (9.3.0, 10.1.0) will have the support.
By the way, could you bump all compilers to the latest version available
at https://mirrors.kernel.org/pub/tools/crosstool/ ?
> 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
Daniel
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 4/4] travis: Remove riscv32 build
2020-09-09 20:02 [PATCH 0/4] Fix Travis CI Petr Vorel
` (2 preceding siblings ...)
2020-09-09 20:02 ` [PATCH 3/4] travis: Remove mips builds Petr Vorel
@ 2020-09-09 20:02 ` Petr Vorel
2020-09-22 9:20 ` Petr Vorel
3 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2020-09-09 20:02 UTC (permalink / raw)
To: grub-devel; +Cc: Petr Vorel
To fix travis error:
grub-mkimage: error: target 1036 not reachable from pc=ba.
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
.travis.yml | 4 ----
1 file changed, 4 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index ca2a1fa5a..8628c526f 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -95,10 +95,6 @@ matrix:
env:
- GRUB_TARGETS="arm64-efi"
- CROSS_TARGETS="aarch64-linux"
- - name: "riscv32"
- env:
- - GRUB_TARGETS="riscv32-efi"
- - CROSS_TARGETS="riscv32-linux"
- name: "riscv64"
env:
- GRUB_TARGETS="riscv64-efi"
--
2.27.0.rc0
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [PATCH 4/4] travis: Remove riscv32 build
2020-09-09 20:02 ` [PATCH 4/4] travis: Remove riscv32 build Petr Vorel
@ 2020-09-22 9:20 ` Petr Vorel
2020-10-27 20:06 ` Daniel Kiper
0 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2020-09-22 9:20 UTC (permalink / raw)
To: Daniel Kiper; +Cc: grub-devel
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
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 4/4] travis: Remove riscv32 build
2020-09-22 9:20 ` Petr Vorel
@ 2020-10-27 20:06 ` Daniel Kiper
2020-12-18 4:39 ` Glenn Washburn
0 siblings, 1 reply; 17+ messages in thread
From: Daniel Kiper @ 2020-10-27 20:06 UTC (permalink / raw)
To: Petr Vorel, agraf; +Cc: grub-devel
Adding Alex...
On Tue, Sep 22, 2020 at 11:20:12AM +0200, Petr Vorel wrote:
> 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
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 4/4] travis: Remove riscv32 build
2020-10-27 20:06 ` Daniel Kiper
@ 2020-12-18 4:39 ` Glenn Washburn
2020-12-18 7:55 ` Petr Vorel
0 siblings, 1 reply; 17+ messages in thread
From: Glenn Washburn @ 2020-12-18 4:39 UTC (permalink / raw)
To: Daniel Kiper; +Cc: The development of GNU GRUB, Petr Vorel, agraf
On Tue, 27 Oct 2020 21:06:45 +0100
Daniel Kiper <dkiper@net-space.pl> wrote:
> Adding Alex...
>
> On Tue, Sep 22, 2020 at 11:20:12AM +0200, Petr Vorel wrote:
> > 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
Did anyone have a chance to look at this? I'm also seeing this with
this target when trying to run the tests, which are failing.
Glenn
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 4/4] travis: Remove riscv32 build
2020-12-18 4:39 ` Glenn Washburn
@ 2020-12-18 7:55 ` Petr Vorel
0 siblings, 0 replies; 17+ messages in thread
From: Petr Vorel @ 2020-12-18 7:55 UTC (permalink / raw)
To: Glenn Washburn; +Cc: Daniel Kiper, The development of GNU GRUB, agraf
Hi Glenn,
> On Tue, 27 Oct 2020 21:06:45 +0100
> Daniel Kiper <dkiper@net-space.pl> wrote:
> > Adding Alex...
> > On Tue, Sep 22, 2020 at 11:20:12AM +0200, Petr Vorel wrote:
> > > 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
> Did anyone have a chance to look at this? I'm also seeing this with
> this target when trying to run the tests, which are failing.
I haven't had a chance yet. Also Travis CI got pretty slow last few weeks, which
does not help.
Kind regards,
Petr
> Glenn
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2020-12-18 7:55 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-09 20:02 [PATCH 0/4] Fix Travis CI Petr Vorel
2020-09-09 20:02 ` [PATCH 1/4] travis: Run bootstrap to fix build Petr Vorel
2020-09-17 13:41 ` Daniel Kiper
2020-09-09 20:02 ` [PATCH 2/4] travis: Fix sparc64 build Petr Vorel
2020-09-17 13:48 ` Daniel Kiper
2020-09-21 22:19 ` Petr Vorel
2020-09-22 9:05 ` Petr Vorel
2020-09-09 20:02 ` [PATCH 3/4] travis: Remove mips builds Petr Vorel
2020-09-17 13:50 ` Daniel Kiper
2020-09-17 14:17 ` Petr Vorel
2020-09-22 9:14 ` Petr Vorel
2020-10-27 19:53 ` Daniel Kiper
2020-09-09 20:02 ` [PATCH 4/4] travis: Remove riscv32 build Petr Vorel
2020-09-22 9:20 ` Petr Vorel
2020-10-27 20:06 ` Daniel Kiper
2020-12-18 4:39 ` Glenn Washburn
2020-12-18 7:55 ` Petr Vorel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.