Fix verbose/debug logging, add G_LOG_DOMAIN for easier tracking

in journalctl.

Verbose mode was broken for gtk-based daemons, as post-gtk_init
is too late to set G_MESSAGES_DEBUG.

Author: mtwebster

plugins/a11y-settings/meson.build

@@ -17,6 +17,7 @@ executable(
     include_directories: [include_dirs, common_inc],
     dependencies: a11y_settings_deps,
     c_args: [
+        '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
         '-DPLUGIN_NAME="@0@"'.format(plugin_name),
     ],
     install: true,

plugins/automount/meson.build

@@ -18,6 +18,7 @@ executable(
     include_directories: [include_dirs, common_inc],
     dependencies: automount_deps,
     c_args: [
+        '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
         '-DPLUGIN_NAME="@0@"'.format(plugin_name),
     ],
     install_rpath: join_paths(prefix, apilibdir),

plugins/background/meson.build

@@ -19,6 +19,7 @@ executable(
     include_directories: [include_dirs, common_inc],
     dependencies: background_deps,
     c_args: [
+        '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
         '-DPLUGIN_NAME="@0@"'.format(plugin_name),
     ],
     install: true,

plugins/clipboard/meson.build

@@ -19,6 +19,7 @@ executable(
     include_directories: [include_dirs, common_inc],
     dependencies: clipboard_deps,
     c_args: [
+        '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
         '-DPLUGIN_NAME="@0@"'.format(plugin_name),
     ],
     install: true,

plugins/color/meson.build

@@ -37,6 +37,7 @@ executable(
   dependencies: color_deps,
   c_args: [
       '-DPLUGIN_NAME="@0@"'.format(plugin_name),
+      '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
       '-DBINDIR="@0@"'.format(bindir),
   ],
   install: true,

plugins/common/daemon-skeleton-gtk.h

@@ -174,6 +174,23 @@ handle_signal (gpointer user_data)
   return G_SOURCE_REMOVE;
 }
 
+static void
+message_handler (const gchar *log_domain,
+                 GLogLevelFlags log_level,
+                 const gchar *message,
+                 gpointer user_data)
+{
+  /* Make this look like normal console output */
+  if (log_level & G_LOG_LEVEL_DEBUG) {
+    g_autoptr(GDateTime) dt = g_date_time_new_now_local ();
+    g_autofree gchar *iso8601 = g_date_time_format (dt, "%F:%T");
+    printf ("csd-%s:%s: %s\n", PLUGIN_NAME, iso8601, message);
+  }
+  else {
+    printf ("%s: %s\n", g_get_prgname (), message);
+  }
+}
+
 int
 main (int argc, char **argv)
 {
@@ -221,7 +238,7 @@ main (int argc, char **argv)
         g_unix_signal_add (SIGTERM, (GSourceFunc) handle_signal, NULL);
 
         if (verbose)
-                g_setenv ("G_MESSAGES_DEBUG", "all", TRUE);
+            g_log_set_handler (G_LOG_DOMAIN, G_LOG_LEVEL_DEBUG, message_handler, NULL);
 
         if (timeout > 0) {
                 guint id;

plugins/common/daemon-skeleton.h

@@ -171,6 +171,23 @@ handle_signal (gpointer user_data)
     return G_SOURCE_REMOVE;
 }
 
+static void
+message_handler (const gchar *log_domain,
+                 GLogLevelFlags log_level,
+                 const gchar *message,
+                 gpointer user_data)
+{
+  /* Make this look like normal console output */
+  if (log_level & G_LOG_LEVEL_DEBUG) {
+    g_autoptr(GDateTime) dt = g_date_time_new_now_local ();
+    g_autofree gchar *iso8601 = g_date_time_format (dt, "%F:%T");
+    printf ("csd-%s:%s: %s\n", PLUGIN_NAME, iso8601, message);
+  }
+  else {
+    printf ("%s: %s\n", g_get_prgname (), message);
+  }
+}
+
 int
 main (int argc, char **argv)
 {
@@ -208,10 +225,8 @@ main (int argc, char **argv)
 
         g_unix_signal_add (SIGTERM, (GSourceFunc) handle_signal, loop);
 
-        if (verbose) {
-                g_setenv ("G_MESSAGES_DEBUG", "all", TRUE);
-                setlinebuf (stdout);
-        }
+        if (verbose)
+            g_log_set_handler (G_LOG_DOMAIN, G_LOG_LEVEL_DEBUG, message_handler, NULL);
 
         if (timeout > 0) {
                 guint id;

plugins/datetime/meson.build

@@ -47,6 +47,7 @@ if polkit.found()
         include_directories: [include_dirs, common_inc],
         dependencies: datetime_deps,
         c_args: [
+            '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
             '-DPLUGIN_NAME="@0@"'.format(plugin_name),
         ],
         install: true,

plugins/housekeeping/meson.build

@@ -25,6 +25,7 @@ executable(
     include_directories: [include_dirs, common_inc],
     dependencies: housekeeping_deps,
     c_args: [
+        '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
         '-DPLUGIN_NAME="@0@"'.format(plugin_name),
     ],
     install: true,

plugins/keyboard/meson.build

@@ -23,6 +23,7 @@ executable(
     include_directories: [include_dirs, common_inc, include_enums],
     dependencies: keyboard_deps,
     c_args: [
+        '-DG_LOG_DOMAIN="csd-@0@"'.format(plugin_name),
         '-DPLUGIN_NAME="@0@"'.format(plugin_name),
         '-DDATADIR="@0@"'.format(datadir),
     ],