CI: add a simple github action

[bilelmoussaoui]

Let's see if this builds fine

[bilelmoussaoui]

The tests fails in a very weird way. It complains about a missing header gio/gdesktopappinfo.h. The header is provided by glib2-devel per dnf provides */gdesktopappinfo.h which is installed... I'm a bit confused.

@GuillaumeGomez any idea?

[bilelmoussaoui]

Ah it actually fails locally as well. Weird

[sdroege]

You need to use the gio-unix-2.0 pkg-config package for getting the right includes to find that (and other) headers.

[sdroege]

Now only fails with the following:

<command-line>: error: invalid application of ‘sizeof’ to incomplete type ‘GSettingsBackend’ {aka ‘struct _GSettingsBackend’}
tests/layout.c:10:31: note: in expansion of macro ‘ABI_TYPE_NAME’
   10 |     printf("%zu\n%zu", sizeof(ABI_TYPE_NAME), alignof(ABI_TYPE_NAME));
      |                               ^~~~~~~~~~~~~
<command-line>: error: invalid application of ‘__alignof__’ to incomplete type ‘GSettingsBackend’ {aka ‘struct _GSettingsBackend’}
tests/layout.c:10:55: note: in expansion of macro ‘ABI_TYPE_NAME’
   10 |     printf("%zu\n%zu", sizeof(ABI_TYPE_NAME), alignof(ABI_TYPE_NAME));
      |                                                       ^~~~~~~~~~~~~
tests/layout.c: In function ‘main’:
<command-line>: error: ‘GSettingsBackendClass’ undeclared (first use in this function); did you mean ‘GSettingsBackend’?
<command-line>: note: in definition of macro ‘ABI_TYPE_NAME’
<command-line>: note: each undeclared identifier is reported only once for each function it appears in
<command-line>: note: in definition of macro ‘ABI_TYPE_NAME’
test cross_validate_layout_with_c ... FAILED
tests/constant.c: In function ‘main’:
<command-line>: error: ‘G_SETTINGS_BACKEND_EXTENSION_POINT_NAME’ undeclared (first use in this function); did you mean ‘G_TLS_BACKEND_EXTENSION_POINT_NAME’?
<command-line>: note: in definition of macro ‘ABI_CONSTANT_NAME’
<command-line>: note: each undeclared identifier is reported only once for each function it appears in
<command-line>: note: in definition of macro ‘ABI_CONSTANT_NAME’

This is a bug in gio: nothing includes <gio/gsettingsbackend.h>, and that's also not easy to include because of

#error "You must define G_SETTINGS_ENABLE_BACKEND before including <gio/gsettingsbackend.h>."

[sdroege]

See https://gitlab.gnome.org/GNOME/glib/-/issues/2239

[GuillaumeGomez]

Once we're done with this, a big squash will be welcome. :)

[sdroege]

You accidentally reverted db778f3, that's why the CI fails now. For this to stay around we need to fix gir.

[sdroege]

Added it back now so we can see if everything passes now at least.

Can you create an issue for this problem in gir?

Also gtk-rs/gir#979 is another issue that currently has changes to autogenerated code. We should clean those two up ASAP.

[sdroege]

Still fails because

compilation command "cc" "-Wno-deprecated-declarations" "-D__USE_MINGW_ANSI_STDIO" "-pthread" "-I/usr/include/gio-unix-2.0" "-I/usr/include/libmount" "-I/usr/include/blkid" "-I/usr/include/glib-2.0" "-I/usr/lib/x86_64-linux-gnu/glib-2.0/include" "-DABI_CONSTANT_NAME=G_SETTINGS_BACKEND_EXTENSION_POINT_NAME" "tests/constant.c" "-o" "/tmp/abil2IMTc/constant" failed, exit code: 1
compilation command "cc" "-Wno-deprecated-declarations" "-D__USE_MINGW_ANSI_STDIO" "-pthread" "-I/usr/include/gio-unix-2.0" "-I/usr/include/libmount" "-I/usr/include/blkid" "-I/usr/include/glib-2.0" "-I/usr/lib/x86_64-linux-gnu/glib-2.0/include" "-DABI_TYPE_NAME=GSettingsBackendClass" "tests/layout.c" "-o" "/tmp/abiTlYfoq/layout" failed, exit code: 1

That constant and the class struct need to be ignored separately in -sys mode. @bilelmoussaoui you update that?

[bilelmoussaoui]

It seems there's an issue with cairo_gobject_extend_get_type as --all-features enables the use_glib feature on cairo I guess? I've included libcairo-gobject but that doesn't seem to help.

[bilelmoussaoui]

This should pass this time :) I have disabled the -sys tests for the problematic ones & added them to the CI issue, I will try to tackle those as soon as I can.

[GuillaumeGomez]

It passed! \o/ If it's ok for you, I think we can merge.

[bilelmoussaoui]

All good, go ahead o/

[GuillaumeGomez]

Thanks!

[sdroege]

@bilelmoussaoui Can you create an issue for the gio-sys pkg-config problem for gir, so we can solve it properly? :)