diff options
author | Colin Walters <walters@verbum.org> | 2013-11-22 15:19:21 -0500 |
---|---|---|
committer | Colin Walters <walters@verbum.org> | 2013-11-22 15:22:26 -0500 |
commit | cc62c15832ee4e2eeff336de4dff76d7f718f432 (patch) | |
tree | 527b1e33861278d9b17d658ec63111789cb5a20d | |
parent | 28a6e3cbeb875eeaca3dd2d186db56201f3f80b0 (diff) | |
download | gdm-cc62c15832ee4e2eeff336de4dff76d7f718f432.tar.gz |
Don't enable debugging by default if the version is "unstable"
I'm open to negotiation for this, but basically...gdm is at
present second only to the kernel in the sheer amount of spew
it emits - when the version is "unstable". But it always will
be for Continuous.
Nowadays of course, I think Continuous helps act as a testing system
for GDM that obviates a lot of the need for lots of debugging info for
gdm during "unstable" cycles.
I suspect that we will over time add other informational messages at
strategic points by default; this patch is not the end of the story.
It's a new beginning.
https://bugzilla.gnome.org/show_bug.cgi?id=715037
-rw-r--r-- | common/gdm-common.c | 21 | ||||
-rw-r--r-- | common/gdm-common.h | 2 | ||||
-rw-r--r-- | daemon/main.c | 8 | ||||
-rw-r--r-- | daemon/session-worker-main.c | 8 | ||||
-rw-r--r-- | daemon/simple-slave-main.c | 8 | ||||
-rw-r--r-- | daemon/xdmcp-chooser-slave-main.c | 8 |
6 files changed, 4 insertions, 51 deletions
diff --git a/common/gdm-common.c b/common/gdm-common.c index ae6129c2..99f9857d 100644 --- a/common/gdm-common.c +++ b/common/gdm-common.c @@ -45,27 +45,6 @@ gdm_make_temp_dir (char *template) } gboolean -gdm_is_version_unstable (void) -{ - char **versions; - gboolean unstable; - - unstable = FALSE; - - versions = g_strsplit (VERSION, ".", 3); - if (versions && versions [0] && versions [1]) { - int major; - major = atoi (versions [1]); - if ((major % 2) != 0) { - unstable = TRUE; - } - } - g_strfreev (versions); - - return unstable; -} - -gboolean gdm_clear_close_on_exec_flag (int fd) { int flags; diff --git a/common/gdm-common.h b/common/gdm-common.h index f9033aa5..be4d1032 100644 --- a/common/gdm-common.h +++ b/common/gdm-common.h @@ -38,8 +38,6 @@ G_BEGIN_DECLS -gboolean gdm_is_version_unstable (void); - int gdm_wait_on_pid (int pid); int gdm_wait_on_and_disown_pid (int pid, int timeout); diff --git a/daemon/main.c b/daemon/main.c index 7bd342d0..b95802f6 100644 --- a/daemon/main.c +++ b/daemon/main.c @@ -300,13 +300,7 @@ on_sigusr1_cb (gpointer user_data) static gboolean is_debug_set (void) { - gboolean debug = FALSE; - - /* enable debugging for unstable builds */ - if (gdm_is_version_unstable ()) { - return TRUE; - } - + gboolean debug; gdm_settings_direct_get_boolean (GDM_KEY_DEBUG, &debug); return debug; } diff --git a/daemon/session-worker-main.c b/daemon/session-worker-main.c index 40cd08be..ede60c59 100644 --- a/daemon/session-worker-main.c +++ b/daemon/session-worker-main.c @@ -68,13 +68,7 @@ on_sigusr1_cb (gpointer user_data) static gboolean is_debug_set (void) { - gboolean debug = FALSE; - - /* enable debugging for unstable builds */ - if (gdm_is_version_unstable ()) { - return TRUE; - } - + gboolean debug; gdm_settings_direct_get_boolean (GDM_KEY_DEBUG, &debug); return debug; } diff --git a/daemon/simple-slave-main.c b/daemon/simple-slave-main.c index 52fc9b08..5c6bfa20 100644 --- a/daemon/simple-slave-main.c +++ b/daemon/simple-slave-main.c @@ -101,13 +101,7 @@ on_slave_stopped (GdmSlave *slave, static gboolean is_debug_set (void) { - gboolean debug = FALSE; - - /* enable debugging for unstable builds */ - if (gdm_is_version_unstable ()) { - return TRUE; - } - + gboolean debug; gdm_settings_direct_get_boolean (GDM_KEY_DEBUG, &debug); return debug; } diff --git a/daemon/xdmcp-chooser-slave-main.c b/daemon/xdmcp-chooser-slave-main.c index 0c122dc4..302466b0 100644 --- a/daemon/xdmcp-chooser-slave-main.c +++ b/daemon/xdmcp-chooser-slave-main.c @@ -100,13 +100,7 @@ on_sigusr2_cb (gpointer user_data) static gboolean is_debug_set (void) { - gboolean debug = FALSE; - - /* enable debugging for unstable builds */ - if (gdm_is_version_unstable ()) { - return TRUE; - } - + gboolean debug; gdm_settings_direct_get_boolean (GDM_KEY_DEBUG, &debug); return debug; } |