[clfswm-devel] [PATCH] Force XDG_CONFIG_HOME first

[Xavier Maillard]

Hi,

Included is a rather small change I would like to submit. This
patch just change the check order when trying to detect the
configuration file to use.

So it is a one-line patch that would not hurt anything.

I think CLFSWM should first respect freedesktop.org
specifications before failing back to "old habits" way. I am
pleased to see you added support for the XDG_CONFIG_HOME
standard, but now has come the time to just switch to it.

[I already updated the wiki :)]

Here it is:

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 19)
+++ ChangeLog	(working copy)
@@ -1,3 +1,9 @@
+2008-03-08  Xavier Maillard  <xma at gnu.org>
+
+	* clfswm.lisp (read-conf-file): Check for the user config file in
+	XDG_CONFIG_HOME *first*. Freedesktop.org standards should be
+	prefered whenever possible.
+
 2008-02-27  Philippe Brochard  <hocwp at free.fr>
 
 	* clfswm-layout.lisp (*-layout): Add an optional raise-p
Index: clfswm.lisp
===================================================================
--- clfswm.lisp	(revision 19)
+++ clfswm.lisp	(working copy)
@@ -247,7 +247,7 @@
 	 (etc-conf (probe-file #p"/etc/clfswmrc"))
 	 (config-user-conf (probe-file (make-pathname :directory (append (xdg-config-home) '("clfswm"))
 						      :name "clfswmrc")))
-	 (conf (or user-conf etc-conf config-user-conf)))
+	 (conf (or config-user-conf user-conf etc-conf)))
     (if conf
 	(handler-case (load conf)
 	  (error (c)
@@ -285,7 +285,6 @@
     (ungrab-main-keys)
     (xlib:destroy-window *no-focus-window*)
     (xlib:close-display *display*)))
-      


Regards, 

	Xavier
-- 
http://www.gnu.org
http://www.april.org
http://www.lolica.org