[Buildroot] [PATCH 01/10] gitlab-ci: introduce main() in generating script

[Romain Naour]

Le 06/09/2020 à 22:12, Yann E. MORIN a écrit :
> This script is currently very crude, but we're going to extend it, at
> which point it will be nicer to have functions, local variables, et al.
> 
> Introduce a main() in preparation of those future evolutions.
> 
> Signed-off-by: Yann E. MORIN <yann.morin.1998 at free.fr>
> Cc: Romain Naour <romain.naour at gmail.com>
> Cc: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
> Cc: Arnout Vandecappelle <arnout at mind.be>

Reviewed-by: Romain Naour <romain.naour at gmail.com>

Best regards,
Romain


> ---
>  support/scripts/generate-gitlab-ci-yml | 18 +++++++++++-------
>  1 file changed, 11 insertions(+), 7 deletions(-)
> 
> diff --git a/support/scripts/generate-gitlab-ci-yml b/support/scripts/generate-gitlab-ci-yml
> index 5cef1146e2..51f7de21dd 100755
> --- a/support/scripts/generate-gitlab-ci-yml
> +++ b/support/scripts/generate-gitlab-ci-yml
> @@ -2,16 +2,20 @@
>  set -e
>  set -o pipefail
>  
> -input="${1}"
> +main() {
> +    local template="${1}"
>  
> -cat "${input}"
> +    cat "${template}"
>  
> -(
> -    cd configs
> -    LC_ALL=C ls -1 *_defconfig
> -) \
> +    (
> +        cd configs
> +        LC_ALL=C ls -1 *_defconfig
> +    ) \
>      | sed -r -e 's/^(.+)$/\1: { extends: .defconfig }\n\1_check: { extends: .defconfig_check }/'
>  
> -./support/testing/run-tests -l 2>&1 \
> +    ./support/testing/run-tests -l 2>&1 \
>      | sed -r -e '/^test_run \((.*)\).*/!d; s//\1: { extends: .runtime_test }/' \
>      | LC_ALL=C sort
> +}
> +
> +main "${@}"
>