| Message ID | 20190222081746.13035-1-agraf@suse.de |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v4] Add travis-ci config file | expand |
On Fri, Feb 22, 2019 at 09:17:46AM +0100, Alexander Graf wrote: > There is a really convenient service for open source project from Travis > CI: They allow for free CI testing using their infrastructure. > > Grub has had issues with broken builds for various targets for a long time > already. The main reason is a lack of CI to just do smoke tests on whether > all targets still at least compile. > > This patch adds a travis config file which builds (almost) all currently > available targets. > > On top of that, this travis config also runs a small execution test on the > x86_64-efi target. > > All of this config file can easily be extended further on. It probably makes > sense to do something similar to the u-boot test infrastructure that > communicates with the payload properly. Going forward, we also will want to > do more qemu runtime checks for other targets. > > Currently, with this config alone, I already see about half of the available > targets as broken. So it's definitely desperately needed :). > > Signed-off-by: Alexander Graf <agraf@suse.de> In general Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com> But... > --- > > v1 -> v2: > > - Fix comment about toolchain variable > > v2 -> v3: > > - s/grub/GRUB/ > - remove unneeded packages > - comment fixes > - determine number of jobs dynamically > - sort targets alphabetically > > v3 -> v4: > > - reduce package list futher > - sort package list alphabetically > --- > .travis.yml | 109 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 109 insertions(+) > create mode 100644 .travis.yml > > diff --git a/.travis.yml b/.travis.yml > new file mode 100644 > index 000000000..a66143cba > --- /dev/null > +++ b/.travis.yml > @@ -0,0 +1,109 @@ > +# SPDX-License-Identifier: GPL-3.0+ > +# Originally Copyright Roger Meier <r.meier@siemens.com> > +# Adapted for GRUB by Alexander Graf <agraf@suse.de> > + > +# build GRUB on Travis CI - https://travis-ci.org/ > + > +dist: xenial > + > +language: c > + > +addons: > + apt: > + packages: > + - build-essential > + - device-tree-compiler Out of curiosity, are build-essential and device-tree-compiler needed to test GRUB or just leftovers by mistake? Daniel _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel
On 22.02.19 14:22, Daniel Kiper wrote: > On Fri, Feb 22, 2019 at 09:17:46AM +0100, Alexander Graf wrote: >> There is a really convenient service for open source project from Travis >> CI: They allow for free CI testing using their infrastructure. >> >> Grub has had issues with broken builds for various targets for a long time >> already. The main reason is a lack of CI to just do smoke tests on whether >> all targets still at least compile. >> >> This patch adds a travis config file which builds (almost) all currently >> available targets. >> >> On top of that, this travis config also runs a small execution test on the >> x86_64-efi target. >> >> All of this config file can easily be extended further on. It probably makes >> sense to do something similar to the u-boot test infrastructure that >> communicates with the payload properly. Going forward, we also will want to >> do more qemu runtime checks for other targets. >> >> Currently, with this config alone, I already see about half of the available >> targets as broken. So it's definitely desperately needed :). >> >> Signed-off-by: Alexander Graf <agraf@suse.de> > > In general Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com> > > But... > >> --- >> >> v1 -> v2: >> >> - Fix comment about toolchain variable >> >> v2 -> v3: >> >> - s/grub/GRUB/ >> - remove unneeded packages >> - comment fixes >> - determine number of jobs dynamically >> - sort targets alphabetically >> >> v3 -> v4: >> >> - reduce package list futher >> - sort package list alphabetically >> --- >> .travis.yml | 109 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 109 insertions(+) >> create mode 100644 .travis.yml >> >> diff --git a/.travis.yml b/.travis.yml >> new file mode 100644 >> index 000000000..a66143cba >> --- /dev/null >> +++ b/.travis.yml >> @@ -0,0 +1,109 @@ >> +# SPDX-License-Identifier: GPL-3.0+ >> +# Originally Copyright Roger Meier <r.meier@siemens.com> >> +# Adapted for GRUB by Alexander Graf <agraf@suse.de> >> + >> +# build GRUB on Travis CI - https://travis-ci.org/ >> + >> +dist: xenial >> + >> +language: c >> + >> +addons: >> + apt: >> + packages: >> + - build-essential >> + - device-tree-compiler > > Out of curiosity, are build-essential and device-tree-compiler needed to > test GRUB or just leftovers by mistake? I think build-essential is a nop (already installed) and device-tree-compiler an oversight. Off to v5 ... Alex _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel
diff --git a/.travis.yml b/.travis.yml new file mode 100644 index 000000000..a66143cba --- /dev/null +++ b/.travis.yml @@ -0,0 +1,109 @@ +# SPDX-License-Identifier: GPL-3.0+ +# Originally Copyright Roger Meier <r.meier@siemens.com> +# Adapted for GRUB by Alexander Graf <agraf@suse.de> + +# build GRUB on Travis CI - https://travis-ci.org/ + +dist: xenial + +language: c + +addons: + apt: + packages: + - build-essential + - device-tree-compiler + - libsdl1.2-dev + - lzop + - ovmf + - python + - qemu-system + - unifont + +env: + global: + # Include all cross toolchain paths, so we can just call them later down + - PATH=/tmp/qemu-install/bin:/tmp/grub/bin:/usr/bin:/bin:/tmp/cross/gcc-8.1.0-nolibc/aarch64-linux/bin:/tmp/cross/gcc-8.1.0-nolibc/arm-linux-gnueabi/bin:/tmp/cross/gcc-8.1.0-nolibc/ia64-linux/bin:/tmp/cross/gcc-8.1.0-nolibc/mips64-linux/bin:/tmp/cross/gcc-8.1.0-nolibc/powerpc64-linux/bin:/tmp/cross/gcc-8.1.0-nolibc/riscv32-linux/bin:/tmp/cross/gcc-8.1.0-nolibc/riscv64-linux/bin:/tmp/cross/gcc-8.1.0-nolibc/sparc64-linux/bin + +before_script: + # Install necessary toolchains based on $CROSS_TARGETS variable + - mkdir /tmp/cross + # These give us binaries like /tmp/cross/gcc-8.1.0-nolibc/ia64-linux/bin/ia64-linux-gcc + - for i in $CROSS_TARGETS; do + ( cd /tmp/cross; wget -t 3 -O - https://mirrors.kernel.org/pub/tools/crosstool/files/bin/x86_64/8.1.0/x86_64-gcc-8.1.0-nolibc-$i.tar.xz | tar xJ ); + done + +script: + # Comments must be outside the command strings below, or the Travis parser + # will get confused. + - ./autogen.sh + + # Build all selected GRUB targets: + - for target in $GRUB_TARGETS; do + plat=${target#*-}; + arch=${target%-*}; + [ "$arch" = "arm64" ] && arch=aarch64-linux; + [ "$arch" = "arm" ] && arch=arm-linux-gnueabi; + [ "$arch" = "ia64" ] && arch=ia64-linux; + [ "$arch" = "mipsel" ] && arch=mips64-linux; + [ "$arch" = "powerpc" ] && arch=powerpc64-linux; + [ "$arch" = "riscv32" ] && arch=riscv32-linux; + [ "$arch" = "riscv64" ] && arch=riscv64-linux; + [ "$arch" = "sparc64" ] && arch=sparc64-linux; + echo "Building $target"; + mkdir obj-$target; + JOBS=`getconf _NPROCESSORS_ONLN 2> /dev/null || echo 1`; + [ "$JOBS" == 1 ] || JOBS=$(($JOBS + 1)); + ( cd obj-$target && ../configure --target=$arch --with-platform=$plat --prefix=/tmp/grub && make -j$JOBS && make -j$JOBS install ) &> log || ( cat log; false ); + done + + # Our test canary + - echo -e "insmod echo\\ninsmod reboot\\necho hello world\\nreboot" > grub.cfg + + # Assemble images and possibly run them + - for target in $GRUB_TARGETS; do grub-mkimage -c grub.cfg -p / -O $target -o grub-$target echo reboot normal; done + + # Run images we know how to run + - if [[ "$GRUB_TARGETS" == *"x86_64-efi"* ]]; then qemu-system-x86_64 -bios /usr/share/ovmf/OVMF.fd -m 512 -no-reboot -nographic -net nic -net user,tftp=.,bootfile=grub-x86_64-efi | tee grub.log && grep "hello world" grub.log; fi + +matrix: + include: + # each env setting here is a dedicated build + - name: "x86_64" + env: + - GRUB_TARGETS="x86_64-efi x86_64-xen" + - name: "i386" + env: + - GRUB_TARGETS="i386-coreboot i386-efi i386-ieee1275 i386-multiboot i386-pc i386-qemu i386-xen i386-xen_pvh" + - name: "powerpc" + env: + - GRUB_TARGETS="powerpc-ieee1275" + - CROSS_TARGETS=powerpc64-linux" + - name: "sparc64" + env: + - GRUB_TARGETS="sparc64-ieee1275" + - CROSS_TARGETS=sparc64-linux" + - name: "ia64" + 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" + - CROSS_TARGETS="arm-linux-gnueabi" + - name: "arm64" + 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" + - CROSS_TARGETS=riscv64-linux"
There is a really convenient service for open source project from Travis CI: They allow for free CI testing using their infrastructure. Grub has had issues with broken builds for various targets for a long time already. The main reason is a lack of CI to just do smoke tests on whether all targets still at least compile. This patch adds a travis config file which builds (almost) all currently available targets. On top of that, this travis config also runs a small execution test on the x86_64-efi target. All of this config file can easily be extended further on. It probably makes sense to do something similar to the u-boot test infrastructure that communicates with the payload properly. Going forward, we also will want to do more qemu runtime checks for other targets. Currently, with this config alone, I already see about half of the available targets as broken. So it's definitely desperately needed :). Signed-off-by: Alexander Graf <agraf@suse.de> --- v1 -> v2: - Fix comment about toolchain variable v2 -> v3: - s/grub/GRUB/ - remove unneeded packages - comment fixes - determine number of jobs dynamically - sort targets alphabetically v3 -> v4: - reduce package list futher - sort package list alphabetically --- .travis.yml | 109 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 109 insertions(+) create mode 100644 .travis.yml -- 2.16.4 _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel