[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 07/11] gitlab-ci: Extract common job definition as 'cross_
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v3 07/11] gitlab-ci: Extract common job definition as 'cross_common_job' |
Date: |
Tue, 10 Nov 2020 11:23:58 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 |
On 11/9/20 10:52 AM, Thomas Huth wrote:
> On 08/11/2020 23.19, Philippe Mathieu-Daudé wrote:
>> Extract the common definitions shared by '.cross_system_build_job'
>> and '.cross_user_build_job' to '.cross_common_job'.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>> .gitlab-ci.d/crossbuilds.yml | 9 +++++----
>> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> 5 insertions, 4 deletions ... is it really that much better?
It becomes useful later we we add a lot into cross_common_job,
rather than duplicating in cross_[system/user]_build_job.
I understand this commit description lacks that information.
Thanks.
[PATCH v3 07/11] gitlab-ci: Extract common job definition as 'cross_common_job', Philippe Mathieu-Daudé, 2020/11/08
[PATCH v3 08/11] gitlab-ci: Extract common job definition as 'native_common_job', Philippe Mathieu-Daudé, 2020/11/08
[PATCH v3 09/11] gitlab-ci: Add rules to select cross-jobs to build, Philippe Mathieu-Daudé, 2020/11/08
[PATCH v3 10/11] gitlab-ci: Add rules to select building/testing native jobs, Philippe Mathieu-Daudé, 2020/11/08
[PATCH v3 11/11] gitlab-ci: Move artifacts expiry rule to common 'native_build_job', Philippe Mathieu-Daudé, 2020/11/08