Merge lp:~darkxst/ubuntu/saucy/gnome-settings-daemon/lp1227537 into lp:~ubuntu-desktop/gnome-settings-daemon/ubuntu

Proposed by Tim Lunn
Status: Rejected
Rejected by: Sebastien Bacher
Proposed branch: lp:~darkxst/ubuntu/saucy/gnome-settings-daemon/lp1227537
Merge into: lp:~ubuntu-desktop/gnome-settings-daemon/ubuntu
Diff against target: 122 lines (+102/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/git_hardcode_deprecated_gtk_settings.patch (+94/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~darkxst/ubuntu/saucy/gnome-settings-daemon/lp1227537
Reviewer Review Type Date Requested Status
Sebastien Bacher Disapprove
Review via email: mp+186490@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Sorry but I'm reverting this one:
- those settings are still there and working in saucy
- they shouldn't have been dropped without discussion
- the fix is to workaround issues with a ppa, which can be solved by simply adding those gsettings key to the schemas in the ppa version

review: Disapprove

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2013-09-15 22:43:51 +0000
+++ debian/changelog 2013-09-19 09:28:18 +0000
@@ -1,3 +1,10 @@
1gnome-settings-daemon (3.8.5-0ubuntu4) UNRELEASED; urgency=low
2
3 * debian/patches/git_hardcode_deprecated_gtk_settings.patch: git patch
4 to make things work with gsettings schemas 3.10 (LP: #1227537)
5
6 -- Tim Lunn <tim@feathertop.org> Thu, 19 Sep 2013 19:20:34 +1000
7
1gnome-settings-daemon (3.8.5-0ubuntu3) saucy; urgency=low8gnome-settings-daemon (3.8.5-0ubuntu3) saucy; urgency=low
29
3 * debian/patches/53_sync_input_sources_to_accountsservice.patch:10 * debian/patches/53_sync_input_sources_to_accountsservice.patch:
411
=== added file 'debian/patches/git_hardcode_deprecated_gtk_settings.patch'
--- debian/patches/git_hardcode_deprecated_gtk_settings.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/git_hardcode_deprecated_gtk_settings.patch 2013-09-19 09:28:18 +0000
@@ -0,0 +1,94 @@
1From 815bae0575a84d93d65d72f0c71d9b247686bb8d Mon Sep 17 00:00:00 2001
2From: William Jon McCann <william.jon.mccann@gmail.com>
3Date: Fri, 12 Jul 2013 15:00:27 +0000
4Subject: xsettings: hardcode deprecated settings instead of removing them
5
6This way we don't break old apps like those that use GTK2.
7---
8diff --git a/plugins/xsettings/gsd-xsettings-manager.c b/plugins/xsettings/gsd-xsettings-manager.c
9index d4e0236..81e94c1 100644
10--- a/plugins/xsettings/gsd-xsettings-manager.c
11+++ b/plugins/xsettings/gsd-xsettings-manager.c
12@@ -231,6 +231,14 @@ struct _TranslationEntry {
13 TranslationFunc translate;
14 };
15
16+typedef struct _FixedEntry FixedEntry;
17+typedef void (* FixedFunc) (GnomeXSettingsManager *manager,
18+ FixedEntry *fixed);
19+struct _FixedEntry {
20+ const char *xsetting_name;
21+ FixedFunc func;
22+};
23+
24 struct GnomeXSettingsManagerPrivate
25 {
26 guint start_idle_id;
27@@ -308,6 +316,36 @@ translate_string_string (GnomeXSettingsManager *manager,
28 }
29 }
30
31+static void
32+fixed_false_int (GnomeXSettingsManager *manager,
33+ FixedEntry *fixed)
34+{
35+ int i;
36+
37+ for (i = 0; manager->priv->managers [i]; i++) {
38+ xsettings_manager_set_int (manager->priv->managers [i], fixed->xsetting_name, FALSE);
39+ }
40+}
41+
42+static void
43+fixed_true_int (GnomeXSettingsManager *manager,
44+ FixedEntry *fixed)
45+{
46+ int i;
47+
48+ for (i = 0; manager->priv->managers [i]; i++) {
49+ xsettings_manager_set_int (manager->priv->managers [i], fixed->xsetting_name, TRUE);
50+ }
51+}
52+
53+static FixedEntry fixed_entries [] = {
54+ { "Gtk/MenuImages", fixed_false_int },
55+ { "Gtk/ButtonImages", fixed_false_int },
56+ { "Gtk/ShowInputMethodMenu", fixed_false_int },
57+ { "Gtk/ShowUnicodeMenu", fixed_false_int },
58+ { "Gtk/AutoMnemonics", fixed_true_int },
59+};
60+
61 static TranslationEntry translations [] = {
62 { "org.gnome.settings-daemon.peripherals.mouse", "double-click", "Net/DoubleClickTime", translate_int_int },
63 { "org.gnome.settings-daemon.peripherals.mouse", "drag-threshold", "Net/DndDragThreshold", translate_int_int },
64@@ -329,15 +367,10 @@ static TranslationEntry translations [] = {
65 { "org.gnome.desktop.interface", "gtk-im-status-style", "Gtk/IMStatusStyle", translate_string_string },
66 { "org.gnome.desktop.interface", "gtk-im-module", "Gtk/IMModule", translate_string_string },
67 { "org.gnome.desktop.interface", "icon-theme", "Net/IconThemeName", translate_string_string },
68- { "org.gnome.desktop.interface", "menus-have-icons", "Gtk/MenuImages", translate_bool_int },
69- { "org.gnome.desktop.interface", "buttons-have-icons", "Gtk/ButtonImages", translate_bool_int },
70 { "org.gnome.desktop.interface", "menubar-accel", "Gtk/MenuBarAccel", translate_string_string },
71 { "org.gnome.desktop.interface", "enable-animations", "Gtk/EnableAnimations", translate_bool_int },
72 { "org.gnome.desktop.interface", "cursor-theme", "Gtk/CursorThemeName", translate_string_string },
73 { "org.gnome.desktop.interface", "cursor-size", "Gtk/CursorThemeSize", translate_int_int },
74- { "org.gnome.desktop.interface", "show-input-method-menu", "Gtk/ShowInputMethodMenu", translate_bool_int },
75- { "org.gnome.desktop.interface", "show-unicode-menu", "Gtk/ShowUnicodeMenu", translate_bool_int },
76- { "org.gnome.desktop.interface", "automatic-mnemonics", "Gtk/AutoMnemonics", translate_bool_int },
77
78 { "org.gnome.desktop.sound", "theme-name", "Net/SoundThemeName", translate_string_string },
79 { "org.gnome.desktop.sound", "event-sounds", "Net/EnableEventSounds" , translate_bool_int },
80@@ -878,6 +911,11 @@ gnome_xsettings_manager_start (GnomeXSettingsManager *manager,
81 g_hash_table_insert (manager->priv->settings,
82 PRIVACY_SETTINGS_SCHEMA, g_settings_new (PRIVACY_SETTINGS_SCHEMA));
83
84+ for (i = 0; i < G_N_ELEMENTS (fixed_entries); i++) {
85+ FixedEntry *fixed = &fixed_entries[i];
86+ (* fixed->func) (manager, fixed);
87+ }
88+
89 for (i = 0; i < G_N_ELEMENTS (translations); i++) {
90 GVariant *val;
91 GSettings *settings;
92--
93cgit v0.9.2
94
095
=== modified file 'debian/patches/series'
--- debian/patches/series 2013-09-09 01:52:56 +0000
+++ debian/patches/series 2013-09-19 09:28:18 +0000
@@ -25,3 +25,4 @@
25git_revert_hardcoded_input_methods.patch25git_revert_hardcoded_input_methods.patch
26ubuntu-fix-desktop-file.patch26ubuntu-fix-desktop-file.patch
27ubuntu-lid-close-suspend.patch27ubuntu-lid-close-suspend.patch
28git_hardcode_deprecated_gtk_settings.patch

Subscribers

People subscribed via source and target branches

to all changes: