[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 1/2] GitLab CI: avoid calling before_scripts on unintende
From: |
Thomas Huth |
Subject: |
Re: [RFC PATCH 1/2] GitLab CI: avoid calling before_scripts on unintended jobs |
Date: |
Mon, 3 Feb 2020 17:08:58 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 03/02/2020 04.23, Cleber Rosa wrote:
> At this point it seems that all jobs depend on those steps, with
> maybe the EDK2 jobs as exceptions.
>
> The jobs that will be added will not want those scripts to be
> run, so let's move these steps to the appropriate jobs, while
> still trying to avoid repetition.
>
> Signed-off-by: Cleber Rosa <address@hidden>
> ---
> .gitlab-ci.yml | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 228783993e..d2c7d2198e 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -1,11 +1,10 @@
> include:
> - local: '/.gitlab-ci-edk2.yml'
>
> -before_script:
> +build-system1:
> + before_script: &before_scr_apt
> - apt-get update -qq
> - apt-get install -y -qq flex bison libglib2.0-dev libpixman-1-dev
> genisoimage
> -
> -build-system1:
> script:
> - apt-get install -y -qq libgtk-3-dev libvte-dev nettle-dev libcacard-dev
> libusb-dev libvde-dev libspice-protocol-dev libgl1-mesa-dev
> libvdeplug-dev
I'm not very familiar with these anchors yet, but would it maybe be
possible to keep the "template" out of build-system1 ? Something like
they show on https://docs.gitlab.com/ee/ci/yaml/#anchors ?
Anyway, patch is also fine for me in its current shape, so FWIW:
Acked-by: Thomas Huth <address@hidden>
Re: [RFC PATCH 1/2] GitLab CI: avoid calling before_scripts on unintended jobs, Wainer dos Santos Moschetta, 2020/02/03
Re: [RFC PATCH 1/2] GitLab CI: avoid calling before_scripts on unintended jobs,
Thomas Huth <=