mjakeman / extension-manager

A utility for browsing and installing GNOME Shell Extensions.
https://mattjakeman.com/apps/extension-manager
GNU General Public License v3.0
1.01k stars 80 forks source link

New GLib brought breaking api changes #647

Closed bill88t closed 6 months ago

bill88t commented 6 months ago

Describe the bug Hello there, I am the current maintainer of the AUR package of this project. As of a few days ago, new stuff landed on Arch with breaking api changes.

Here is what the current build output would look like with only #638 patched in on top of tag/v0.5.1:

+ exec meson setup --prefix /usr --libexecdir lib --sbindir bin --buildtype plain --auto-features enabled --wrap-mode nodownload -D b_pie=true -D python.bytecompile=1 extension-manager-0.5.1 build -Dbacktrace=false
The Meson build system
Version: 1.4.0
Source dir: /home/bill88t/git/extension-manager/src/extension-manager-0.5.1
Build dir: /home/bill88t/git/extension-manager/src/build
Build type: native build
Project name: extension-manager
Project version: 0.5.1
C compiler for the host machine: ccache cc (gcc 14.1.1 "cc (GCC) 14.1.1 20240507")
C linker for the host machine: cc ld.bfd 2.42.0
Host machine cpu family: x86_64
Host machine cpu: x86_64
Configuring exm-config.h using configuration
Configuring com.mattjakeman.ExtensionManager.desktop.in using configuration
Program msgfmt found: YES (/usr/bin/msgfmt)
Program desktop-file-validate found: YES (/usr/bin/desktop-file-validate)
Configuring com.mattjakeman.ExtensionManager.metainfo.xml.in using configuration
Program appstreamcli found: YES (/usr/bin/appstreamcli)
Configuring com.mattjakeman.ExtensionManager.gschema.xml using configuration
Program glib-compile-schemas found: YES (/usr/bin/glib-compile-schemas)
Library backtrace found: NO
Found pkg-config: YES (/usr/bin/pkg-config) 2.1.1
Run-time dependency gtk4 found: YES 4.14.4
Run-time dependency libadwaita-1 found: YES 1.5.0
Run-time dependency gio-unix-2.0 found: YES 2.80.2
Run-time dependency json-glib-1.0 found: YES 1.8.0
Run-time dependency libsoup-3.0 found: YES 3.4.4
Run-time dependency text-engine-0.1 found: YES 0.1.0
Program blueprint-compiler found: YES (/usr/bin/blueprint-compiler)
Build-time dependency glib-2.0 found: YES 2.80.2
Program /usr/bin/glib-mkenums found: YES (/usr/bin/glib-mkenums)
Dependency glib-2.0 found: YES 2.80.2 (cached)
Program /usr/bin/glib-mkenums found: YES (/usr/bin/glib-mkenums)
Configuring exm.gresource.xml using configuration
Build-time dependency gio-2.0 found: YES 2.80.2
Program /usr/bin/glib-compile-resources found: YES (/usr/bin/glib-compile-resources)
Program msginit found: YES (/usr/bin/msginit)
Program msgmerge found: YES (/usr/bin/msgmerge)
Program xgettext found: YES (/usr/bin/xgettext)
Dependency gio-2.0 found: YES 2.80.2 (cached)
Program /usr/bin/glib-compile-schemas found: YES (/usr/bin/glib-compile-schemas)
Program gtk4-update-icon-cache found: YES (/usr/bin/gtk4-update-icon-cache)
Program update-desktop-database found: YES (/usr/bin/update-desktop-database)
Build targets in project: 43

extension-manager 0.5.1

  User defined options
    auto_features     : enabled
    buildtype         : plain
    libexecdir        : lib
    prefix            : /usr
    sbindir           : bin
    wrap_mode         : nodownload
    python.bytecompile: 1
    b_pie             : true
    backtrace         : false

Found ninja-1.12.1 at /usr/bin/ninja
INFO: autodetecting backend as ninja                                                                                                                                                                                                           
INFO: calculating backend command to run: /usr/bin/ninja -C /home/bill88t/git/extension-manager/src/build
ninja: Entering directory `/home/bill88t/git/extension-manager/src/build'
[40/75] Compiling C object src/extension-manager.p/exm-application.c.o
../extension-manager-0.5.1/src/exm-application.c: In function ‘exm_application_finalize’:
../extension-manager-0.5.1/src/exm-application.c:48:21: warning: unused variable ‘self’ [-Wunused-variable]
   48 |     ExmApplication *self = (ExmApplication *)object;
      |                     ^~~~
../extension-manager-0.5.1/src/exm-application.c: In function ‘exm_application_open’:
../extension-manager-0.5.1/src/exm-application.c:116:38: warning: unused parameter ‘hint’ [-Wunused-parameter]
  116 |                       const gchar   *hint)
      |                       ~~~~~~~~~~~~~~~^~~~
../extension-manager-0.5.1/src/exm-application.c: In function ‘exm_application_show_about’:
../extension-manager-0.5.1/src/exm-application.c:172:44: warning: unused parameter ‘action’ [-Wunused-parameter]
  172 | exm_application_show_about (GSimpleAction *action,
      |                             ~~~~~~~~~~~~~~~^~~~~~
../extension-manager-0.5.1/src/exm-application.c:173:44: warning: unused parameter ‘parameter’ [-Wunused-parameter]
  173 |                             GVariant      *parameter,
      |                             ~~~~~~~~~~~~~~~^~~~~~~~~
[42/75] Compiling C object src/extension-manager.p/exm-zoom-picture.c.o
../extension-manager-0.5.1/src/exm-zoom-picture.c: In function ‘exm_zoom_picture_finalize’:
../extension-manager-0.5.1/src/exm-zoom-picture.c:74:25: warning: unused variable ‘self’ [-Wunused-variable]
   74 |         ExmZoomPicture *self = (ExmZoomPicture *)object;
      |                         ^~~~
../extension-manager-0.5.1/src/exm-zoom-picture.c: In function ‘exm_zoom_picture_get_zoom_level_max’:
../extension-manager-0.5.1/src/exm-zoom-picture.c:151:54: warning: unused parameter ‘self’ [-Wunused-parameter]
  151 | exm_zoom_picture_get_zoom_level_max (ExmZoomPicture *self)
      |                                      ~~~~~~~~~~~~~~~~^~~~
../extension-manager-0.5.1/src/exm-zoom-picture.c: In function ‘exm_zoom_picture_get_zoom_level_min’:
../extension-manager-0.5.1/src/exm-zoom-picture.c:157:54: warning: unused parameter ‘self’ [-Wunused-parameter]
  157 | exm_zoom_picture_get_zoom_level_min (ExmZoomPicture *self)
      |                                      ~~~~~~~~~~~~~~~~^~~~
../extension-manager-0.5.1/src/exm-zoom-picture.c: In function ‘exm_zoom_picture_get_zoom_level_step’:
../extension-manager-0.5.1/src/exm-zoom-picture.c:163:55: warning: unused parameter ‘self’ [-Wunused-parameter]
  163 | exm_zoom_picture_get_zoom_level_step (ExmZoomPicture *self)
      |                                       ~~~~~~~~~~~~~~~~^~~~
../extension-manager-0.5.1/src/exm-zoom-picture.c: In function ‘on_gesture_begin’:
../extension-manager-0.5.1/src/exm-zoom-picture.c:284:53: warning: unused parameter ‘sequence’ [-Wunused-parameter]
  284 |                                   GdkEventSequence *sequence,
      |                                   ~~~~~~~~~~~~~~~~~~^~~~~~~~
../extension-manager-0.5.1/src/exm-zoom-picture.c: In function ‘on_drag_update’:
../extension-manager-0.5.1/src/exm-zoom-picture.c:322:33: warning: unused parameter ‘gesture’ [-Wunused-parameter]
  322 | on_drag_update (GtkGestureDrag *gesture,
      |                 ~~~~~~~~~~~~~~~~^~~~~~~
[43/75] Compiling C object src/extension-manager.p/main.c.o
../extension-manager-0.5.1/src/main.c: In function ‘handler’:
../extension-manager-0.5.1/src/main.c:49:9: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
   49 |         write (pipe_fd[1], backtrace, strlen (backtrace));
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[44/75] Compiling C object src/extension-manager.p/exm-screenshot.c.o
../extension-manager-0.5.1/src/exm-screenshot.c: In function ‘exm_screenshot_get_property’:
../extension-manager-0.5.1/src/exm-screenshot.c:46:20: warning: unused variable ‘self’ [-Wunused-variable]
   46 |     ExmScreenshot *self = EXM_SCREENSHOT (object);
      |                    ^~~~
../extension-manager-0.5.1/src/exm-screenshot.c:43:42: warning: unused parameter ‘value’ [-Wunused-parameter]
   43 |                              GValue     *value,
      |                              ~~~~~~~~~~~~^~~~~
../extension-manager-0.5.1/src/exm-screenshot.c: In function ‘exm_screenshot_set_property’:
../extension-manager-0.5.1/src/exm-screenshot.c:61:20: warning: unused variable ‘self’ [-Wunused-variable]
   61 |     ExmScreenshot *self = EXM_SCREENSHOT (object);
      |                    ^~~~
../extension-manager-0.5.1/src/exm-screenshot.c:58:44: warning: unused parameter ‘value’ [-Wunused-parameter]
   58 |                              const GValue *value,
      |                              ~~~~~~~~~~~~~~^~~~~
../extension-manager-0.5.1/src/exm-screenshot.c: In function ‘exm_screenshot_get_request_mode’:
../extension-manager-0.5.1/src/exm-screenshot.c:117:45: warning: unused parameter ‘widget’ [-Wunused-parameter]
  117 | exm_screenshot_get_request_mode (GtkWidget *widget)
      |                                  ~~~~~~~~~~~^~~~~~
../extension-manager-0.5.1/src/exm-screenshot.c: At top level:
../extension-manager-0.5.1/src/exm-screenshot.c:20:20: warning: ‘properties’ defined but not used [-Wunused-variable]
   20 | static GParamSpec *properties [N_PROPS];
      |                    ^~~~~~~~~~
[45/75] Compiling C object src/extension-manager.p/exm-comment-tile.c.o
../extension-manager-0.5.1/src/exm-comment-tile.c: In function ‘exm_comment_tile_finalize’:
../extension-manager-0.5.1/src/exm-comment-tile.c:41:21: warning: unused variable ‘self’ [-Wunused-variable]
   41 |     ExmCommentTile *self = (ExmCommentTile *)object;
      |                     ^~~~
[46/75] Compiling C object src/extension-manager.p/exm-comment-dialog.c.o
../extension-manager-0.5.1/src/exm-comment-dialog.c: In function ‘on_get_comments’:
../extension-manager-0.5.1/src/exm-comment-dialog.c:145:30: warning: cast between incompatible function types from ‘GtkWidget * (*)(ExmComment *)’ {aka ‘struct _GtkWidget * (*)(struct _ExmComment *)’} to ‘GtkWidget * (*)(void *, void *)’ {aka ‘struct _GtkWidget * (*)(void *, void *)’} [-Wcast-function-type]
  145 |                              (GtkListBoxCreateWidgetFunc) comment_factory,
      |                              ^
[47/75] Compiling C object src/extension-manager.p/exm-rating.c.o
../extension-manager-0.5.1/src/exm-rating.c: In function ‘exm_rating_finalize’:
../extension-manager-0.5.1/src/exm-rating.c:35:16: warning: unused variable ‘self’ [-Wunused-variable]
   35 |     ExmRating *self = (ExmRating *)object;
      |                ^~~~
../extension-manager-0.5.1/src/exm-rating.c: In function ‘update_rating’:
../extension-manager-0.5.1/src/exm-rating.c:93:9: warning: this statement may fall through [-Wimplicit-fallthrough=]
   93 |         gtk_image_set_from_icon_name (self->star_five, FILLED_ICON_NAME);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-rating.c:94:5: note: here
   94 |     case 4:
      |     ^~~~
../extension-manager-0.5.1/src/exm-rating.c:95:9: warning: this statement may fall through [-Wimplicit-fallthrough=]
   95 |         gtk_image_set_from_icon_name (self->star_four, FILLED_ICON_NAME);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-rating.c:96:5: note: here
   96 |     case 3:
      |     ^~~~
../extension-manager-0.5.1/src/exm-rating.c:97:9: warning: this statement may fall through [-Wimplicit-fallthrough=]
   97 |         gtk_image_set_from_icon_name (self->star_three, FILLED_ICON_NAME);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-rating.c:98:5: note: here
   98 |     case 2:
      |     ^~~~
../extension-manager-0.5.1/src/exm-rating.c:99:9: warning: this statement may fall through [-Wimplicit-fallthrough=]
   99 |         gtk_image_set_from_icon_name (self->star_two, FILLED_ICON_NAME);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-rating.c:100:5: note: here
  100 |     case 1:
      |     ^~~~
[49/75] Compiling C object src/extension-manager.p/exm-info-bar.c.o
../extension-manager-0.5.1/src/exm-info-bar.c: In function ‘exm_info_bar_finalize’:
../extension-manager-0.5.1/src/exm-info-bar.c:35:17: warning: unused variable ‘self’ [-Wunused-variable]
   35 |     ExmInfoBar *self = (ExmInfoBar *)object;
      |                 ^~~~
../extension-manager-0.5.1/src/exm-info-bar.c: In function ‘exm_info_bar_get_property’:
../extension-manager-0.5.1/src/exm-info-bar.c:46:17: warning: unused variable ‘self’ [-Wunused-variable]
   46 |     ExmInfoBar *self = EXM_INFO_BAR (object);
      |                 ^~~~
../extension-manager-0.5.1/src/exm-info-bar.c:43:40: warning: unused parameter ‘value’ [-Wunused-parameter]
   43 |                            GValue     *value,
      |                            ~~~~~~~~~~~~^~~~~
../extension-manager-0.5.1/src/exm-info-bar.c: In function ‘exm_info_bar_set_property’:
../extension-manager-0.5.1/src/exm-info-bar.c:61:17: warning: unused variable ‘self’ [-Wunused-variable]
   61 |     ExmInfoBar *self = EXM_INFO_BAR (object);
      |                 ^~~~
../extension-manager-0.5.1/src/exm-info-bar.c:58:42: warning: unused parameter ‘value’ [-Wunused-parameter]
   58 |                            const GValue *value,
      |                            ~~~~~~~~~~~~~~^~~~~
../extension-manager-0.5.1/src/exm-info-bar.c: At top level:
../extension-manager-0.5.1/src/exm-info-bar.c:24:20: warning: ‘properties’ defined but not used [-Wunused-variable]
   24 | static GParamSpec *properties [N_PROPS];
      |                    ^~~~~~~~~~
[50/75] Compiling C object src/extension-manager.p/exm-info-bar-item.c.o
../extension-manager-0.5.1/src/exm-info-bar-item.c: In function ‘exm_info_bar_item_finalize’:
../extension-manager-0.5.1/src/exm-info-bar-item.c:33:21: warning: unused variable ‘self’ [-Wunused-variable]
   33 |     ExmInfoBarItem *self = (ExmInfoBarItem *)object;
      |                     ^~~~
../extension-manager-0.5.1/src/exm-info-bar-item.c: In function ‘exm_info_bar_item_get_property’:
../extension-manager-0.5.1/src/exm-info-bar-item.c:77:9: warning: this statement may fall through [-Wimplicit-fallthrough=]
   77 |         g_value_set_string (value, gtk_label_get_text (exm_info_bar_item_get_subtitle (self)));
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-info-bar-item.c:78:5: note: here
   78 |     default:
      |     ^~~~~~~
[51/75] Compiling C object src/extension-manager.p/exm-screenshot-view.c.o
../extension-manager-0.5.1/src/exm-screenshot-view.c: In function ‘exm_screenshot_view_finalize’:
../extension-manager-0.5.1/src/exm-screenshot-view.c:27:24: warning: unused variable ‘self’ [-Wunused-variable]
   27 |     ExmScreenshotView *self = (ExmScreenshotView *)object;
      |                        ^~~~
../extension-manager-0.5.1/src/exm-screenshot-view.c: In function ‘notify_zoom’:
../extension-manager-0.5.1/src/exm-screenshot-view.c:42:33: warning: unused parameter ‘pspec’ [-Wunused-parameter]
   42 |              GParamSpec        *pspec,
      |              ~~~~~~~~~~~~~~~~~~~^~~~~
[52/75] Compiling C object src/extension-manager.p/exm-browse-page.c.o
../extension-manager-0.5.1/src/exm-browse-page.c: In function ‘on_first_page_result’:
../extension-manager-0.5.1/src/exm-browse-page.c:184:9: warning: unused variable ‘i’ [-Wunused-variable]
  184 |     int i;
      |         ^
../extension-manager-0.5.1/src/exm-browse-page.c:183:9: warning: variable ‘n_items’ set but not used [-Wunused-but-set-variable]
  183 |     int n_items;
      |         ^~~~~~~
../extension-manager-0.5.1/src/exm-browse-page.c: In function ‘on_load_more_results’:
../extension-manager-0.5.1/src/exm-browse-page.c:235:38: warning: unused parameter ‘btn’ [-Wunused-parameter]
  235 | on_load_more_results (GtkButton     *btn,
      |                       ~~~~~~~~~~~~~~~^~~
[53/75] Compiling C object src/extension-manager.p/exm-search-row.c.o
../extension-manager-0.5.1/src/exm-search-row.c: In function ‘exm_search_row_finalize’:
../extension-manager-0.5.1/src/exm-search-row.c:49:19: warning: unused variable ‘self’ [-Wunused-variable]
   49 |     ExmSearchRow *self = (ExmSearchRow *)object;
      |                   ^~~~
[54/75] Compiling C object src/extension-manager.p/exm-error-dialog.c.o
../extension-manager-0.5.1/src/exm-error-dialog.c: In function ‘exm_error_dialog_finalize’:
../extension-manager-0.5.1/src/exm-error-dialog.c:58:21: warning: unused variable ‘self’ [-Wunused-variable]
   58 |     ExmErrorDialog *self = (ExmErrorDialog *)object;
      |                     ^~~~
[55/75] Compiling C object src/extension-manager.p/exm-upgrade-result.c.o
../extension-manager-0.5.1/src/exm-upgrade-result.c: In function ‘exm_upgrade_result_set_property’:
../extension-manager-0.5.1/src/exm-upgrade-result.c:85:23: warning: unused variable ‘self’ [-Wunused-variable]
   85 |     ExmUpgradeResult *self = EXM_UPGRADE_RESULT (object);
      |                       ^~~~
../extension-manager-0.5.1/src/exm-upgrade-result.c:82:48: warning: unused parameter ‘value’ [-Wunused-parameter]
   82 |                                  const GValue *value,
      |                                  ~~~~~~~~~~~~~~^~~~~
[57/75] Compiling C object src/extension-manager.p/exm-install-button.c.o
../extension-manager-0.5.1/src/exm-install-button.c: In function ‘exm_install_button_finalize’:
../extension-manager-0.5.1/src/exm-install-button.c:57:23: warning: unused variable ‘self’ [-Wunused-variable]
   57 |     ExmInstallButton *self = (ExmInstallButton *)object;
      |                       ^~~~
../extension-manager-0.5.1/src/exm-install-button.c: In function ‘exm_install_button_init’:
../extension-manager-0.5.1/src/exm-install-button.c:153:44: warning: unused parameter ‘self’ [-Wunused-parameter]
  153 | exm_install_button_init (ExmInstallButton *self)
      |                          ~~~~~~~~~~~~~~~~~~^~~~
[58/75] Compiling C object src/extension-manager.p/exm-backtrace.c.o
../extension-manager-0.5.1/src/exm-backtrace.c: In function ‘exm_backtrace_init’:
../extension-manager-0.5.1/src/exm-backtrace.c:76:27: warning: unused parameter ‘filename’ [-Wunused-parameter]
   76 | exm_backtrace_init (char *filename)
      |                     ~~~~~~^~~~~~~~
[59/75] Compiling C object src/extension-manager.p/local_exm-extension.c.o
../extension-manager-0.5.1/src/local/exm-extension.c: In function ‘exm_extension_finalize’:
../extension-manager-0.5.1/src/local/exm-extension.c:53:19: warning: unused variable ‘self’ [-Wunused-variable]
   53 |     ExmExtension *self = (ExmExtension *)object;
      |                   ^~~~
../extension-manager-0.5.1/src/local/exm-extension.c: In function ‘exm_extension_init’:
../extension-manager-0.5.1/src/local/exm-extension.c:249:35: warning: unused parameter ‘self’ [-Wunused-parameter]
  249 | exm_extension_init (ExmExtension *self)
      |                     ~~~~~~~~~~~~~~^~~~
[60/75] Compiling C object src/extension-manager.p/local_exm-manager.c.o
../extension-manager-0.5.1/src/local/exm-manager.c: In function ‘exm_manager_finalize’:
../extension-manager-0.5.1/src/local/exm-manager.c:54:17: warning: unused variable ‘self’ [-Wunused-variable]
   54 |     ExmManager *self = (ExmManager *)object;
      |                 ^~~~
../extension-manager-0.5.1/src/local/exm-manager.c: In function ‘on_state_changed’:
../extension-manager-0.5.1/src/local/exm-manager.c:731:36: warning: unused parameter ‘object’ [-Wunused-parameter]
  731 | on_state_changed (ShellExtensions *object,
      |                   ~~~~~~~~~~~~~~~~~^~~~~~
../extension-manager-0.5.1/src/local/exm-manager.c: In function ‘exm_manager_init’:
../extension-manager-0.5.1/src/local/exm-manager.c:805:15: warning: variable ‘error_text’ set but not used [-Wunused-but-set-variable]
  805 |         char *error_text;
      |               ^~~~~~~~~~
../extension-manager-0.5.1/src/local/exm-manager.c: In function ‘parse_single_extension’:
../extension-manager-0.5.1/src/local/exm-manager.c:634:5: warning: ‘state’ may be used uninitialized [-Wmaybe-uninitialized]
  634 |     g_object_set (*extension,
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~
  635 |                   "display-name", display_name,
      |                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  636 |                   "description", description,
      |                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~
  637 |                   "state", state,
      |                   ~~~~~~~~~~~~~~~
  638 |                   "enabled", enabled,
      |                   ~~~~~~~~~~~~~~~~~~~
  639 |                   "is-user", *is_user,
      |                   ~~~~~~~~~~~~~~~~~~~~
  640 |                   "has-prefs", has_prefs,
      |                   ~~~~~~~~~~~~~~~~~~~~~~~
  641 |                   "has-update", has_update,
      |                   ~~~~~~~~~~~~~~~~~~~~~~~~~
  642 |                   "can-change", can_change,
      |                   ~~~~~~~~~~~~~~~~~~~~~~~~~
  643 |                   "version", version,
      |                   ~~~~~~~~~~~~~~~~~~~
  644 |                   "error-msg", error_msg,
      |                   ~~~~~~~~~~~~~~~~~~~~~~~
  645 |                   NULL);
      |                   ~~~~~
../extension-manager-0.5.1/src/local/exm-manager.c:553:23: note: ‘state’ was declared here
  553 |     ExmExtensionState state;
      |                       ^~~~~
../extension-manager-0.5.1/src/local/exm-manager.c:631:22: warning: ‘type’ may be used uninitialized [-Wmaybe-uninitialized]
  631 |     *is_user = (type == EXM_EXTENSION_TYPE_PER_USER);
      |                ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/local/exm-manager.c:554:22: note: ‘type’ was declared here
  554 |     ExmExtensionType type;
      |                      ^~~~
../extension-manager-0.5.1/src/local/exm-manager.c: In function ‘exm_manager_init’:
../extension-manager-0.5.1/src/local/exm-manager.c:141:22: warning: ‘error_text’ may be used uninitialized [-Wmaybe-uninitialized]
  141 |         error_text = g_strdup_printf (f_, __VA_ARGS__);\
      |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/local/exm-manager.c:807:9: note: in expansion of macro ‘notify_error’
  807 |         notify_error (self, "%s", error_text);
      |         ^~~~~~~~~~~~
../extension-manager-0.5.1/src/local/exm-manager.c:140:15: note: ‘error_text’ was declared here
  140 |         char *error_text;\
      |               ^~~~~~~~~~
../extension-manager-0.5.1/src/local/exm-manager.c:807:9: note: in expansion of macro ‘notify_error’
  807 |         notify_error (self, "%s", error_text);
      |         ^~~~~~~~~~~~
[61/75] Compiling C object src/extension-manager.p/web_exm-search-provider.c.o
../extension-manager-0.5.1/src/web/exm-search-provider.c: In function ‘exm_search_provider_finalize’:
../extension-manager-0.5.1/src/web/exm-search-provider.c:40:24: warning: unused variable ‘self’ [-Wunused-variable]
   40 |     ExmSearchProvider *self = (ExmSearchProvider *)object;
      |                        ^~~~
[63/75] Compiling C object src/extension-manager.p/web_exm-image-resolver.c.o
../extension-manager-0.5.1/src/web/exm-image-resolver.c:20:20: warning: ‘properties’ defined but not used [-Wunused-variable]
   20 | static GParamSpec *properties [N_PROPS];
      |                    ^~~~~~~~~~
[64/75] Compiling C object src/extension-manager.p/web_exm-data-provider.c.o
../extension-manager-0.5.1/src/web/exm-data-provider.c: In function ‘exm_data_provider_finalize’:
../extension-manager-0.5.1/src/web/exm-data-provider.c:23:22: warning: unused variable ‘self’ [-Wunused-variable]
   23 |     ExmDataProvider *self = (ExmDataProvider *)object;
      |                      ^~~~
../extension-manager-0.5.1/src/web/exm-data-provider.c: In function ‘exm_data_provider_init’:
../extension-manager-0.5.1/src/web/exm-data-provider.c:107:42: warning: unused parameter ‘self’ [-Wunused-parameter]
  107 | exm_data_provider_init (ExmDataProvider *self)
      |                         ~~~~~~~~~~~~~~~~~^~~~
[65/75] Compiling C object src/extension-manager.p/web_exm-request-handler.c.o
../extension-manager-0.5.1/src/web/exm-request-handler.c: In function ‘default_handle_response’:
../extension-manager-0.5.1/src/web/exm-request-handler.c:38:35: warning: unused parameter ‘bytes’ [-Wunused-parameter]
   38 | default_handle_response (GBytes  *bytes,
      |                          ~~~~~~~~~^~~~~
[66/75] Compiling C object src/extension-manager.p/web_exm-comment-provider.c.o
../extension-manager-0.5.1/src/web/exm-comment-provider.c: In function ‘exm_comment_provider_finalize’:
../extension-manager-0.5.1/src/web/exm-comment-provider.c:30:25: warning: unused variable ‘self’ [-Wunused-variable]
   30 |     ExmCommentProvider *self = (ExmCommentProvider *)object;
      |                         ^~~~
../extension-manager-0.5.1/src/web/exm-comment-provider.c: In function ‘exm_comment_provider_init’:
../extension-manager-0.5.1/src/web/exm-comment-provider.c:130:48: warning: unused parameter ‘self’ [-Wunused-parameter]
  130 | exm_comment_provider_init (ExmCommentProvider *self)
      |                            ~~~~~~~~~~~~~~~~~~~~^~~~
../extension-manager-0.5.1/src/web/exm-comment-provider.c: At top level:
../extension-manager-0.5.1/src/web/exm-comment-provider.c:19:20: warning: ‘properties’ defined but not used [-Wunused-variable]
   19 | static GParamSpec *properties [N_PROPS];
      |                    ^~~~~~~~~~
[67/75] Compiling C object src/extension-manager.p/web_model_exm-search-result.c.o
../extension-manager-0.5.1/src/web/model/exm-search-result.c: In function ‘exm_search_result_set_property’:
../extension-manager-0.5.1/src/web/model/exm-search-result.c:120:31: warning: unused variable ‘map’ [-Wunused-variable]
  120 |     const ExmShellVersionMap *map;
      |                               ^~~
../extension-manager-0.5.1/src/web/model/exm-search-result.c: In function ‘deserialize_version’:
../extension-manager-0.5.1/src/web/model/exm-search-result.c:176:42: warning: unused parameter ‘object’ [-Wunused-parameter]
  176 | deserialize_version (JsonObject         *object,
      |                      ~~~~~~~~~~~~~~~~~~~~^~~~~~
../extension-manager-0.5.1/src/web/model/exm-search-result.c: In function ‘exm_search_result_init’:
../extension-manager-0.5.1/src/web/model/exm-search-result.c:303:42: warning: unused parameter ‘self’ [-Wunused-parameter]
  303 | exm_search_result_init (ExmSearchResult *self)
      |                         ~~~~~~~~~~~~~~~~~^~~~
../extension-manager-0.5.1/src/web/model/exm-search-result.c: In function ‘json_serializable_iface_init’:
../extension-manager-0.5.1/src/web/model/exm-search-result.c:308:54: warning: unused parameter ‘iface’ [-Wunused-parameter]
  308 | json_serializable_iface_init (JsonSerializableIface *iface)
      |                               ~~~~~~~~~~~~~~~~~~~~~~~^~~~~
[68/75] Compiling C object src/extension-manager.p/web_model_exm-comment.c.o
../extension-manager-0.5.1/src/web/model/exm-comment.c: In function ‘exm_comment_finalize’:
../extension-manager-0.5.1/src/web/model/exm-comment.c:43:17: warning: unused variable ‘self’ [-Wunused-variable]
   43 |     ExmComment *self = (ExmComment *)object;
      |                 ^~~~
[70/75] Compiling C object src/extension-manager.p/exm-window.c.o
FAILED: src/extension-manager.p/exm-window.c.o 
ccache cc -Isrc/extension-manager.p -Isrc -I../extension-manager-0.5.1/src -I/usr/include/gtk-4.0 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/freetype2 -I/usr/include/graphene-1.0 -I/usr/lib/graphene-1.0/include -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/sysprof-6 -I/usr/include/fribidi -I/usr/include/libpng16 -I/usr/include/pixman-1 -I/usr/include/libadwaita-1 -I/usr/include/appstream -I/usr/include/gio-unix-2.0 -I/usr/include/json-glib-1.0 -I/usr/include/libsoup-3.0 -I/usr/include/text-engine -I/home/bill88t/git/extension-manager/src/build -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -Werror=implicit-function-declaration -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -fPIE -pthread -mfpmath=sse -msse -msse2 -mfpmath=sse -msse -msse2 -MD -MQ src/extension-manager.p/exm-window.c.o -MF src/extension-manager.p/exm-window.c.o.d -o src/extension-manager.p/exm-window.c.o -c ../extension-manager-0.5.1/src/exm-window.c
../extension-manager-0.5.1/src/exm-window.c: In function ‘exm_window_finalize’:
../extension-manager-0.5.1/src/exm-window.c:66:16: warning: unused variable ‘self’ [-Wunused-variable]
   66 |     ExmWindow *self = (ExmWindow *)object;
      |                ^~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘exm_window_set_property’:
../extension-manager-0.5.1/src/exm-window.c:95:16: warning: unused variable ‘self’ [-Wunused-variable]
   95 |     ExmWindow *self = EXM_WINDOW (object);
      |                ^~~~
../extension-manager-0.5.1/src/exm-window.c:92:40: warning: unused parameter ‘value’ [-Wunused-parameter]
   92 |                          const GValue *value,
      |                          ~~~~~~~~~~~~~~^~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘extension_open_prefs’:
../extension-manager-0.5.1/src/exm-window.c:106:35: warning: unused parameter ‘action_name’ [-Wunused-parameter]
  106 |                       const char *action_name,
      |                       ~~~~~~~~~~~~^~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘extension_remove’:
../extension-manager-0.5.1/src/exm-window.c:146:31: warning: unused parameter ‘action_name’ [-Wunused-parameter]
  146 |                   const char *action_name,
      |                   ~~~~~~~~~~~~^~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘extension_unsupported_dialog_response’:
../extension-manager-0.5.1/src/exm-window.c:194:36: warning: cast between incompatible function types from ‘void (*)(GObject *, GAsyncResult *)’ {aka ‘void (*)(struct _GObject *, struct _GAsyncResult *)’} to ‘void (*)(GObject *, GAsyncResult *, void *)’ {aka ‘void (*)(struct _GObject *, struct _GAsyncResult *, void *)’} [-Wcast-function-type]
  194 |                                    (GAsyncReadyCallback) on_install_done,
      |                                    ^
../extension-manager-0.5.1/src/exm-window.c: In function ‘extension_install’:
../extension-manager-0.5.1/src/exm-window.c:228:32: warning: cast between incompatible function types from ‘void (*)(GObject *, GAsyncResult *)’ {aka ‘void (*)(struct _GObject *, struct _GAsyncResult *)’} to ‘void (*)(GObject *, GAsyncResult *, void *)’ {aka ‘void (*)(struct _GObject *, struct _GAsyncResult *, void *)’} [-Wcast-function-type]
  228 |                                (GAsyncReadyCallback) on_install_done,
      |                                ^
../extension-manager-0.5.1/src/exm-window.c:205:32: warning: unused parameter ‘action_name’ [-Wunused-parameter]
  205 |                    const char *action_name,
      |                    ~~~~~~~~~~~~^~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘show_page’:
../extension-manager-0.5.1/src/exm-window.c:234:24: warning: unused parameter ‘action_name’ [-Wunused-parameter]
  234 |            const char *action_name,
      |            ~~~~~~~~~~~~^~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘show_upgrade_assistant’:
../extension-manager-0.5.1/src/exm-window.c:274:37: warning: unused parameter ‘action_name’ [-Wunused-parameter]
  274 |                         const char *action_name,
      |                         ~~~~~~~~~~~~^~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c:275:37: warning: unused parameter ‘param’ [-Wunused-parameter]
  275 |                         GVariant   *param)
      |                         ~~~~~~~~~~~~^~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘show_error_dialog’:
../extension-manager-0.5.1/src/exm-window.c:287:32: warning: unused parameter ‘action_name’ [-Wunused-parameter]
  287 |                    const char *action_name,
      |                    ~~~~~~~~~~~~^~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘show_error’:
../extension-manager-0.5.1/src/exm-window.c:304:25: warning: unused parameter ‘action_name’ [-Wunused-parameter]
  304 |             const char *action_name,
      |             ~~~~~~~~~~~~^~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘on_error’:
../extension-manager-0.5.1/src/exm-window.c:327:23: warning: unused parameter ‘manager’ [-Wunused-parameter]
  327 | on_error (ExmManager *manager,
      |           ~~~~~~~~~~~~^~~~~~~
../extension-manager-0.5.1/src/exm-window.c: In function ‘do_version_check’:
../extension-manager-0.5.1/src/exm-window.c:385:12: warning: variable ‘version_string’ set but not used [-Wunused-but-set-variable]
  385 |     gchar *version_string;
      |            ^~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-window.c:382:30: warning: unused parameter ‘self’ [-Wunused-parameter]
  382 | do_version_check (ExmWindow *self)
      |                   ~~~~~~~~~~~^~~~
In file included from /usr/include/glib-2.0/gobject/gobject.h:30,
                 from /usr/include/glib-2.0/gobject/gbinding.h:31,
                 from /usr/include/glib-2.0/glib-object.h:24,
                 from /usr/include/gtk-4.0/gtk/css/gtkcssenumtypes.h:11,
                 from /usr/include/gtk-4.0/gtk/css/gtkcss.h:33,
                 from /usr/include/gtk-4.0/gtk/gtk.h:29,
                 from /usr/include/libadwaita-1/adwaita.h:9,
                 from ../extension-manager-0.5.1/src/exm-window.h:21,
                 from ../extension-manager-0.5.1/src/exm-window.c:20:
../extension-manager-0.5.1/src/exm-window.c: In function ‘exm_window_init’:
/usr/include/glib-2.0/gobject/gsignal.h:519:59: error: passing argument 3 of ‘g_signal_connect_data’ from incompatible pointer type [-Wincompatible-pointer-types]
  519 |     g_signal_connect_data ((instance), (detailed_signal), (c_handler), (data), NULL, (GConnectFlags) 0)
      |                                                           ^~~~~~~~~~~
      |                                                           |
      |                                                           void (*)(ExmManager *, char *, ExmWindow *) {aka void (*)(struct _ExmManager *, char *, struct _ExmWindow *)}
../extension-manager-0.5.1/src/exm-window.c:407:5: note: in expansion of macro ‘g_signal_connect’
  407 |     g_signal_connect (self->manager, "error-occurred", on_error, self);
      |     ^~~~~~~~~~~~~~~~
/usr/include/glib-2.0/gobject/gsignal.h:411:67: note: expected ‘GCallback’ {aka ‘void (*)(void)’} but argument is of type ‘void (*)(ExmManager *, char *, ExmWindow *)’ {aka ‘void (*)(struct _ExmManager *, char *, struct _ExmWindow *)’}
  411 |                                                GCallback          c_handler,
      |                                                ~~~~~~~~~~~~~~~~~~~^~~~~~~~~
[71/75] Compiling C object src/extension-manager.p/exm-upgrade-assistant.c.o
FAILED: src/extension-manager.p/exm-upgrade-assistant.c.o 
ccache cc -Isrc/extension-manager.p -Isrc -I../extension-manager-0.5.1/src -I/usr/include/gtk-4.0 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/freetype2 -I/usr/include/graphene-1.0 -I/usr/lib/graphene-1.0/include -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/sysprof-6 -I/usr/include/fribidi -I/usr/include/libpng16 -I/usr/include/pixman-1 -I/usr/include/libadwaita-1 -I/usr/include/appstream -I/usr/include/gio-unix-2.0 -I/usr/include/json-glib-1.0 -I/usr/include/libsoup-3.0 -I/usr/include/text-engine -I/home/bill88t/git/extension-manager/src/build -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -Werror=implicit-function-declaration -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -fPIE -pthread -mfpmath=sse -msse -msse2 -mfpmath=sse -msse -msse2 -MD -MQ src/extension-manager.p/exm-upgrade-assistant.c.o -MF src/extension-manager.p/exm-upgrade-assistant.c.o.d -o src/extension-manager.p/exm-upgrade-assistant.c.o -c ../extension-manager-0.5.1/src/exm-upgrade-assistant.c
../extension-manager-0.5.1/src/exm-upgrade-assistant.c: In function ‘exm_upgrade_assistant_finalize’:
../extension-manager-0.5.1/src/exm-upgrade-assistant.c:90:26: warning: unused variable ‘self’ [-Wunused-variable]
   90 |     ExmUpgradeAssistant *self = (ExmUpgradeAssistant *)object;
      |                          ^~~~
../extension-manager-0.5.1/src/exm-upgrade-assistant.c: In function ‘print_list_model’:
../extension-manager-0.5.1/src/exm-upgrade-assistant.c:253:46: warning: unused variable ‘url’ [-Wunused-variable]
  253 |         const gchar *name, *creator, *uuid, *url, *supported_text;
      |                                              ^~~
../extension-manager-0.5.1/src/exm-upgrade-assistant.c: In function ‘do_compatibility_check’:
../extension-manager-0.5.1/src/exm-upgrade-assistant.c:409:22: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
  409 |     target_version = (int) g_hash_table_lookup (self->version_map, key);
      |                      ^
../extension-manager-0.5.1/src/exm-upgrade-assistant.c: In function ‘populate_drop_down’:
../extension-manager-0.5.1/src/exm-upgrade-assistant.c:598:47: error: passing argument 3 of ‘g_hash_table_insert’ makes pointer from integer without a cast [-Wint-conversion]
  598 |         g_hash_table_insert (hash_table, key, index);
      |                                               ^~~~~
      |                                               |
      |                                               int
In file included from /usr/include/glib-2.0/glib.h:52,
                 from /usr/include/gtk-4.0/gtk/css/gtkcss.h:29,
                 from /usr/include/gtk-4.0/gtk/gtk.h:29,
                 from /usr/include/libadwaita-1/adwaita.h:9,
                 from ../extension-manager-0.5.1/src/exm-upgrade-assistant.h:23,
                 from ../extension-manager-0.5.1/src/exm-upgrade-assistant.c:22:
/usr/include/glib-2.0/glib/ghash.h:74:61: note: expected ‘gpointer’ {aka ‘void *’} but argument is of type ‘int’
   74 |                                             gpointer        value);
      |                                             ~~~~~~~~~~~~~~~~^~~~~
[72/75] Compiling C object src/extension-manager.p/exm-detail-view.c.o
FAILED: src/extension-manager.p/exm-detail-view.c.o 
ccache cc -Isrc/extension-manager.p -Isrc -I../extension-manager-0.5.1/src -I/usr/include/gtk-4.0 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/freetype2 -I/usr/include/graphene-1.0 -I/usr/lib/graphene-1.0/include -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/sysprof-6 -I/usr/include/fribidi -I/usr/include/libpng16 -I/usr/include/pixman-1 -I/usr/include/libadwaita-1 -I/usr/include/appstream -I/usr/include/gio-unix-2.0 -I/usr/include/json-glib-1.0 -I/usr/include/libsoup-3.0 -I/usr/include/text-engine -I/home/bill88t/git/extension-manager/src/build -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -Werror=implicit-function-declaration -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -fPIE -pthread -mfpmath=sse -msse -msse2 -mfpmath=sse -msse -msse2 -MD -MQ src/extension-manager.p/exm-detail-view.c.o -MF src/extension-manager.p/exm-detail-view.c.o.d -o src/extension-manager.p/exm-detail-view.c.o -c ../extension-manager-0.5.1/src/exm-detail-view.c
../extension-manager-0.5.1/src/exm-detail-view.c: In function ‘exm_detail_view_finalize’:
../extension-manager-0.5.1/src/exm-detail-view.c:103:20: warning: unused variable ‘self’ [-Wunused-variable]
  103 |     ExmDetailView *self = (ExmDetailView *)object;
      |                    ^~~~
../extension-manager-0.5.1/src/exm-detail-view.c: In function ‘on_get_comments’:
../extension-manager-0.5.1/src/exm-detail-view.c:253:30: warning: cast between incompatible function types from ‘GtkWidget * (*)(ExmComment *)’ {aka ‘struct _GtkWidget * (*)(struct _ExmComment *)’} to ‘GtkWidget * (*)(void *, void *)’ {aka ‘struct _GtkWidget * (*)(void *, void *)’} [-Wcast-function-type]
  253 |                              (GtkListBoxCreateWidgetFunc) comment_factory,
      |                              ^
../extension-manager-0.5.1/src/exm-detail-view.c: In function ‘show_more_comments’:
../extension-manager-0.5.1/src/exm-detail-view.c:269:32: warning: unused parameter ‘button’ [-Wunused-parameter]
  269 | show_more_comments (GtkButton *button,
      |                     ~~~~~~~~~~~^~~~~~
../extension-manager-0.5.1/src/exm-detail-view.c: In function ‘open_link’:
../extension-manager-0.5.1/src/exm-detail-view.c:473:27: warning: unused parameter ‘param’ [-Wunused-parameter]
  473 |            GVariant      *param)
      |            ~~~~~~~~~~~~~~~^~~~~
../extension-manager-0.5.1/src/exm-detail-view.c: In function ‘exm_detail_view_class_init’:
../extension-manager-0.5.1/src/exm-detail-view.c:598:84: error: passing argument 4 of ‘gtk_widget_class_install_action’ from incompatible pointer type [-Wincompatible-pointer-types]
  598 |     gtk_widget_class_install_action (widget_class, "detail.open-extensions", NULL, open_link);
      |                                                                                    ^~~~~~~~~
      |                                                                                    |
      |                                                                                    void (*)(ExmDetailView *, const char *, GVariant *) {aka void (*)(struct _ExmDetailView *, const char *, struct _GVariant *)}
In file included from /usr/include/gtk-4.0/gtk/gtkapplication.h:26,
                 from /usr/include/gtk-4.0/gtk/gtkwindow.h:32,
                 from /usr/include/gtk-4.0/gtk/gtkaboutdialog.h:29,
                 from /usr/include/gtk-4.0/gtk/gtk.h:33,
                 from /usr/include/libadwaita-1/adwaita.h:9,
                 from ../extension-manager-0.5.1/src/exm-detail-view.h:3,
                 from ../extension-manager-0.5.1/src/exm-detail-view.c:21:
/usr/include/gtk-4.0/gtk/gtkwidget.h:955:87: note: expected ‘GtkWidgetActionActivateFunc’ {aka ‘void (*)(struct _GtkWidget *, const char *, struct _GVariant *)’} but argument is of type ‘void (*)(ExmDetailView *, const char *, GVariant *)’ {aka ‘void (*)(struct _ExmDetailView *, const char *, struct _GVariant *)’}
  955 |                                                          GtkWidgetActionActivateFunc  activate);
      |                                                          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
../extension-manager-0.5.1/src/exm-detail-view.c:599:82: error: passing argument 4 of ‘gtk_widget_class_install_action’ from incompatible pointer type [-Wincompatible-pointer-types]
  599 |     gtk_widget_class_install_action (widget_class, "detail.open-homepage", NULL, open_link);
      |                                                                                  ^~~~~~~~~
      |                                                                                  |
      |                                                                                  void (*)(ExmDetailView *, const char *, GVariant *) {aka void (*)(struct _ExmDetailView *, const char *, struct _GVariant *)}
/usr/include/gtk-4.0/gtk/gtkwidget.h:955:87: note: expected ‘GtkWidgetActionActivateFunc’ {aka ‘void (*)(struct _GtkWidget *, const char *, struct _GVariant *)’} but argument is of type ‘void (*)(ExmDetailView *, const char *, GVariant *)’ {aka ‘void (*)(struct _ExmDetailView *, const char *, struct _GVariant *)’}
  955 |                                                          GtkWidgetActionActivateFunc  activate);
      |                                                          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
[73/75] Compiling C object src/extension-manager.p/exm-extension-row.c.o
FAILED: src/extension-manager.p/exm-extension-row.c.o 
ccache cc -Isrc/extension-manager.p -Isrc -I../extension-manager-0.5.1/src -I/usr/include/gtk-4.0 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/freetype2 -I/usr/include/graphene-1.0 -I/usr/lib/graphene-1.0/include -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/sysprof-6 -I/usr/include/fribidi -I/usr/include/libpng16 -I/usr/include/pixman-1 -I/usr/include/libadwaita-1 -I/usr/include/appstream -I/usr/include/gio-unix-2.0 -I/usr/include/json-glib-1.0 -I/usr/include/libsoup-3.0 -I/usr/include/text-engine -I/home/bill88t/git/extension-manager/src/build -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -Werror=implicit-function-declaration -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -fPIE -pthread -mfpmath=sse -msse -msse2 -mfpmath=sse -msse -msse2 -MD -MQ src/extension-manager.p/exm-extension-row.c.o -MF src/extension-manager.p/exm-extension-row.c.o.d -o src/extension-manager.p/exm-extension-row.c.o -c ../extension-manager-0.5.1/src/exm-extension-row.c
../extension-manager-0.5.1/src/exm-extension-row.c: In function ‘exm_extension_row_finalize’:
../extension-manager-0.5.1/src/exm-extension-row.c:63:22: warning: unused variable ‘self’ [-Wunused-variable]
   63 |     ExmExtensionRow *self = (ExmExtensionRow *)object;
      |                      ^~~~
../extension-manager-0.5.1/src/exm-extension-row.c: In function ‘transform_to_state’:
../extension-manager-0.5.1/src/exm-extension-row.c:129:35: warning: unused parameter ‘binding’ [-Wunused-parameter]
  129 | transform_to_state (GBinding     *binding,
      |                     ~~~~~~~~~~~~~~^~~~~~~
../extension-manager-0.5.1/src/exm-extension-row.c:132:35: warning: unused parameter ‘user_data’ [-Wunused-parameter]
  132 |                     gpointer      user_data)
      |                     ~~~~~~~~~~~~~~^~~~~~~~~
In file included from /usr/include/glib-2.0/glib.h:64,
                 from /usr/include/gtk-4.0/gtk/css/gtkcss.h:29,
                 from /usr/include/gtk-4.0/gtk/gtk.h:29,
                 from /usr/include/libadwaita-1/adwaita.h:9,
                 from ../extension-manager-0.5.1/src/exm-extension-row.h:3,
                 from ../extension-manager-0.5.1/src/exm-extension-row.c:1:
../extension-manager-0.5.1/src/exm-extension-row.c: In function ‘on_state_changed’:
/usr/include/glib-2.0/glib/gmessages.h:658:9: error: ‘return’ with no value, in function returning non-void [-Wreturn-mismatch]
  658 |         return; \
      |         ^~~~~~
../extension-manager-0.5.1/src/exm-extension-row.c:237:5: note: in expansion of macro ‘g_return_if_fail’
  237 |     g_return_if_fail (EXM_IS_EXTENSION_ROW (self));
      |     ^~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-extension-row.c:233:1: note: declared here
  233 | on_state_changed (GtkSwitch        *toggle,
      | ^~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-extension-row.c: In function ‘open_prefs’:
../extension-manager-0.5.1/src/exm-extension-row.c:308:30: warning: unused parameter ‘action’ [-Wunused-parameter]
  308 | open_prefs (GSimpleAction   *action,
      |             ~~~~~~~~~~~~~~~~~^~~~~~
../extension-manager-0.5.1/src/exm-extension-row.c:309:30: warning: unused parameter ‘new_value’ [-Wunused-parameter]
  309 |             GVariant        *new_value,
      |             ~~~~~~~~~~~~~~~~~^~~~~~~~~
../extension-manager-0.5.1/src/exm-extension-row.c: In function ‘uninstall’:
../extension-manager-0.5.1/src/exm-extension-row.c:320:29: warning: unused parameter ‘action’ [-Wunused-parameter]
  320 | uninstall (GSimpleAction   *action,
      |            ~~~~~~~~~~~~~~~~~^~~~~~
../extension-manager-0.5.1/src/exm-extension-row.c:321:29: warning: unused parameter ‘new_value’ [-Wunused-parameter]
  321 |            GVariant        *new_value,
      |            ~~~~~~~~~~~~~~~~~^~~~~~~~~
[74/75] Compiling C object src/extension-manager.p/exm-installed-page.c.o
FAILED: src/extension-manager.p/exm-installed-page.c.o 
ccache cc -Isrc/extension-manager.p -Isrc -I../extension-manager-0.5.1/src -I/usr/include/gtk-4.0 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/freetype2 -I/usr/include/graphene-1.0 -I/usr/lib/graphene-1.0/include -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/sysprof-6 -I/usr/include/fribidi -I/usr/include/libpng16 -I/usr/include/pixman-1 -I/usr/include/libadwaita-1 -I/usr/include/appstream -I/usr/include/gio-unix-2.0 -I/usr/include/json-glib-1.0 -I/usr/include/libsoup-3.0 -I/usr/include/text-engine -I/home/bill88t/git/extension-manager/src/build -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -Werror=implicit-function-declaration -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -fPIE -pthread -mfpmath=sse -msse -msse2 -mfpmath=sse -msse -msse2 -MD -MQ src/extension-manager.p/exm-installed-page.c.o -MF src/extension-manager.p/exm-installed-page.c.o.d -o src/extension-manager.p/exm-installed-page.c.o -c ../extension-manager-0.5.1/src/exm-installed-page.c
In file included from /usr/include/glib-2.0/glib.h:64,
                 from /usr/include/gtk-4.0/gtk/css/gtkcss.h:29,
                 from /usr/include/gtk-4.0/gtk/gtk.h:29,
                 from ../extension-manager-0.5.1/src/exm-installed-page.h:3,
                 from ../extension-manager-0.5.1/src/exm-installed-page.c:1:
../extension-manager-0.5.1/src/exm-installed-page.c: In function ‘widget_factory’:
/usr/include/glib-2.0/glib/gmessages.h:658:9: error: ‘return’ with no value, in function returning non-void [-Wreturn-mismatch]
  658 |         return; \
      |         ^~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:110:5: note: in expansion of macro ‘g_return_if_fail’
  110 |     g_return_if_fail (EXM_IS_EXTENSION (extension));
      |     ^~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:105:1: note: declared here
  105 | widget_factory (ExmExtension     *extension,
      | ^~~~~~~~~~~~~~
/usr/include/glib-2.0/glib/gmessages.h:658:9: error: ‘return’ with no value, in function returning non-void [-Wreturn-mismatch]
  658 |         return; \
      |         ^~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:111:5: note: in expansion of macro ‘g_return_if_fail’
  111 |     g_return_if_fail (EXM_IS_INSTALLED_PAGE (self));
      |     ^~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:105:1: note: declared here
  105 | widget_factory (ExmExtension     *extension,
      | ^~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c: In function ‘compare_enabled’:
/usr/include/glib-2.0/glib/gmessages.h:658:9: error: ‘return’ with no value, in function returning non-void [-Wreturn-mismatch]
  658 |         return; \
      |         ^~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:120:5: note: in expansion of macro ‘g_return_if_fail’
  120 |     g_return_if_fail (EXM_IS_EXTENSION (this));
      |     ^~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:118:1: note: declared here
  118 | compare_enabled (ExmExtension *this, ExmExtension *other)
      | ^~~~~~~~~~~~~~~
/usr/include/glib-2.0/glib/gmessages.h:658:9: error: ‘return’ with no value, in function returning non-void [-Wreturn-mismatch]
  658 |         return; \
      |         ^~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:121:5: note: in expansion of macro ‘g_return_if_fail’
  121 |     g_return_if_fail (EXM_IS_EXTENSION (other));
      |     ^~~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c:118:1: note: declared here
  118 | compare_enabled (ExmExtension *this, ExmExtension *other)
      | ^~~~~~~~~~~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c: In function ‘bind_list_box’:
../extension-manager-0.5.1/src/exm-installed-page.c:163:49: warning: cast between incompatible function types from ‘int (*)(ExmExtension *, ExmExtension *)’ {aka ‘int (*)(struct _ExmExtension *, struct _ExmExtension *)’} to ‘gint (*)(const void *, const void *, void *)’ {aka ‘int (*)(const void *, const void *, void *)’} [-Wcast-function-type]
  163 |         enabled_sorter = gtk_custom_sorter_new ((GCompareDataFunc) compare_enabled, NULL, NULL);
      |                                                 ^
../extension-manager-0.5.1/src/exm-installed-page.c: In function ‘on_updates_available’:
../extension-manager-0.5.1/src/exm-installed-page.c:190:41: warning: unused parameter ‘manager’ [-Wunused-parameter]
  190 | on_updates_available (ExmManager       *manager,
      |                       ~~~~~~~~~~~~~~~~~~^~~~~~~
../extension-manager-0.5.1/src/exm-installed-page.c: In function ‘compare_enabled’:
../extension-manager-0.5.1/src/exm-installed-page.c:138:1: warning: control reaches end of non-void function [-Wreturn-type]
  138 | }
      | ^
ninja: build stopped: subcommand failed.
==> ERROR: A failure occurred in build().
    Aborting...

This lommg error list might seem scary, but don't worry I already wasted my time on it. There are only 4 types of errors, missing C handlers, one cast of interger into pointer, NULL returns and open_link arguments.

The cast just needs a & and it's fixed. I don't know how can I make this patch work across old and new versions, so.. just letting you know I guess..

The return errors are because of the use of g_return_if_fail instead of g_return_val_if_fail. I created some patches solving this downsteam, I will PR them in a bit.

The C handler and open_link errors I don't know about. I know C (and embedded C at that) but I am not very aquainted with all the gnome stuff.

System Information:

Additional context The AUR repo, including the PKGBUILD are available in ssh://aur@aur.archlinux.org/extension-manager.git if you feel like viewing them.

oscfdezdz commented 6 months ago

Thanks for the heads up! Does the following patch together with your patches fix it?

From 1fe44e88a343d347f9703a4ee1086bbc0ebb50b7 Mon Sep 17 00:00:00 2001
From: oscfdezdz <42654671+oscfdezdz@users.noreply.github.com>
Date: Fri, 17 May 2024 01:20:57 +0200
Subject: [PATCH] Fix some warnings

---
 src/exm-detail-view.c | 4 ++--
 src/exm-window.c      | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/exm-detail-view.c b/src/exm-detail-view.c
index f91bb1a..072513e 100644
--- a/src/exm-detail-view.c
+++ b/src/exm-detail-view.c
@@ -595,8 +595,8 @@ exm_detail_view_class_init (ExmDetailViewClass *klass)
     gtk_widget_class_bind_template_callback (widget_class, breakpoint_unapply_cb);
     gtk_widget_class_bind_template_callback (widget_class, screenshot_view_cb);

-    gtk_widget_class_install_action (widget_class, "detail.open-extensions", NULL, open_link);
-    gtk_widget_class_install_action (widget_class, "detail.open-homepage", NULL, open_link);
+    gtk_widget_class_install_action (widget_class, "detail.open-extensions", NULL, (GtkWidgetActionActivateFunc) open_link);
+    gtk_widget_class_install_action (widget_class, "detail.open-homepage", NULL, (GtkWidgetActionActivateFunc) open_link);
 }

 static void
diff --git a/src/exm-window.c b/src/exm-window.c
index 472c199..ade4ffd 100644
--- a/src/exm-window.c
+++ b/src/exm-window.c
@@ -404,7 +404,7 @@ exm_window_init (ExmWindow *self)
     }

     self->manager = exm_manager_new ();
-    g_signal_connect (self->manager, "error-occurred", on_error, self);
+    g_signal_connect (self->manager, "error-occurred", G_CALLBACK (on_error), self);

     g_object_set (self->installed_page, "manager", self->manager, NULL);
     g_object_set (self->browse_page, "manager", self->manager, NULL);
-- 
2.45.0
bill88t commented 6 months ago

Sorry for the late response. Indeed with this patch, it built and ran successfully. I opened it and tested it briefly, nothing seems amiss.

oscfdezdz commented 6 months ago

Great, I'll open a PR with it and review yours.

oscfdezdz commented 6 months ago

Everything seems to work correctly in the Flatpak stable runtime as well. I'll close the issue if there are no more problems. Again, thanks for the report!