[gnome-shell/wip/smcv/gir-api-versions] Specify API versions for all non-bundled GIR APIs, except GLib




commit 8612cf2c0114fc0ab40199131640305679c34fb2
Author: Simon McVittie <smcv debian org>
Date:   Mon Apr 4 11:26:43 2022 +0100

    Specify API versions for all non-bundled GIR APIs, except GLib
    
    If one of these libraries breaks its GIR API in future, then upgrading
    packages unrelated to gnome-shell might pull in the newer version,
    causing gnome-shell to crash when it gets a newer GIR API that is
    incompatible with its expectations. For example, this seems to be
    happening in Debian testing at the moment, when GNOME Shell 41.4
    imports GWeather and can get version 4.0 instead of the version 3.0 that
    it expected.
    
    Adding explicit API versions at the time the newer version is released
    is too late, because that will still let the newer version of the GIR API
    break pre-existing GNOME Shell packages. Prevent similar crashes in
    future by making the desired versions explicit.
    
    This is done for all third-party libraries except GLib, similar to the
    common practice in Python code; if GLib breaks API, then that will be
    a disruptive change to the whole GLib/GObject ecosystem, regardless.
    
    Signed-off-by: Simon McVittie <smcv debian org>

 js/dbusServices/extensions/main.js              |  1 +
 js/dbusServices/screencast/screencastService.js |  1 +
 js/ui/environment.js                            | 14 ++++++++++++++
 subprojects/extensions-app/js/main.js           |  1 +
 4 files changed, 17 insertions(+)
---
diff --git a/js/dbusServices/extensions/main.js b/js/dbusServices/extensions/main.js
index b13e52e3cf..306fe36f5e 100644
--- a/js/dbusServices/extensions/main.js
+++ b/js/dbusServices/extensions/main.js
@@ -1,5 +1,6 @@
 /* exported main */
 
+imports.gi.versions.Adw = '1';
 imports.gi.versions.Gdk = '4.0';
 imports.gi.versions.Gtk = '4.0';
 
diff --git a/js/dbusServices/screencast/screencastService.js b/js/dbusServices/screencast/screencastService.js
index e91d0b60c0..1fdf5c766e 100644
--- a/js/dbusServices/screencast/screencastService.js
+++ b/js/dbusServices/screencast/screencastService.js
@@ -1,6 +1,7 @@
 // -*- mode: js; js-indent-level: 4; indent-tabs-mode: nil -*-
 /* exported ScreencastService */
 
+imports.gi.versions.Gst = '1.0';
 imports.gi.versions.Gtk = '4.0';
 
 const { Gio, GLib, Gst, Gtk } = imports.gi;
diff --git a/js/ui/environment.js b/js/ui/environment.js
index 7bd848f1ab..92954969f3 100644
--- a/js/ui/environment.js
+++ b/js/ui/environment.js
@@ -3,16 +3,30 @@
 
 const Config = imports.misc.config;
 
+imports.gi.versions.Atspi = '2.0';
+imports.gi.versions.Atk = '1.0';
 imports.gi.versions.Clutter = Config.LIBMUTTER_API_VERSION;
+imports.gi.versions.Cogl = Config.LIBMUTTER_API_VERSION;
+imports.gi.versions.Gcr = '3';
+imports.gi.versions.Gdm = '1.0';
 imports.gi.versions.Gio = '2.0';
 imports.gi.versions.GdkPixbuf = '2.0';
 imports.gi.versions.GnomeBluetooth = '3.0';
 imports.gi.versions.GnomeDesktop = '3.0';
+imports.gi.versions.Graphene = '1.0';
 imports.gi.versions.Gtk = '3.0';
 imports.gi.versions.GWeather = '4.0';
+imports.gi.versions.IBus = '1.0';
+imports.gi.versions.Malcontent = '0';
+imports.gi.versions.Meta = Config.LIBMUTTER_API_VERSION;
+imports.gi.versions.NM = '1.0';
+imports.gi.versions.NMA = '1.0';
+imports.gi.versions.Pango = '1.0';
+imports.gi.versions.Polkit = '1.0';
 imports.gi.versions.Soup = '3.0';
 imports.gi.versions.TelepathyGLib = '0.12';
 imports.gi.versions.TelepathyLogger = '0.2';
+imports.gi.versions.UPowerGlib = '1.0';
 
 try {
     if (Config.HAVE_SOUP2)
diff --git a/subprojects/extensions-app/js/main.js b/subprojects/extensions-app/js/main.js
index 361477ddd2..56e10769fe 100644
--- a/subprojects/extensions-app/js/main.js
+++ b/subprojects/extensions-app/js/main.js
@@ -1,4 +1,5 @@
 /* exported main */
+imports.gi.versions.Adw = '1';
 imports.gi.versions.Gtk = '4.0';
 
 const Gettext = imports.gettext;


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