Re: [PATCH spice-server V2] gitlab-ci: build out-of-tree too

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> One test is enough -- do it in makecheck.
> 
> Must git clean first any previous builds.

No, not at the beginning.
spice-gtk CI script execute multiple test in a single job
so between jobs clean everything.

> 
> Signed-off-by: Uri Lublin <uril@xxxxxxxxxx>
> ---
> 
> Since V1:
>    - Add a comment about out-of-tree
>    - cleanup previous builds using git clean
> 
> pipeline task:
> https://gitlab.freedesktop.org/uril/spice/-/jobs/272659
> 
> BTW, should we cleanup before/after every task ?
>      should we rm -rf builddir ? spice-protocol ?

Well, we should clean between, not at the beginning, at
the beginning repository is supposed to came from a git clone.

> 
> ---
>  .gitlab-ci.yml | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 9ce95c00e..2c1f46adf 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -14,12 +14,18 @@ before_script:
>  
>  makecheck:
>    script:
> +  # Also check out-of-tree build
> +  - git clean -fdx # cleanup after previous builds
> +  - git submodule foreach --recursive git clean -fdx

There 2 lines here are useless.

> +  - mkdir builddir
> +  - cd builddir
>    - >
>      CFLAGS='-O2 -pipe -g -fsanitize=address -fno-omit-frame-pointer
>      -Wframe-larger-than=40920'
>      LDFLAGS='-fsanitize=address -lasan'
> -    ./autogen.sh --enable-celt051
> +    ../autogen.sh --enable-celt051
>    - make
>    - make -C server check || (cat server/tests/test-suite.log && exit 1)
> +  - cd ..
>  
>  meson-makecheck:
>    script:

Otherwise,

Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]