[virglrenderer-devel] [PATCH 2/2] Add CI configuration for Gitlab

[emil.velikov at collabora.com]

On Fri, Apr 20, 2018 at 04:16:49PM +0200, Tomeu Vizoso wrote:
> Adds a simple script that configures a CI pipeline to be used in Gitlab
> installations.
> 
> For now it only makes sure that make distcheck works when running under
> LLVMPipe.
> 
> Signed-off-by: Tomeu Vizoso <tomeu.vizoso at collabora.com>
> ---
>  .gitlab-ci.yml | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>  create mode 100644 .gitlab-ci.yml
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> new file mode 100644
> index 000000000000..a6000ea1d480
> --- /dev/null
> +++ b/.gitlab-ci.yml
> @@ -0,0 +1,20 @@
> +image: debian:unstable
> +
> +before_script:
> +   - echo deb-src http://deb.debian.org/debian unstable main >> /etc/apt/sources.list
> +   - apt-get update
> +   - apt-get -y build-dep virglrenderer
> +   - apt-get -y install check
> +   - mkdir -p /dev/dri
> +   - touch /dev/dri/renderD128 # Both surfaceless and kms_swrast need a device file
> +
Gut suggests that we need a real node here, yet I haven't looked at the
tests :-\ There might be a bug lurking in there.


> +  - export LIBGL_ALWAYS_SOFTWARE=true GALLIUM_DRIVER=llvmpipe VIRGL_EGL_SURFACELESS=yes
As mentioned in 1/2 the LIBGL and GALLIUM variable should no be needed.
If that's not the case please add a comment - better yet report a Mesa
bug ;-)

> +  - make -j$(nproc) distcheck
> +
> +after_script:
> +  - cat ./virglrenderer-*/_build/sub/tests/test-suite.log # This file will be there only if the tests failed
Something like "test -f ... && cat ... || echo "That's all folks" might
be a better idea. As-it:
 a) a confusing "cat ... No such file or directory" will be printed
 b) the command will return !EXIT_SUCCESS, which might confuse gitlab


Same as 1/2 above are fly-by comments. My virgl knowledge is limited.

HTH
Emil