Merge lp:~lightdm-gtk-greeter-team/lightdm-gtk-greeter/show_indicators_proposal into lp:~lightdm-gtk-greeter-team/lightdm-gtk-greeter/trunk

Proposed by Sean Davis
Status: Merged
Merged at revision: 242
Proposed branch: lp:~lightdm-gtk-greeter-team/lightdm-gtk-greeter/show_indicators_proposal
Merge into: lp:~lightdm-gtk-greeter-team/lightdm-gtk-greeter/trunk
Diff against target: 33 lines (+13/-2)
1 file modified
src/lightdm-gtk-greeter.c (+13/-2)
To merge this branch: bzr merge lp:~lightdm-gtk-greeter-team/lightdm-gtk-greeter/show_indicators_proposal
Reviewer Review Type Date Requested Status
LightDM Gtk+ Greeter Development Team Pending
Review via email: mp+208580@code.launchpad.net

Description of the change

Deprecate "show-indicators", replace with "indicators" to improve upgrade path of configuration files pre-1.8 series.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/lightdm-gtk-greeter.c'
2--- src/lightdm-gtk-greeter.c 2014-02-26 01:26:13 +0000
3+++ src/lightdm-gtk-greeter.c 2014-02-27 11:54:33 +0000
4@@ -379,6 +379,7 @@
5 GHashTableIter iter;
6 gpointer iter_value;
7 gboolean inited = FALSE;
8+ gboolean fallback = FALSE;
9
10 #ifdef START_INDICATOR_SERVICES
11 GError *error = NULL;
12@@ -386,9 +387,19 @@
13 gchar *INDICATORS_CMD[] = {"init", "--user", "--startup-event", "indicator-services-start", NULL};
14 #endif
15
16- if (g_key_file_has_key (config, "greeter", "show-indicators", NULL))
17- {
18+ if (g_key_file_has_key (config, "greeter", "indicators", NULL))
19+ { /* no option = default list, empty value = empty list */
20+ names = g_key_file_get_string_list (config, "greeter", "indicators", &length, NULL);
21+ }
22+ else if (g_key_file_has_key (config, "greeter", "show-indicators", NULL))
23+ { /* fallback mode: no option = empty value = default list */
24 names = g_key_file_get_string_list (config, "greeter", "show-indicators", &length, NULL);
25+ if (length == 0)
26+ fallback = TRUE;
27+ }
28+
29+ if (names && !fallback)
30+ {
31 builtin_items = g_hash_table_new (g_str_hash, g_str_equal);
32
33 g_hash_table_insert (builtin_items, "~power", power_menuitem);

Subscribers

People subscribed via source and target branches