Accepting request 110460 from home:vuntz:branches:GNOME:Factory
Rebase patch + cleanup OBS-URL: https://build.opensuse.org/request/show/110460 OBS-URL: https://build.opensuse.org/package/show/GNOME:Factory/gdm?expand=0&rev=205
This commit is contained in:
parent
6854176c28
commit
53a5ce2532
@ -426,7 +426,7 @@ Index: gdm-3.2.0/gui/simple-greeter/gdm-greeter-login-window.ui
|
||||
- <property name="visible">True</property>
|
||||
- <property name="can_focus">True</property>
|
||||
- <property name="can_default">True</property>
|
||||
- <property name="has_default">True</property>
|
||||
- <property name="has_default">False</property>
|
||||
- <property name="receives_default">False</property>
|
||||
- </object>
|
||||
- <packing>
|
||||
@ -501,7 +501,7 @@ Index: gdm-3.2.0/gui/simple-greeter/gdm-greeter-login-window.ui
|
||||
+ <property name="visible">True</property>
|
||||
+ <property name="can_focus">True</property>
|
||||
+ <property name="can_default">True</property>
|
||||
+ <property name="has_default">True</property>
|
||||
+ <property name="has_default">False</property>
|
||||
+ <property name="receives_default">False</property>
|
||||
+ <property name="use_action_appearance">False</property>
|
||||
+ </object>
|
||||
@ -745,7 +745,7 @@ Index: gdm-3.2.0/gui/simple-greeter/gdm-greeter-login-window.ui.new
|
||||
+ <property name="visible">True</property>
|
||||
+ <property name="can_focus">True</property>
|
||||
+ <property name="can_default">True</property>
|
||||
+ <property name="has_default">True</property>
|
||||
+ <property name="has_default">False</property>
|
||||
+ <property name="receives_default">False</property>
|
||||
+ <property name="use_action_appearance">False</property>
|
||||
+ </object>
|
||||
|
@ -1,4 +1,4 @@
|
||||
From b865a1ec2965d26dea2db9de085d4d09c3830434 Mon Sep 17 00:00:00 2001
|
||||
From e6ed6f48c35a6c736a5cde2dcfb6c10941e07809 Mon Sep 17 00:00:00 2001
|
||||
From: Ray Strode <rstrode@redhat.com>
|
||||
Date: Fri, 27 Nov 2009 18:27:53 -0500
|
||||
Subject: [PATCH 1/2] Save root window to pixmap at _XROOTPMAP_ID
|
||||
@ -11,104 +11,14 @@ will give us a nice fade transition when g-s-d starts
|
||||
daemon/gdm-slave.h | 1 +
|
||||
3 files changed, 81 insertions(+), 0 deletions(-)
|
||||
|
||||
Index: gdm-3.3.92.1/daemon/gdm-simple-slave.c
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-simple-slave.c
|
||||
+++ gdm-3.3.92.1/daemon/gdm-simple-slave.c
|
||||
@@ -90,6 +90,7 @@ struct GdmSimpleSlavePrivate
|
||||
|
||||
guint start_session_when_ready : 1;
|
||||
guint waiting_to_start_session : 1;
|
||||
+ guint plymouth_is_running : 1;
|
||||
#ifdef HAVE_LOGINDEVPERM
|
||||
gboolean use_logindevperm;
|
||||
#endif
|
||||
@@ -97,6 +98,7 @@ struct GdmSimpleSlavePrivate
|
||||
|
||||
enum {
|
||||
PROP_0,
|
||||
+ FORCE_ACTIVE_VT
|
||||
};
|
||||
|
||||
static void gdm_simple_slave_class_init (GdmSimpleSlaveClass *klass);
|
||||
@@ -1204,6 +1206,72 @@ on_start_session_later (GdmGreeterServer
|
||||
slave->priv->start_session_when_ready = FALSE;
|
||||
}
|
||||
|
||||
+static gboolean
|
||||
+plymouth_is_running (void)
|
||||
+{
|
||||
+ int status;
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth --ping",
|
||||
+ NULL, NULL, &status, &error);
|
||||
+ if (! res) {
|
||||
+ g_debug ("Could not ping plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ return FALSE;
|
||||
+ }
|
||||
+
|
||||
+ return WIFEXITED (status) && WEXITSTATUS (status) == 0;
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
+plymouth_prepare_for_transition (GdmSimpleSlave *slave)
|
||||
+{
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth deactivate",
|
||||
+ NULL, NULL, NULL, &error);
|
||||
+ if (! res) {
|
||||
+ g_warning ("Could not deactivate plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ }
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
+plymouth_quit_with_transition (GdmSimpleSlave *slave)
|
||||
+{
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth quit --retain-splash",
|
||||
+ NULL, NULL, NULL, &error);
|
||||
+ if (! res) {
|
||||
+ g_warning ("Could not quit plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ }
|
||||
+ slave->priv->plymouth_is_running = FALSE;
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
+plymouth_quit_without_transition (GdmSimpleSlave *slave)
|
||||
+{
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth quit",
|
||||
+ NULL, NULL, NULL, &error);
|
||||
+ if (! res) {
|
||||
+ g_warning ("Could not quit plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ }
|
||||
+ slave->priv->plymouth_is_running = FALSE;
|
||||
+}
|
||||
+
|
||||
static void
|
||||
setup_server (GdmSimpleSlave *slave)
|
||||
{
|
||||
@@ -1215,6 +1283,19 @@ setup_server (GdmSimpleSlave *slave)
|
||||
diff --git a/daemon/gdm-simple-slave.c b/daemon/gdm-simple-slave.c
|
||||
index 9d1347a..fc8649c 100644
|
||||
--- a/daemon/gdm-simple-slave.c
|
||||
+++ b/daemon/gdm-simple-slave.c
|
||||
@@ -1215,6 +1215,14 @@ setup_server (GdmSimpleSlave *slave)
|
||||
|
||||
/* Set the initial keyboard layout to something reasonable */
|
||||
gdm_slave_set_initial_keyboard_layout (GDM_SLAVE (slave));
|
||||
+
|
||||
+ /* The root window has a background that may be useful
|
||||
+ * to cross fade or transition from when setting the
|
||||
+ * login screen background. We read it here, and stuff
|
||||
@ -117,99 +27,13 @@ Index: gdm-3.3.92.1/daemon/gdm-simple-slave.c
|
||||
+ */
|
||||
+ gdm_slave_save_root_windows (GDM_SLAVE (slave));
|
||||
+
|
||||
+ /* Plymouth is waiting for the go-ahead to exit */
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_with_transition (slave);
|
||||
+ }
|
||||
}
|
||||
|
||||
static void
|
||||
@@ -1418,6 +1499,10 @@ on_server_exited (GdmServer *server
|
||||
g_debug ("GdmSimpleSlave: server exited with code %d\n", exit_code);
|
||||
|
||||
gdm_slave_stopped (GDM_SLAVE (slave));
|
||||
+
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_without_transition (slave);
|
||||
+ }
|
||||
}
|
||||
|
||||
static void
|
||||
@@ -1430,6 +1515,10 @@ on_server_died (GdmServer *server,
|
||||
g_strsignal (signal_number));
|
||||
|
||||
gdm_slave_stopped (GDM_SLAVE (slave));
|
||||
+
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_without_transition (slave);
|
||||
+ }
|
||||
}
|
||||
|
||||
static gboolean
|
||||
@@ -1439,12 +1528,14 @@ gdm_simple_slave_run (GdmSimpleSlave *sl
|
||||
char *auth_file;
|
||||
char *seat_id;
|
||||
gboolean display_is_local;
|
||||
+ gboolean force_active_vt;
|
||||
|
||||
g_object_get (slave,
|
||||
"display-is-local", &display_is_local,
|
||||
"display-name", &display_name,
|
||||
"display-seat-id", &seat_id,
|
||||
"display-x11-authority-file", &auth_file,
|
||||
+ "force-active-vt", &force_active_vt,
|
||||
NULL);
|
||||
|
||||
/* if this is local display start a server if one doesn't
|
||||
@@ -1476,7 +1567,17 @@ gdm_simple_slave_run (GdmSimpleSlave *sl
|
||||
G_CALLBACK (on_server_ready),
|
||||
slave);
|
||||
|
||||
- res = gdm_server_start (slave->priv->server);
|
||||
+ slave->priv->plymouth_is_running = plymouth_is_running ();
|
||||
+
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_prepare_for_transition (slave);
|
||||
+ res = gdm_server_start_on_active_vt (slave->priv->server);
|
||||
+ } else {
|
||||
+ if (force_active_vt)
|
||||
+ res = gdm_server_start_on_active_vt (slave->priv->server);
|
||||
+ else
|
||||
+ res = gdm_server_start (slave->priv->server);
|
||||
+ }
|
||||
if (! res) {
|
||||
g_warning (_("Could not start the X "
|
||||
"server (your graphical environment) "
|
||||
@@ -1486,6 +1587,9 @@ gdm_simple_slave_run (GdmSimpleSlave *sl
|
||||
"In the meantime this display will be "
|
||||
"disabled. Please restart GDM when "
|
||||
"the problem is corrected."));
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_without_transition (slave);
|
||||
+ }
|
||||
exit (1);
|
||||
}
|
||||
|
||||
@@ -1642,12 +1746,14 @@ gdm_simple_slave_finalize (GObject *obje
|
||||
}
|
||||
|
||||
GdmSlave *
|
||||
-gdm_simple_slave_new (const char *id)
|
||||
+gdm_simple_slave_new (const char *id,
|
||||
+ gboolean force_active_vt)
|
||||
{
|
||||
GObject *object;
|
||||
|
||||
object = g_object_new (GDM_TYPE_SIMPLE_SLAVE,
|
||||
"display-id", id,
|
||||
+ "force-active-vt", force_active_vt,
|
||||
NULL);
|
||||
|
||||
return GDM_SLAVE (object);
|
||||
Index: gdm-3.3.92.1/daemon/gdm-slave.c
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-slave.c
|
||||
+++ gdm-3.3.92.1/daemon/gdm-slave.c
|
||||
diff --git a/daemon/gdm-slave.c b/daemon/gdm-slave.c
|
||||
index 371b9bd..694dcde 100644
|
||||
--- a/daemon/gdm-slave.c
|
||||
+++ b/daemon/gdm-slave.c
|
||||
@@ -43,6 +43,7 @@
|
||||
#include <dbus/dbus-glib-lowlevel.h>
|
||||
|
||||
@ -218,24 +42,8 @@ Index: gdm-3.3.92.1/daemon/gdm-slave.c
|
||||
#include <X11/cursorfont.h> /* for watch cursor */
|
||||
#include <X11/extensions/Xrandr.h>
|
||||
#include <X11/Xatom.h>
|
||||
@@ -96,6 +97,7 @@ struct GdmSlavePrivate
|
||||
char *display_hostname;
|
||||
gboolean display_is_local;
|
||||
gboolean display_is_parented;
|
||||
+ gboolean force_active_vt;
|
||||
char *display_seat_id;
|
||||
char *display_x11_authority_file;
|
||||
char *parent_display_name;
|
||||
@@ -115,6 +117,7 @@ enum {
|
||||
PROP_DISPLAY_NUMBER,
|
||||
PROP_DISPLAY_HOSTNAME,
|
||||
PROP_DISPLAY_IS_LOCAL,
|
||||
+ PROP_FORCE_ACTIVE_VT,
|
||||
PROP_DISPLAY_SEAT_ID,
|
||||
PROP_DISPLAY_X11_AUTHORITY_FILE
|
||||
};
|
||||
@@ -488,6 +491,77 @@ gdm_slave_set_initial_cursor_position (G
|
||||
}
|
||||
@@ -364,6 +365,77 @@ gdm_slave_run_script (GdmSlave *slave,
|
||||
return ret;
|
||||
}
|
||||
|
||||
+static void
|
||||
@ -310,62 +118,13 @@ Index: gdm-3.3.92.1/daemon/gdm-slave.c
|
||||
+}
|
||||
+
|
||||
void
|
||||
gdm_slave_set_busy_cursor (GdmSlave *slave)
|
||||
gdm_slave_set_initial_keyboard_layout (GdmSlave *slave)
|
||||
{
|
||||
@@ -1825,6 +1899,13 @@ _gdm_slave_set_display_is_local (GdmSlav
|
||||
}
|
||||
|
||||
static void
|
||||
+_gdm_slave_set_force_active_vt (GdmSlave *slave,
|
||||
+ gboolean force_active_vt)
|
||||
+{
|
||||
+ slave->priv->force_active_vt = force_active_vt;
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
gdm_slave_set_property (GObject *object,
|
||||
guint prop_id,
|
||||
const GValue *value,
|
||||
@@ -1856,6 +1937,9 @@ gdm_slave_set_property (GObject *ob
|
||||
case PROP_DISPLAY_IS_LOCAL:
|
||||
_gdm_slave_set_display_is_local (self, g_value_get_boolean (value));
|
||||
break;
|
||||
+ case PROP_FORCE_ACTIVE_VT:
|
||||
+ _gdm_slave_set_force_active_vt (self, g_value_get_boolean (value));
|
||||
+ break;
|
||||
default:
|
||||
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
|
||||
break;
|
||||
@@ -1894,6 +1978,9 @@ gdm_slave_get_property (GObject *obje
|
||||
case PROP_DISPLAY_IS_LOCAL:
|
||||
g_value_set_boolean (value, self->priv->display_is_local);
|
||||
break;
|
||||
+ case PROP_FORCE_ACTIVE_VT:
|
||||
+ g_value_set_boolean (value, self->priv->force_active_vt);
|
||||
+ break;
|
||||
default:
|
||||
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
|
||||
break;
|
||||
@@ -2019,6 +2106,14 @@ gdm_slave_class_init (GdmSlaveClass *kla
|
||||
TRUE,
|
||||
G_PARAM_READWRITE | G_PARAM_CONSTRUCT_ONLY));
|
||||
|
||||
+ g_object_class_install_property (object_class,
|
||||
+ PROP_FORCE_ACTIVE_VT,
|
||||
+ g_param_spec_boolean ("force-active-vt",
|
||||
+ "Force Active VT",
|
||||
+ "Force display to active VT",
|
||||
+ TRUE,
|
||||
+ G_PARAM_READWRITE | G_PARAM_CONSTRUCT_ONLY));
|
||||
+
|
||||
signals [STOPPED] =
|
||||
g_signal_new ("stopped",
|
||||
G_TYPE_FROM_CLASS (object_class),
|
||||
Index: gdm-3.3.92.1/daemon/gdm-slave.h
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-slave.h
|
||||
+++ gdm-3.3.92.1/daemon/gdm-slave.h
|
||||
@@ -78,6 +78,7 @@ void gdm_slave_set_initia
|
||||
diff --git a/daemon/gdm-slave.h b/daemon/gdm-slave.h
|
||||
index 7af20ed..aaaa8f2 100644
|
||||
--- a/daemon/gdm-slave.h
|
||||
+++ b/daemon/gdm-slave.h
|
||||
@@ -78,6 +78,7 @@ void gdm_slave_set_initial_keyboard_layout (GdmSlave *slave);
|
||||
void gdm_slave_set_initial_cursor_position (GdmSlave *slave);
|
||||
|
||||
void gdm_slave_set_busy_cursor (GdmSlave *slave);
|
||||
@ -373,147 +132,44 @@ Index: gdm-3.3.92.1/daemon/gdm-slave.h
|
||||
gboolean gdm_slave_run_script (GdmSlave *slave,
|
||||
const char *dir,
|
||||
const char *username);
|
||||
Index: gdm-3.3.92.1/configure.ac
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/configure.ac
|
||||
+++ gdm-3.3.92.1/configure.ac
|
||||
@@ -1332,6 +1332,23 @@ GDM_SIMPLE_GREETER_EXTENSIONS_DATA_DIR=$
|
||||
AC_SUBST(GDM_SIMPLE_GREETER_EXTENSIONS_DATA_DIR)
|
||||
|
||||
dnl ---------------------------------------------------------------------------
|
||||
+dnl - Directory to spool events from other processes
|
||||
+dnl ---------------------------------------------------------------------------
|
||||
+
|
||||
+AC_ARG_WITH(spool-dir,
|
||||
+ AS_HELP_STRING([--with-spool-dir=<dir>],
|
||||
+ [spool directory]))
|
||||
+
|
||||
+if ! test -z "$with_spool_dir"; then
|
||||
+ GDM_SPOOL_DIR=$with_spool_dir
|
||||
+else
|
||||
+ GDM_SPOOL_DIR=${localstatedir}/spool/gdm
|
||||
+fi
|
||||
+
|
||||
+AC_SUBST(GDM_SPOOL_DIR)
|
||||
+
|
||||
+
|
||||
+dnl ---------------------------------------------------------------------------
|
||||
dnl - Finish
|
||||
dnl ---------------------------------------------------------------------------
|
||||
|
||||
Index: gdm-3.3.92.1/daemon/Makefile.am
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/Makefile.am
|
||||
+++ gdm-3.3.92.1/daemon/Makefile.am
|
||||
@@ -15,6 +15,7 @@ AM_CPPFLAGS = \
|
||||
-DLOGDIR=\"$(logdir)\" \
|
||||
-DSBINDIR=\"$(sbindir)\" \
|
||||
-DGNOMELOCALEDIR=\""$(datadir)/locale"\" \
|
||||
+ -DGDM_SPOOL_DIR=\"$(GDM_SPOOL_DIR)\" \
|
||||
-DGDM_XAUTH_DIR=\"$(GDM_XAUTH_DIR)\" \
|
||||
-DGDM_SCREENSHOT_DIR=\"$(GDM_SCREENSHOT_DIR)\" \
|
||||
-DGDM_CACHE_DIR=\""$(localstatedir)/cache/gdm"\" \
|
||||
Index: gdm-3.3.92.1/daemon/gdm-display.c
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-display.c
|
||||
+++ gdm-3.3.92.1/daemon/gdm-display.c
|
||||
@@ -65,7 +65,9 @@ struct GdmDisplayPrivate
|
||||
gsize x11_cookie_size;
|
||||
GdmDisplayAccessFile *access_file;
|
||||
|
||||
- gboolean is_local;
|
||||
+ guint is_local : 1;
|
||||
+ guint force_active_vt : 1;
|
||||
+
|
||||
guint finish_idle_id;
|
||||
|
||||
GdmSlaveProxy *slave_proxy;
|
||||
@@ -84,6 +86,7 @@ enum {
|
||||
PROP_X11_COOKIE,
|
||||
PROP_X11_AUTHORITY_FILE,
|
||||
PROP_IS_LOCAL,
|
||||
+ PROP_FORCE_ACTIVE_VT,
|
||||
PROP_SLAVE_COMMAND,
|
||||
};
|
||||
|
||||
@@ -584,9 +587,10 @@ gdm_display_real_prepare (GdmDisplay *di
|
||||
gdm_slave_proxy_set_log_path (display->priv->slave_proxy, log_path);
|
||||
g_free (log_path);
|
||||
|
||||
- command = g_strdup_printf ("%s --display-id %s",
|
||||
+ command = g_strdup_printf ("%s --display-id %s %s",
|
||||
display->priv->slave_command,
|
||||
- display->priv->id);
|
||||
+ display->priv->id,
|
||||
+ display->priv->force_active_vt? "--force-active-vt" : "");
|
||||
gdm_slave_proxy_set_command (display->priv->slave_proxy, command);
|
||||
g_free (command);
|
||||
|
||||
@@ -834,6 +838,13 @@ _gdm_display_set_is_local (GdmDisplay
|
||||
}
|
||||
|
||||
static void
|
||||
+_gdm_display_set_force_active_vt (GdmDisplay *display,
|
||||
+ gboolean force_active_vt)
|
||||
+{
|
||||
+ display->priv->force_active_vt = force_active_vt;
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
_gdm_display_set_slave_command (GdmDisplay *display,
|
||||
const char *command)
|
||||
{
|
||||
@@ -876,6 +887,9 @@ gdm_display_set_property (GObject
|
||||
case PROP_IS_LOCAL:
|
||||
_gdm_display_set_is_local (self, g_value_get_boolean (value));
|
||||
break;
|
||||
+ case PROP_FORCE_ACTIVE_VT:
|
||||
+ _gdm_display_set_force_active_vt (self, g_value_get_boolean (value));
|
||||
+ break;
|
||||
case PROP_SLAVE_COMMAND:
|
||||
_gdm_display_set_slave_command (self, g_value_get_string (value));
|
||||
break;
|
||||
@@ -924,6 +938,9 @@ gdm_display_get_property (GObject
|
||||
case PROP_IS_LOCAL:
|
||||
g_value_set_boolean (value, self->priv->is_local);
|
||||
break;
|
||||
+ case PROP_FORCE_ACTIVE_VT:
|
||||
+ g_value_set_boolean (value, self->priv->force_active_vt);
|
||||
+ break;
|
||||
case PROP_SLAVE_COMMAND:
|
||||
g_value_set_string (value, self->priv->slave_command);
|
||||
break;
|
||||
@@ -1094,6 +1111,13 @@ gdm_display_class_init (GdmDisplayClass
|
||||
NULL,
|
||||
TRUE,
|
||||
G_PARAM_READWRITE | G_PARAM_CONSTRUCT));
|
||||
+ g_object_class_install_property (object_class,
|
||||
+ PROP_FORCE_ACTIVE_VT,
|
||||
+ g_param_spec_boolean ("force-active-vt",
|
||||
+ NULL,
|
||||
+ NULL,
|
||||
+ FALSE,
|
||||
+ G_PARAM_READWRITE | G_PARAM_CONSTRUCT));
|
||||
|
||||
g_object_class_install_property (object_class,
|
||||
PROP_SLAVE_COMMAND,
|
||||
Index: gdm-3.3.92.1/daemon/gdm-server.c
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-server.c
|
||||
+++ gdm-3.3.92.1/daemon/gdm-server.c
|
||||
@@ -32,8 +32,11 @@
|
||||
--
|
||||
1.7.9
|
||||
|
||||
|
||||
From 03d92cf39f74b265cc9936bdeabfcf071b102767 Mon Sep 17 00:00:00 2001
|
||||
From: Ray Strode <rstrode@redhat.com>
|
||||
Date: Fri, 27 Nov 2009 18:52:54 -0500
|
||||
Subject: [PATCH 2/2] Enable smooth transition between plymouth and X
|
||||
|
||||
This commit checks if plymouth is running, and if so,
|
||||
turns on the smooth transition between plymouth and X.
|
||||
---
|
||||
daemon/gdm-server.c | 55 +++++++++++++++++++++++++++
|
||||
daemon/gdm-server.h | 1 +
|
||||
daemon/gdm-simple-slave.c | 91 ++++++++++++++++++++++++++++++++++++++++++++-
|
||||
3 files changed, 146 insertions(+), 1 deletions(-)
|
||||
|
||||
diff --git a/daemon/gdm-server.c b/daemon/gdm-server.c
|
||||
index 54bf8b3..365590f 100644
|
||||
--- a/daemon/gdm-server.c
|
||||
+++ b/daemon/gdm-server.c
|
||||
@@ -32,6 +32,7 @@
|
||||
#include <pwd.h>
|
||||
#include <grp.h>
|
||||
#include <signal.h>
|
||||
+#include <sys/ioctl.h>
|
||||
#include <sys/resource.h>
|
||||
|
||||
+#include <linux/vt.h>
|
||||
+
|
||||
#ifdef HAVE_SYS_PRCTL_H
|
||||
#include <sys/prctl.h>
|
||||
@@ -41,6 +42,7 @@
|
||||
#ifdef WITH_SYSTEMD
|
||||
#include <systemd/sd-daemon.h>
|
||||
#endif
|
||||
@@ -751,6 +754,44 @@ gdm_server_spawn (GdmServer *server,
|
||||
+#include <linux/vt.h>
|
||||
|
||||
#include <glib.h>
|
||||
#include <glib/gi18n.h>
|
||||
@@ -732,6 +734,44 @@ gdm_server_spawn (GdmServer *server,
|
||||
return ret;
|
||||
}
|
||||
|
||||
@ -558,11 +214,10 @@ Index: gdm-3.3.92.1/daemon/gdm-server.c
|
||||
/**
|
||||
* gdm_server_start:
|
||||
* @disp: Pointer to a GdmDisplay structure
|
||||
@@ -768,6 +809,21 @@ gdm_server_start (GdmServer *server)
|
||||
|
||||
@@ -750,6 +790,21 @@ gdm_server_start (GdmServer *server)
|
||||
return res;
|
||||
}
|
||||
+
|
||||
|
||||
+gboolean
|
||||
+gdm_server_start_on_active_vt (GdmServer *server)
|
||||
+{
|
||||
@ -570,21 +225,22 @@ Index: gdm-3.3.92.1/daemon/gdm-server.c
|
||||
+ char *vt;
|
||||
+
|
||||
+ g_free (server->priv->command);
|
||||
+ server->priv->command = g_strdup (X_SERVER " -background none -verbose");
|
||||
+ server->priv->command = g_strdup (X_SERVER " -background none -logverbose 7");
|
||||
+ vt = get_active_vt_as_string ();
|
||||
+ res = gdm_server_spawn (server, vt);
|
||||
+ g_free (vt);
|
||||
+
|
||||
+ return res;
|
||||
+}
|
||||
|
||||
+
|
||||
static void
|
||||
server_died (GdmServer *server)
|
||||
Index: gdm-3.3.92.1/daemon/gdm-server.h
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-server.h
|
||||
+++ gdm-3.3.92.1/daemon/gdm-server.h
|
||||
@@ -57,6 +57,7 @@ GdmServer * gdm_server_new
|
||||
{
|
||||
diff --git a/daemon/gdm-server.h b/daemon/gdm-server.h
|
||||
index b53d68e..4939382 100644
|
||||
--- a/daemon/gdm-server.h
|
||||
+++ b/daemon/gdm-server.h
|
||||
@@ -57,6 +57,7 @@ GdmServer * gdm_server_new (const char *display_id,
|
||||
const char *seat_id,
|
||||
const char *auth_file);
|
||||
gboolean gdm_server_start (GdmServer *server);
|
||||
@ -592,110 +248,149 @@ Index: gdm-3.3.92.1/daemon/gdm-server.h
|
||||
gboolean gdm_server_stop (GdmServer *server);
|
||||
char * gdm_server_get_display_device (GdmServer *server);
|
||||
|
||||
Index: gdm-3.3.92.1/daemon/gdm-simple-slave.h
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-simple-slave.h
|
||||
+++ gdm-3.3.92.1/daemon/gdm-simple-slave.h
|
||||
@@ -48,7 +48,8 @@ typedef struct
|
||||
} GdmSimpleSlaveClass;
|
||||
diff --git a/daemon/gdm-simple-slave.c b/daemon/gdm-simple-slave.c
|
||||
index fc8649c..dd19ced 100644
|
||||
--- a/daemon/gdm-simple-slave.c
|
||||
+++ b/daemon/gdm-simple-slave.c
|
||||
@@ -93,6 +93,7 @@ struct GdmSimpleSlavePrivate
|
||||
#ifdef HAVE_LOGINDEVPERM
|
||||
gboolean use_logindevperm;
|
||||
#endif
|
||||
+ guint plymouth_is_running : 1;
|
||||
};
|
||||
|
||||
GType gdm_simple_slave_get_type (void);
|
||||
-GdmSlave * gdm_simple_slave_new (const char *id);
|
||||
+GdmSlave * gdm_simple_slave_new (const char *id,
|
||||
+ gboolean force_active_vt);
|
||||
enum {
|
||||
@@ -1204,6 +1205,72 @@ on_start_session_later (GdmGreeterServer *session,
|
||||
slave->priv->start_session_when_ready = FALSE;
|
||||
}
|
||||
|
||||
G_END_DECLS
|
||||
+static gboolean
|
||||
+plymouth_is_running (void)
|
||||
+{
|
||||
+ int status;
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth --ping",
|
||||
+ NULL, NULL, &status, &error);
|
||||
+ if (! res) {
|
||||
+ g_debug ("Could not ping plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ return FALSE;
|
||||
+ }
|
||||
+
|
||||
+ return WIFEXITED (status) && WEXITSTATUS (status) == 0;
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
+plymouth_prepare_for_transition (GdmSimpleSlave *slave)
|
||||
+{
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth deactivate",
|
||||
+ NULL, NULL, NULL, &error);
|
||||
+ if (! res) {
|
||||
+ g_warning ("Could not deactivate plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ }
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
+plymouth_quit_with_transition (GdmSimpleSlave *slave)
|
||||
+{
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth quit --retain-splash",
|
||||
+ NULL, NULL, NULL, &error);
|
||||
+ if (! res) {
|
||||
+ g_warning ("Could not quit plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ }
|
||||
+ slave->priv->plymouth_is_running = FALSE;
|
||||
+}
|
||||
+
|
||||
+static void
|
||||
+plymouth_quit_without_transition (GdmSimpleSlave *slave)
|
||||
+{
|
||||
+ gboolean res;
|
||||
+ GError *error;
|
||||
+
|
||||
+ error = NULL;
|
||||
+ res = g_spawn_command_line_sync ("/bin/plymouth quit",
|
||||
+ NULL, NULL, NULL, &error);
|
||||
+ if (! res) {
|
||||
+ g_warning ("Could not quit plymouth: %s", error->message);
|
||||
+ g_error_free (error);
|
||||
+ }
|
||||
+ slave->priv->plymouth_is_running = FALSE;
|
||||
+}
|
||||
+
|
||||
static void
|
||||
setup_server (GdmSimpleSlave *slave)
|
||||
{
|
||||
@@ -1223,6 +1290,10 @@ setup_server (GdmSimpleSlave *slave)
|
||||
*/
|
||||
gdm_slave_save_root_windows (GDM_SLAVE (slave));
|
||||
|
||||
Index: gdm-3.3.92.1/daemon/gdm-static-display.c
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/gdm-static-display.c
|
||||
+++ gdm-3.3.92.1/daemon/gdm-static-display.c
|
||||
@@ -81,10 +81,27 @@ gdm_static_display_remove_user_authoriza
|
||||
+ /* Plymouth is waiting for the go-ahead to exit */
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_with_transition (slave);
|
||||
+ }
|
||||
}
|
||||
|
||||
static void
|
||||
@@ -1426,6 +1497,10 @@ on_server_exited (GdmServer *server,
|
||||
g_debug ("GdmSimpleSlave: server exited with code %d\n", exit_code);
|
||||
|
||||
gdm_slave_stopped (GDM_SLAVE (slave));
|
||||
+
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_without_transition (slave);
|
||||
+ }
|
||||
}
|
||||
|
||||
static void
|
||||
@@ -1438,6 +1513,10 @@ on_server_died (GdmServer *server,
|
||||
g_strsignal (signal_number));
|
||||
|
||||
gdm_slave_stopped (GDM_SLAVE (slave));
|
||||
+
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_without_transition (slave);
|
||||
+ }
|
||||
}
|
||||
|
||||
static gboolean
|
||||
+triggered_to_force_display_on_active_vt (void)
|
||||
+{
|
||||
+ gboolean should_force_display_on_active_vt;
|
||||
@@ -1484,7 +1563,14 @@ gdm_simple_slave_run (GdmSimpleSlave *slave)
|
||||
G_CALLBACK (on_server_ready),
|
||||
slave);
|
||||
|
||||
- res = gdm_server_start (slave->priv->server);
|
||||
+ slave->priv->plymouth_is_running = plymouth_is_running ();
|
||||
+
|
||||
+ should_force_display_on_active_vt = g_file_test (GDM_SPOOL_DIR "/force-display-on-active-vt",
|
||||
+ G_FILE_TEST_EXISTS);
|
||||
+ g_unlink (GDM_SPOOL_DIR "/force-display-on-active-vt");
|
||||
+
|
||||
+ return should_force_display_on_active_vt;
|
||||
+}
|
||||
+
|
||||
+static gboolean
|
||||
gdm_static_display_manage (GdmDisplay *display)
|
||||
{
|
||||
g_return_val_if_fail (GDM_IS_DISPLAY (display), FALSE);
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_prepare_for_transition (slave);
|
||||
+ res = gdm_server_start_on_active_vt (slave->priv->server);
|
||||
+ } else {
|
||||
+ res = gdm_server_start (slave->priv->server);
|
||||
+ }
|
||||
if (! res) {
|
||||
g_warning (_("Could not start the X "
|
||||
"server (your graphical environment) "
|
||||
@@ -1494,6 +1580,9 @@ gdm_simple_slave_run (GdmSimpleSlave *slave)
|
||||
"In the meantime this display will be "
|
||||
"disabled. Please restart GDM when "
|
||||
"the problem is corrected."));
|
||||
+ if (slave->priv->plymouth_is_running) {
|
||||
+ plymouth_quit_without_transition (slave);
|
||||
+ }
|
||||
exit (1);
|
||||
}
|
||||
|
||||
+ if (triggered_to_force_display_on_active_vt ()) {
|
||||
+ g_object_set (display, "force-active-vt", TRUE, NULL);
|
||||
+ } else {
|
||||
+ g_object_set (display, "force-active-vt", FALSE, NULL);
|
||||
+ }
|
||||
GDM_DISPLAY_CLASS (gdm_static_display_parent_class)->manage (display);
|
||||
|
||||
return TRUE;
|
||||
Index: gdm-3.3.92.1/daemon/simple-slave-main.c
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/daemon/simple-slave-main.c
|
||||
+++ gdm-3.3.92.1/daemon/simple-slave-main.c
|
||||
@@ -177,9 +177,11 @@ main (int argc,
|
||||
DBusGConnection *connection;
|
||||
GdmSlave *slave;
|
||||
static char *display_id = NULL;
|
||||
+ static gboolean force_active_vt = FALSE;
|
||||
GdmSignalHandler *signal_handler;
|
||||
static GOptionEntry entries [] = {
|
||||
{ "display-id", 0, 0, G_OPTION_ARG_STRING, &display_id, N_("Display ID"), N_("ID") },
|
||||
+ { "force-active-vt", 0, 0, G_OPTION_ARG_NONE, &force_active_vt, N_("Force X to start on active vt"), NULL },
|
||||
{ NULL }
|
||||
};
|
||||
|
||||
@@ -246,7 +248,7 @@ main (int argc,
|
||||
gdm_signal_handler_add (signal_handler, SIGUSR1, signal_cb, NULL);
|
||||
gdm_signal_handler_add (signal_handler, SIGUSR2, signal_cb, NULL);
|
||||
|
||||
- slave = gdm_simple_slave_new (display_id);
|
||||
+ slave = gdm_simple_slave_new (display_id, force_active_vt);
|
||||
if (slave == NULL) {
|
||||
goto out;
|
||||
}
|
||||
Index: gdm-3.3.92.1/data/Makefile.am
|
||||
===================================================================
|
||||
--- gdm-3.3.92.1.orig/data/Makefile.am
|
||||
+++ gdm-3.3.92.1/data/Makefile.am
|
||||
@@ -12,6 +12,7 @@ predir = $(gdmconfdir)/PreSession
|
||||
postlogindir = $(gdmconfdir)/PostLogin
|
||||
workingdir = $(GDM_WORKING_DIR)
|
||||
xauthdir = $(GDM_XAUTH_DIR)
|
||||
+spooldir = $(GDM_SPOOL_DIR)
|
||||
screenshotdir = $(GDM_SCREENSHOT_DIR)
|
||||
cachedir = $(localstatedir)/cache/gdm
|
||||
|
||||
@@ -162,7 +163,8 @@ uninstall-hook:
|
||||
$(DESTDIR)$(sysconfdir)/dconf/db/gdm.d \
|
||||
$(DESTDIR)$(workingdir)/.config/dconf \
|
||||
$(DESTDIR)$(screenshotdir) \
|
||||
- $(DESTDIR)$(xauthdir)
|
||||
+ $(DESTDIR)$(xauthdir) \
|
||||
+ $(DESTDIR)$(spooldir)
|
||||
|
||||
-rmdir \
|
||||
$(DESTDIR)$(sysconfdir)/dconf/db/gdm.d/locks \
|
||||
@@ -307,6 +309,13 @@ install-data-hook: gdm.conf-custom Xsess
|
||||
chown root:gdm $(DESTDIR)$(cachedir) || : ; \
|
||||
fi
|
||||
|
||||
+ if test '!' -d $(DESTDIR)$(spooldir); then \
|
||||
+ $(mkinstalldirs) $(DESTDIR)$(spooldir); \
|
||||
+ chmod 775 $(DESTDIR)$(spooldir); \
|
||||
+ chown root:gdm $(DESTDIR)$(spooldir) || : ; \
|
||||
+ fi
|
||||
+
|
||||
+
|
||||
$(INSTALL_DATA) $(srcdir)/dconf-profile $(DESTDIR)$(sysconfdir)/dconf/profile/gdm
|
||||
$(INSTALL_DATA) $(srcdir)/00-upstream-settings $(DESTDIR)$(sysconfdir)/dconf/db/gdm.d
|
||||
$(INSTALL_DATA) $(srcdir)/00-upstream-settings-locks $(DESTDIR)$(sysconfdir)/dconf/db/gdm.d/locks
|
||||
--
|
||||
1.7.9
|
||||
|
29
gdm.changes
29
gdm.changes
@ -1,3 +1,17 @@
|
||||
-------------------------------------------------------------------
|
||||
Wed Mar 21 15:02:48 UTC 2012 - vuntz@opensuse.org
|
||||
|
||||
- Also pass --with-console-kit to configure to make sure we also
|
||||
build the ConsoleKit support.
|
||||
- Move %glib2_gsettings_schema_requires from
|
||||
gdm-simple-greeter-extensions to main subpackage, where the
|
||||
GSettings schemas live.
|
||||
- Update gdm-plymouth.patch with Fedora's version.
|
||||
- Rebase and re-enable gdm-greeter-greater-ui.patch.
|
||||
- Disable gnome-patch-translation-{prepare,update} calls in %prep
|
||||
as they're only needed for gdm-domain-logon.patch, which needs to
|
||||
be rebased.
|
||||
|
||||
-------------------------------------------------------------------
|
||||
Tue Mar 20 20:41:28 UTC 2012 - dimstar@opensuse.org
|
||||
|
||||
@ -10,12 +24,15 @@ Tue Mar 20 20:41:28 UTC 2012 - dimstar@opensuse.org
|
||||
+ Auth fixes.
|
||||
+ Miscellaneous fixes.
|
||||
+ Updated translations.
|
||||
- Drop gdm-gconf-path.patch: code converted to gsettings.
|
||||
- Rebase patches: gdm-desktop-session-env-pam.patch and
|
||||
gdm-plymouth.patch.
|
||||
- Temporary disable gdm-greeter-greater-ui.patch: needs rebase.
|
||||
- Drop gconf2-devel BuildRequires: ported to gsettings.
|
||||
- Prepare for the switch to systemd:
|
||||
- Drop gdm-gconf-path.patch: not needed after port to GSettings.
|
||||
- Rebase gdm-desktop-session-env-pam.patch and gdm-plymouth.patch.
|
||||
- Temporarily disable gdm-greeter-greater-ui.patch: it needs to be
|
||||
rebased.
|
||||
- Drop gconf2-devel BuildRequires and gconf packaging: gdm got
|
||||
ported to GSettings.
|
||||
- Enable systemd support:
|
||||
+ Note that it nicely falls back to ConsoleKit if systemd is not
|
||||
used on the system.
|
||||
+ Pass --with-systemd to configure
|
||||
+ Add pkgconfig(libsystemd-login) and
|
||||
pkgconfig(libsystemd-daemon) BuildRequires.
|
||||
|
13
gdm.spec
13
gdm.spec
@ -18,7 +18,6 @@
|
||||
|
||||
# FIXME: need to check what should be done to enable this (at least adapt the pam files). See bnc#699999
|
||||
%define enable_split_authentication 0
|
||||
%define with_systemd 1
|
||||
|
||||
Name: gdm
|
||||
Version: 3.3.92.1
|
||||
@ -54,7 +53,7 @@ Patch8: gdm-sysconfig-settings.patch
|
||||
Patch9: gdm-passwordless-login.patch
|
||||
# PATCH-NEEDS-REBASE gdm-domain-logon.patch hpj@novell.com -- Add UI to log in a specific domain (was: PATCH_FEATURE-SLED bnc#627575)
|
||||
Patch10: gdm-domain-logon.patch
|
||||
# PATCH-NEEDS-REBASE gdm-greeter-greater-ui.patch bnc436431 bgo560508 vuntz@novell.com -- Improve the layout of the greeter. Note: the patch contains a patched glade file *and* the result glade file (for reference only, in case we need to rebase the patch) (WAS: PATCH-FIX-UPSTREAM)
|
||||
# PATCH-FIX-UPSTREAM gdm-greeter-greater-ui.patch bnc436431 bgo560508 vuntz@novell.com -- Improve the layout of the greeter. Note: the patch contains a patched glade file *and* the result glade file (for reference only, in case we need to rebase the patch)
|
||||
Patch15: gdm-greeter-greater-ui.patch
|
||||
# PATCH-FIX-UPSTREAM gdm-always-reflect-keyboard-layout.patch bnc438159 bgo561771 hpj@novell.com -- Make keyboard selector not neglect to apply the selected keyboard in some situations.
|
||||
Patch18: gdm-always-reflect-keyboard-layout.patch
|
||||
@ -133,6 +132,7 @@ Provides: gnome-applets-gdm = %{version}
|
||||
Obsoletes: gnome-applets-gdm < %{version}
|
||||
BuildRoot: %{_tmppath}/%{name}-%{version}-build
|
||||
DocDir: %{_defaultdocdir}
|
||||
%glib2_gsettings_schema_requires
|
||||
|
||||
%description
|
||||
The GNOME Display Manager is a system service that is responsible for
|
||||
@ -173,7 +173,6 @@ providing graphical log-ins and managing local and remote displays.
|
||||
Summary: GDM Simple Greeter Extensions -- Fingerprint and Smartcard Support
|
||||
Group: System/Libraries
|
||||
Supplements: gdm
|
||||
%glib2_gsettings_schema_requires
|
||||
|
||||
%description simple-greeter-extensions
|
||||
The GNOME Display Manager is a system service that is responsible for
|
||||
@ -228,7 +227,7 @@ running display manager.
|
||||
%setup -q
|
||||
translation-update-upstream
|
||||
#disabled, needed for patch10
|
||||
gnome-patch-translation-prepare
|
||||
#gnome-patch-translation-prepare
|
||||
%patch0 -p1
|
||||
%patch2 -p1
|
||||
%patch3 -p1
|
||||
@ -238,14 +237,13 @@ gnome-patch-translation-prepare
|
||||
%patch8 -p1
|
||||
%patch9 -p1
|
||||
#%%patch10 -p1
|
||||
# needs rebase (bnc#689552)
|
||||
#patch15 -p1
|
||||
%patch15 -p1
|
||||
%patch18 -p1
|
||||
%patch34 -p1
|
||||
%patch35 -p0
|
||||
%patch40 -p1
|
||||
%patch60
|
||||
gnome-patch-translation-update
|
||||
#gnome-patch-translation-update
|
||||
|
||||
%build
|
||||
libtoolize -f -i
|
||||
@ -260,6 +258,7 @@ autoreconf -f -i
|
||||
--with-selinux \
|
||||
--enable-ipv6 \
|
||||
--with-systemd \
|
||||
--with-console-kit \
|
||||
%if %{enable_split_authentication}
|
||||
--enable-split-authentication \
|
||||
%else
|
||||
|
Loading…
x
Reference in New Issue
Block a user