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

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

 



On 5/1/19 2:30 PM, Frediano Ziglio wrote:

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.

Well, it fails without it, with the following error messages:
  configure: error: source directory already configured; run "make
             distclean" there first
  configure: error: ../../../subprojects/spice-common/configure failed
             for subprojects/spice-common

https://gitlab.freedesktop.org/uril/spice/-/jobs/271029



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.

I see "Fetching changes...", which seems to me like there is no
new clone, but a git fetch.

I'm not sure yet why it works in spice-gtk.

Thanks,
    Uri.



---
  .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]