[spice-server] ci: Workaround an issue with GLib on Fedora 30

Submitted by Frediano Ziglio on July 17, 2019, 9:46 a.m.

Details

Message ID 20190717094603.12234-1-fziglio@redhat.com
State Superseded
Headers show
Series "ci: Workaround an issue with GLib on Fedora 30" ( rev: 1 ) in Spice

Not browsing as part of any series.

Commit Message

Frediano Ziglio July 17, 2019, 9:46 a.m.
This remove this error running test-listen test on a Fedora 30 docker
image:

(/builds/spice/spice/build/server/tests/test-listen:2233): GLib-GIO-CRITICAL **: 15:29:03.123: g_file_new_for_path: assertion 'path != NULL' failed

This error is due to some missing configuration on the image.
On a fully installed Desktop/Server machine these configuration
are usually in place so you won't note the issue but on recent
docker images these configuration are missing.

Running the dconf command add required configuration.

Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
---
 .gitlab-ci.yml | 2 ++
 1 file changed, 2 insertions(+)

Patch hide | download patch | download mbox

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index b303d4656..3c5298f84 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -8,9 +8,11 @@  before_script:
     glib2-devel celt051-devel pixman-devel alsa-lib-devel openssl-devel libjpeg-turbo-devel
     libcacard-devel cyrus-sasl-devel lz4-devel opus-devel
     gstreamer1-devel gstreamer1-plugins-base-devel
+    dconf
     -y
   - git clone ${CI_REPOSITORY_URL/spice.git/spice-protocol.git}
   - (cd spice-protocol && ./autogen.sh --prefix=/usr && make install)
+  - dconf reset /org/gnome/system/location/enabled || true
 
 makecheck:
   script:

Comments

On 7/17/19 12:46 PM, Frediano Ziglio wrote:
> This remove this error running test-listen test on a Fedora 30 docker
> image:
> 
> (/builds/spice/spice/build/server/tests/test-listen:2233): GLib-GIO-CRITICAL **: 15:29:03.123: g_file_new_for_path: assertion 'path != NULL' failed
> 
> This error is due to some missing configuration on the image.
> On a fully installed Desktop/Server machine these configuration
> are usually in place so you won't note the issue but on recent
> docker images these configuration are missing.
> 
> Running the dconf command add required configuration.


Nice. It seems it does indeed fix the weird failure of test-listen.
This is magic -- the patch resets 'location' (==GPS).

Thanks,
     Uri.


> 
> Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
> ---
>   .gitlab-ci.yml | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index b303d4656..3c5298f84 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -8,9 +8,11 @@ before_script:
>       glib2-devel celt051-devel pixman-devel alsa-lib-devel openssl-devel libjpeg-turbo-devel
>       libcacard-devel cyrus-sasl-devel lz4-devel opus-devel
>       gstreamer1-devel gstreamer1-plugins-base-devel
> +    dconf
>       -y
>     - git clone ${CI_REPOSITORY_URL/spice.git/spice-protocol.git}
>     - (cd spice-protocol && ./autogen.sh --prefix=/usr && make install)
> +  - dconf reset /org/gnome/system/location/enabled || true
>   
>   makecheck:
>     script:
>