diff mbox series

[PATCH-for-5.2,v2,3/4] gitlab-ci: Add a job to cover the --without-default-devices config

Message ID 20201104084327.3010593-4-philmd@redhat.com
State Superseded
Headers show
Series ci: Move --without-default-devices job from Travis to GitLab | expand

Commit Message

Philippe Mathieu-Daudé Nov. 4, 2020, 8:43 a.m. UTC
We test './configure --without-default-devices' since commit
20885b5b169 (".travis.yml: test that no-default-device builds
do not regress") in Travis-CI.

Since having a single CI to look at is easier, and GitLab-CI
is the preferred one, add the equivalent job there.

As smoke test, run the qtests on the AVR target. Since the
boards are simple SoC, there is not issue with unavailable
default devices there.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 .gitlab-ci.yml | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Thomas Huth Nov. 4, 2020, 9:17 a.m. UTC | #1
On 04/11/2020 09.43, Philippe Mathieu-Daudé wrote:
> We test './configure --without-default-devices' since commit
> 20885b5b169 (".travis.yml: test that no-default-device builds
> do not regress") in Travis-CI.
> 
> Since having a single CI to look at is easier, and GitLab-CI
> is the preferred one, add the equivalent job there.
> 
> As smoke test, run the qtests on the AVR target. Since the
> boards are simple SoC, there is not issue with unavailable
> default devices there.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  .gitlab-ci.yml | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 3b15ae5c302..321cca2c216 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -262,6 +262,13 @@ build-user-plugins:
>      MAKE_CHECK_ARGS: check-tcg
>    timeout: 1h 30m
>  
> +build-system-ubuntu-without-default-devices:
> +  <<: *native_build_job_definition
> +  variables:
> +    IMAGE: ubuntu2004
> +    CONFIGURE_ARGS: --without-default-devices --disable-user --disable-tools --disable-docs
> +    MAKE_CHECK_ARGS: check-qtest-avr

As mentioned in my other mail, we can also use -m68k if you prefer that
instead of -avr.

>  build-clang:
>    <<: *native_build_job_definition
>    variables:
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>
Philippe Mathieu-Daudé Nov. 4, 2020, 11:46 a.m. UTC | #2
On 11/4/20 10:17 AM, Thomas Huth wrote:
> On 04/11/2020 09.43, Philippe Mathieu-Daudé wrote:

>> We test './configure --without-default-devices' since commit

>> 20885b5b169 (".travis.yml: test that no-default-device builds

>> do not regress") in Travis-CI.

>>

>> Since having a single CI to look at is easier, and GitLab-CI

>> is the preferred one, add the equivalent job there.

>>

>> As smoke test, run the qtests on the AVR target. Since the

>> boards are simple SoC, there is not issue with unavailable

>> default devices there.

>>

>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

>> ---

>>  .gitlab-ci.yml | 7 +++++++

>>  1 file changed, 7 insertions(+)

>>

>> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml

>> index 3b15ae5c302..321cca2c216 100644

>> --- a/.gitlab-ci.yml

>> +++ b/.gitlab-ci.yml

>> @@ -262,6 +262,13 @@ build-user-plugins:

>>      MAKE_CHECK_ARGS: check-tcg

>>    timeout: 1h 30m

>>  

>> +build-system-ubuntu-without-default-devices:

>> +  <<: *native_build_job_definition

>> +  variables:

>> +    IMAGE: ubuntu2004

>> +    CONFIGURE_ARGS: --without-default-devices --disable-user --disable-tools --disable-docs

>> +    MAKE_CHECK_ARGS: check-qtest-avr

> 

> As mentioned in my other mail, we can also use -m68k if you prefer that

> instead of -avr.


I guess I send this series before reading it.
Let's use both, to support each others =)

> 

>>  build-clang:

>>    <<: *native_build_job_definition

>>    variables:

>>

> 

> Reviewed-by: Thomas Huth <thuth@redhat.com>


Thanks!
diff mbox series

Patch

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 3b15ae5c302..321cca2c216 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -262,6 +262,13 @@  build-user-plugins:
     MAKE_CHECK_ARGS: check-tcg
   timeout: 1h 30m
 
+build-system-ubuntu-without-default-devices:
+  <<: *native_build_job_definition
+  variables:
+    IMAGE: ubuntu2004
+    CONFIGURE_ARGS: --without-default-devices --disable-user --disable-tools --disable-docs
+    MAKE_CHECK_ARGS: check-qtest-avr
+
 build-clang:
   <<: *native_build_job_definition
   variables: