From 3e5779207e2bfab25978e80ac62e8af8c3685f8e Mon Sep 17 00:00:00 2001

From: Hans de Goede <hdegoede@redhat.com>

Date: Tue, 14 Aug 2018 14:41:43 +0200

Subject: [PATCH 1/2] Add support for new ConfirmedRebootToBootOptions signal

 send by org.gnome.SessionManager.EndSessionDialog

In order to allow the user to choose advanced boot-options on systems where

the boot menu is not shown by default at boot, gnome-shell's endSessionDialog.js

may emit a new ConfirmedRebootToBootOptions signal to indicate that

gnome-session should reboot the system in such a way that the boot-menu will

be shown next boot.

This implements the backend for the "Boot Options" button from:

https://wiki.gnome.org/Design/OS/BootOptions

Note that at the moment ConfirmedRebootToBootOptions is just treated as an

alias to ConfirmedReboot, since there is no bootloader agnostic way yet to

indicate the menu should be shown.

For now the FIXME may be patched with a distro specific patch to implement

this for the distro's chosen bootloader. I will start a discussion on

systemd-devel to come up with a distro / bootloader agnostic interface

for this.

---

 gnome-session/gsm-manager.c | 28 ++++++++++++++++++++++++++++

 gnome-session/gsm-shell.c   | 10 ++++++++++

 gnome-session/gsm-shell.h   |  7 ++++---

 3 files changed, 42 insertions(+), 3 deletions(-)

diff --git a/gnome-session/gsm-manager.c b/gnome-session/gsm-manager.c

index d41bdcac..e0c05d82 100644

--- a/gnome-session/gsm-manager.c

+++ b/gnome-session/gsm-manager.c

@@ -117,6 +117,7 @@ typedef enum

         GSM_MANAGER_LOGOUT_NONE,

         GSM_MANAGER_LOGOUT_LOGOUT,

         GSM_MANAGER_LOGOUT_REBOOT,

+        GSM_MANAGER_LOGOUT_REBOOT_TO_BOOT_OPTIONS,

         GSM_MANAGER_LOGOUT_REBOOT_INTERACT,

         GSM_MANAGER_LOGOUT_SHUTDOWN,

         GSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT,

@@ -176,6 +177,7 @@ struct GsmManagerPrivate

         guint                   shell_end_session_dialog_confirmed_logout_id;

         guint                   shell_end_session_dialog_confirmed_shutdown_id;

         guint                   shell_end_session_dialog_confirmed_reboot_id;

+        guint                   shell_end_session_dialog_confirmed_reboot_to_boot_options_id;

 };

 enum {

@@ -490,6 +492,7 @@ gsm_manager_quit (GsmManager *manager)

                 gsm_quit ();

                 break;

         case GSM_MANAGER_LOGOUT_REBOOT:

+        case GSM_MANAGER_LOGOUT_REBOOT_TO_BOOT_OPTIONS:

         case GSM_MANAGER_LOGOUT_REBOOT_INTERACT:

                 gsm_system_complete_shutdown (manager->priv->system);

                 gsm_quit ();

@@ -1161,6 +1164,7 @@ end_session_or_show_shell_dialog (GsmManager *manager)

                 type = GSM_SHELL_END_SESSION_DIALOG_TYPE_LOGOUT;

                 break;

         case GSM_MANAGER_LOGOUT_REBOOT:

+        case GSM_MANAGER_LOGOUT_REBOOT_TO_BOOT_OPTIONS:

         case GSM_MANAGER_LOGOUT_REBOOT_INTERACT:

                 type = GSM_SHELL_END_SESSION_DIALOG_TYPE_RESTART;

                 break;

@@ -3450,6 +3454,12 @@ disconnect_shell_dialog_signals (GsmManager *manager)

                 manager->priv->shell_end_session_dialog_confirmed_reboot_id = 0;

         }

+        if (manager->priv->shell_end_session_dialog_confirmed_reboot_to_boot_options_id != 0) {

+                g_signal_handler_disconnect (manager->priv->shell,

+                                             manager->priv->shell_end_session_dialog_confirmed_reboot_to_boot_options_id);

+                manager->priv->shell_end_session_dialog_confirmed_reboot_to_boot_options_id = 0;

+        }

+

         if (manager->priv->shell_end_session_dialog_open_failed_id != 0) {

                 g_signal_handler_disconnect (manager->priv->shell,

                                              manager->priv->shell_end_session_dialog_open_failed_id);

@@ -3512,6 +3522,14 @@ on_shell_end_session_dialog_confirmed_reboot (GsmShell   *shell,

         disconnect_shell_dialog_signals (manager);

 }

+static void

+on_shell_end_session_dialog_confirmed_reboot_to_boot_options (GsmShell   *shell,

+                                                           GsmManager *manager)

+{

+        _handle_end_session_dialog_response (manager, GSM_MANAGER_LOGOUT_REBOOT_TO_BOOT_OPTIONS);

+        disconnect_shell_dialog_signals (manager);

+}

+

 static void

 connect_shell_dialog_signals (GsmManager *manager)

 {

@@ -3547,6 +3565,12 @@ connect_shell_dialog_signals (GsmManager *manager)

                                   "end-session-dialog-confirmed-reboot",

                                   G_CALLBACK (on_shell_end_session_dialog_confirmed_reboot),

                                   manager);

+

+        manager->priv->shell_end_session_dialog_confirmed_reboot_to_boot_options_id =

+                g_signal_connect (manager->priv->shell,

+                                  "end-session-dialog-confirmed-reboot-to-boot-options",

+                                  G_CALLBACK (on_shell_end_session_dialog_confirmed_reboot_to_boot_options),

+                                  manager);

 }

 static void

@@ -3820,6 +3844,10 @@ do_query_end_session_exit (GsmManager *manager)

         case GSM_MANAGER_LOGOUT_REBOOT_INTERACT:

                 reboot = TRUE;

                 break;

+        case GSM_MANAGER_LOGOUT_REBOOT_TO_BOOT_OPTIONS:

+                /* FIXME tell bootmanager to show menu on next boot */

+                reboot = TRUE;

+                break;

         case GSM_MANAGER_LOGOUT_SHUTDOWN:

         case GSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT:

                 shutdown = TRUE;

diff --git a/gnome-session/gsm-shell.c b/gnome-session/gsm-shell.c

index 04cfa2f5..d727232d 100644

--- a/gnome-session/gsm-shell.c

+++ b/gnome-session/gsm-shell.c

@@ -69,6 +69,7 @@ enum {

         END_SESSION_DIALOG_CONFIRMED_LOGOUT,

         END_SESSION_DIALOG_CONFIRMED_SHUTDOWN,

         END_SESSION_DIALOG_CONFIRMED_REBOOT,

+        END_SESSION_DIALOG_CONFIRMED_REBOOT_TO_BOOT_OPTIONS,

         NUMBER_OF_SIGNALS

 };

@@ -179,6 +180,14 @@ gsm_shell_class_init (GsmShellClass *shell_class)

                               NULL, NULL, NULL,

                               G_TYPE_NONE, 0);

+        signals [END_SESSION_DIALOG_CONFIRMED_REBOOT_TO_BOOT_OPTIONS] =

+                g_signal_new ("end-session-dialog-confirmed-reboot-to-boot-options",

+                              G_OBJECT_CLASS_TYPE (object_class),

+                              G_SIGNAL_RUN_LAST,

+                              G_STRUCT_OFFSET (GsmShellClass, end_session_dialog_confirmed_reboot_to_boot_options),

+                              NULL, NULL, NULL,

+                              G_TYPE_NONE, 0);

+

         g_type_class_add_private (shell_class, sizeof (GsmShellPrivate));

 }

@@ -342,6 +351,7 @@ on_end_session_dialog_dbus_signal (GDBusProxy *proxy,

                 { "Canceled", END_SESSION_DIALOG_CANCELED },

                 { "ConfirmedLogout", END_SESSION_DIALOG_CONFIRMED_LOGOUT },

                 { "ConfirmedReboot", END_SESSION_DIALOG_CONFIRMED_REBOOT },

+                { "ConfirmedRebootToBootOptions", END_SESSION_DIALOG_CONFIRMED_REBOOT_TO_BOOT_OPTIONS },

                 { "ConfirmedShutdown", END_SESSION_DIALOG_CONFIRMED_SHUTDOWN },

                 { NULL, -1 }

         };

diff --git a/gnome-session/gsm-shell.h b/gnome-session/gsm-shell.h

index e236493a..b1cdd457 100644

--- a/gnome-session/gsm-shell.h

+++ b/gnome-session/gsm-shell.h

@@ -64,9 +64,10 @@ struct _GsmShellClass

         void (* end_session_dialog_closed)        (GsmShell *shell);

         void (* end_session_dialog_canceled)      (GsmShell *shell);

-        void (* end_session_dialog_confirmed_logout)   (GsmShell *shell);

-        void (* end_session_dialog_confirmed_shutdown) (GsmShell *shell);

-        void (* end_session_dialog_confirmed_reboot)   (GsmShell *shell);

+        void (* end_session_dialog_confirmed_logout)                 (GsmShell *shell);

+        void (* end_session_dialog_confirmed_shutdown)               (GsmShell *shell);

+        void (* end_session_dialog_confirmed_reboot)                 (GsmShell *shell);

+        void (* end_session_dialog_confirmed_reboot_to_boot_options) (GsmShell *shell);

 };

-- 

2.23.0