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

Subscribers

People subscribed via source and target branches

to all changes: