[v1] ci: Unify the GitLab CI jobs with local executions && adopt lcitool container executions

[libvirt PATCH 09/33] ci: build.sh: Add a wrapper function over the 'codestyle' job

Posted by Erik Skultety 2 years, 5 months ago

This helper is a shell function transcript of its original GitLab CI
counterpart.

Signed-off-by: Erik Skultety <eskultet@redhat.com>
---
 ci/build.sh | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/ci/build.sh b/ci/build.sh
index ab471f2741..b2891a0c0f 100644
--- a/ci/build.sh
+++ b/ci/build.sh
@@ -73,3 +73,12 @@ run_test() {
     run_meson_setup
     run_cmd "$CMD"
 }
+
+run_codestyle() {
+    BUILD_ARGS="libvirt-pot-dep"
+    TEST_ARGS="--suite syntax-check --no-rebuild --print-errorlogs"
+
+    run_meson_setup
+    run_build
+    run_test
+}
-- 
2.41.0

Re: [libvirt PATCH 09/33] ci: build.sh: Add a wrapper function over the 'codestyle' job

Posted by Daniel P. Berrangé 2 years, 5 months ago

On Fri, Aug 25, 2023 at 07:55:17PM +0200, Erik Skultety wrote:
> This helper is a shell function transcript of its original GitLab CI
> counterpart.
> 
> Signed-off-by: Erik Skultety <eskultet@redhat.com>
> ---
>  ci/build.sh | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/ci/build.sh b/ci/build.sh
> index ab471f2741..b2891a0c0f 100644
> --- a/ci/build.sh
> +++ b/ci/build.sh
> @@ -73,3 +73,12 @@ run_test() {
>      run_meson_setup
>      run_cmd "$CMD"
>  }
> +
> +run_codestyle() {
> +    BUILD_ARGS="libvirt-pot-dep"
> +    TEST_ARGS="--suite syntax-check --no-rebuild --print-errorlogs"
> +
> +    run_meson_setup
> +    run_build

run_build already implies run_meson_setup so we can drop it in this
case.

> +    run_test
> +}


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|