Re: [PATCH] .travis.yml: Add description to each job

[Thomas Huth]

On 29/01/2020 08.27, Philippe Mathieu-Daudé wrote:
> On 1/29/20 8:20 AM, Thomas Huth wrote:
>> On 28/01/2020 14.33, Wainer dos Santos Moschetta wrote:
>>>
>>> On 1/28/20 11:18 AM, Philippe Mathieu-Daudé wrote:
>>>> On Tue, Jan 28, 2020 at 1:55 PM Alex Bennée <address@hidden>
>>>> wrote:
>>>>> Thomas Huth <address@hidden> writes:
>>>>>> On 25/01/2020 19.31, Philippe Mathieu-Daudé wrote:
>>>>>>> The NAME variable can be used to describe nicely a job (see [*]).
>>>>>>> As we currently have 32 jobs, use it. This helps for quickly
>>>>>>> finding a particular job.
>>>>>>>
>>>>>>>     before: https://travis-ci.org/qemu/qemu/builds/639887646
>>>>>>>     after: https://travis-ci.org/philmd/qemu/builds/641795043
>>>>>> Very good idea, correlating a job in the GUI to an entry in the yml
>>>>>> file
>>>>>> was really a pain, so far.
>>>>>>
>>>>>>> [*]
>>>>>>> https://docs.travis-ci.com/user/customizing-the-build/#naming-jobs-within-matrices
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>>>>>>> ---
>>>>>>>    .travis.yml | 101
>>>>>>> ++++++++++++++++++++++++++++++++++------------------
>>>>>>>    1 file changed, 67 insertions(+), 34 deletions(-)
>>>>>>>
>>>>>>> diff --git a/.travis.yml b/.travis.yml
>>>>>>> index 6c1038a0f1..d68e35a2c5 100644
>>>>>>> --- a/.travis.yml
>>>>>>> +++ b/.travis.yml
>>>>>>> @@ -94,24 +94,28 @@ after_script:
>>>>>>>
>>>>>>>    matrix:
>>>>>>>      include:
>>>>>>> -    - env:
>>>>>>> +    - name: "[x86] GCC static (user)"
>>>>>> Could you please drop the [x86] and other architectures from the
>>>>>> names?
>>>>>> Travis already lists the build architecture in the job status
>>>>>> page, so
>>>>>> this information is redundant.
>>>>> Hmm for me the Travis page mis-renders the architecture (on
>>>>> firefox) so
>>>>> I do find the arch in the text fairly handy.
>>>> This might be a font problem, I can't see the architecture on neither
>>>> Firefox nor Chrome:
>>>>
>>>> https://pasteboard.co/IS3O358.png
>>>
>>>
>>> It is the partially hidden column between the job number and the penguin
>>> (or apple if MacOS).
>>>
>>> Funny, I can see the arch on Philippe's dashboard
>>> (https://travis-ci.org/philmd/qemu) but it disappears on my own
>>> (https://travis-ci.org/wainersm/qemu).
>>
>> I've never had problems here, for me the column shows up correctly
>> everywhere. It looks like this:
>>
>>   http://people.redhat.com/~thuth/travis.png
> 
> OK now I understand better your comment :)
> 
>>
>>> Anyway, most of the jobs run on x86_64. So perhaps mark only the non-x86
>>> ones?
>>
>> Sounds like a good compromise to me!
> 
> I'd rather use one style, rather ARCH explicit on all, or not used at all.

Then I'd vote to drop it (if I've got a vote here at all ;-)), hoping
that Travis fixes their HTML for that column, soon...

 Thomas