Attachment 640536 Details for Bug 936842

[patch] u_log-to-xdg-data-home.patch

u_log-to-xdg-data-home.patch (text/plain), 4.48 KB, created by Federico Mena Quintero on 2015-07-09 21:33:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Federico Mena Quintero

Created: 2015-07-09 21:33:48 UTC

Size: 4.48 KB

patch

obsolete

>From dd40c9486a90764d1ba77c0f0114ac5b8e9893ef Mon Sep 17 00:00:00 2001
>From: Hans de Goede <hdegoede@redhat.com>
>Date: Thu, 9 Jul 2015 15:10:42 -0500
>Subject: [PATCH] xf86LogInit: log to XDG_DATA_HOME when not running as root
>
>When no logfile was specified (xf86LogFileFrom == X_DEFAULT) and we're not
>running as root log to $XDG_DATA_HOME/xorg/Xorg.#.log as Xorg won't be able to
>log to the default /var/log/... when it is not running as root.
>
>Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
>Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
>---
> configure.ac                   |  4 ++++
> hw/xfree86/common/xf86Helper.c | 31 ++++++++++++++++++++++++++++++-
> hw/xfree86/man/xorg.conf.man   |  6 +++++-
> include/xorg-config.h.in       |  6 ++++++
> 4 files changed, 45 insertions(+), 2 deletions(-)
>
>diff --git a/configure.ac b/configure.ac
>index 5951737..7427448 100644
>--- a/configure.ac
>+++ b/configure.ac
>@@ -2012,6 +2012,8 @@ if test "x$XORG" = xyes; then
> 	AC_SUBST(XF86CONFIGDIR)
> 	CONFIGFILE="$sysconfdir/$XF86CONFIGFILE"
> 	LOGPREFIX="$logdir/Xorg."
>+	XDG_DATA_HOME=".local/share"
>+	XDG_DATA_HOME_LOGDIR="xorg"
> 	AC_DEFINE(XORG_SERVER, 1, [Building Xorg server])
> 	AC_DEFINE(XORGSERVER, 1, [Building Xorg server])
> 	AC_DEFINE(XFree86Server, 1, [Building XFree86 server])
>@@ -2025,6 +2027,8 @@ if test "x$XORG" = xyes; then
> 	AC_DEFINE_DIR(DEFAULT_MODULE_PATH, moduledir, [Default module search path])
> 	AC_DEFINE_DIR(DEFAULT_LIBRARY_PATH, libdir, [Default library install path])
> 	AC_DEFINE_DIR(DEFAULT_LOGPREFIX, LOGPREFIX, [Default log location])
>+	AC_DEFINE_DIR(DEFAULT_XDG_DATA_HOME, XDG_DATA_HOME, [Default XDG_DATA dir under HOME])
>+	AC_DEFINE_DIR(DEFAULT_XDG_DATA_HOME_LOGDIR, XDG_DATA_HOME_LOGDIR, [Default log dir under XDG_DATA_HOME])
> 	AC_DEFINE_UNQUOTED(__VENDORDWEBSUPPORT__, ["$VENDOR_WEB"], [Vendor web address for support])
> 	if test "x$VGAHW" = xyes; then
> 		AC_DEFINE(WITH_VGAHW, 1, [Building vgahw module])
>diff --git a/hw/xfree86/common/xf86Helper.c b/hw/xfree86/common/xf86Helper.c
>index 839aa5c..fe1093c 100644
>--- a/hw/xfree86/common/xf86Helper.c
>+++ b/hw/xfree86/common/xf86Helper.c
>@@ -1240,16 +1240,45 @@ xf86ErrorF(const char *format, ...)
>     va_end(ap);
> }
> 
>+/* Note temporarily modifies the passed in buffer! */
>+static void xf86_mkdir_p(char *path)
>+{
>+    char *sep = path;
>+
>+    while ((sep = strchr(sep + 1, '/'))) {
>+        *sep = 0;
>+        (void)mkdir(path, 0777);
>+        *sep = '/';
>+    }
>+    (void)mkdir(path, 0777);
>+}
>+
> void
> xf86LogInit(void)
> {
>-    char *lf = NULL;
>+    char *env, *lf = NULL;
>+    char buf[PATH_MAX];
> 
> #define LOGSUFFIX ".log"
> #define LOGOLDSUFFIX ".old"
> 
>     /* Get the log file name */
>     if (xf86LogFileFrom == X_DEFAULT) {
>+        /* When not running as root, we won't be able to write to /var/log */
>+        if (geteuid() != 0) {
>+            if ((env = getenv("XDG_DATA_HOME")))
>+                snprintf(buf, sizeof(buf), "%s/%s", env,
>+                         DEFAULT_XDG_DATA_HOME_LOGDIR);
>+            else if ((env = getenv("HOME")))
>+                snprintf(buf, sizeof(buf), "%s/%s/%s", env,
>+                         DEFAULT_XDG_DATA_HOME, DEFAULT_XDG_DATA_HOME_LOGDIR);
>+
>+            if (env) {
>+                xf86_mkdir_p(buf);
>+                strlcat(buf, "/" DEFAULT_LOGPREFIX, sizeof(buf));
>+                xf86LogFile = buf;
>+            }
>+        }
>         /* Append the display number and ".log" */
>         if (asprintf(&lf, "%s%%s" LOGSUFFIX, xf86LogFile) == -1)
>             FatalError("Cannot allocate space for the log file name\n");
>diff --git a/hw/xfree86/man/xorg.conf.man b/hw/xfree86/man/xorg.conf.man
>index 85f9f2e..6d2652e 100644
>--- a/hw/xfree86/man/xorg.conf.man
>+++ b/hw/xfree86/man/xorg.conf.man
>@@ -442,11 +442,15 @@ __modulepath__
> .TP 7
> .BI "LogFile \*q" path \*q
> sets the name of the Xorg server log file.
>-The default log file name is
>+The default log file name when running as root is
> .PP
> .RS 11
> .RI __logdir__/Xorg. <n> .log
> .RE
>+and for non root it is
>+.RS 11
>+.RI $XDG_DATA_HOME/xorg/Xorg. <n> .log
>+.RE
> .PP
> .RS 7
> where
>diff --git a/include/xorg-config.h.in b/include/xorg-config.h.in
>index 487d7ad..aa9f6d5 100644
>--- a/include/xorg-config.h.in
>+++ b/include/xorg-config.h.in
>@@ -48,6 +48,12 @@
> /* Path to server log file. */
> #undef DEFAULT_LOGPREFIX
> 
>+/* Default XDG_DATA dir under HOME */
>+#undef DEFAULT_XDG_DATA_HOME
>+
>+/* Default log dir under XDG_DATA_HOME */
>+#undef DEFAULT_XDG_DATA_HOME_LOGDIR
>+
> /* Building DRI-capable DDX. */
> #undef XF86DRI
> 
>-- 
>2.1.4
>

Actions: View | Diff

Attachments on bug 936842: 640536 | 640656 | 640780 | 641132 | 641618 | 641619 | 641620 | 641656 | 641703 | 641704 | 641714