grub-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] Add travis-ci config file


From: Alexander Graf
Subject: Re: [PATCH] Add travis-ci config file
Date: Thu, 14 Feb 2019 14:07:25 +0100
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:60.0) Gecko/20100101 Thunderbird/60.5.0


On 13.02.19 13:04, Paul Menzel wrote:
> 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?

Oh, definitely!

  https://travis-ci.org/agraf/grub/builds/492262122

> 
> 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 }?

Well, the whole comment is stale. Thanks for catching it :)


Alex

> 
>> +  - 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
> 



reply via email to

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