Re: [PATCH] Add travis-ci config file

[Paul Menzel]

Dear Alexander,


On 02/12/19 18:00, 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 :).

Thank you very much for this patch. Could you please post the URL where the
test results are available for the run you did?

There is one small typo further down.

> Signed-off-by: Alexander Graf <address@hidden>
> ---
>  .travis.yml | 118 
> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 118 insertions(+)
>  create mode 100644 .travis.yml
> 
> diff --git a/.travis.yml b/.travis.yml
> new file mode 100644
> index 000000000..05608fc21
> --- /dev/null
> +++ b/.travis.yml
> @@ -0,0 +1,118 @@

[…]

> +before_script:
> +  # install toolchains based on TOOLCHAIN} variable

Remove the bracket }?

> +  - mkdir /tmp/cross
> +  # results in 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 -O - 
> https://mirrors.edge.kernel.org/pub/tools/crosstool/files/bin/x86_64/8.1.0/x86_64-gcc-8.1.0-nolibc-$i.tar.xz
>  | xzcat | tar x );
> +    done

[…]


Thanks,

Paul

smime.p7s
Description: S/MIME Cryptographic Signature