Implement option parsing (-c <configfile>)

diff --git a/man/i3.man b/man/i3.man
index fc62ef37b98fae35e9e44b5b3de13288afc5e884..789862c53f5caf15d312ae7f374daab249f06759 100644 (file)
--- a/man/i3.man
+++ b/man/i3.man
@@ -9,7 +9,7 @@ i3 - an improved dynamic, tiling window manager
 
 == SYNOPSIS
 
-i3
+i3 [-c configfile]
 
 == DESCRIPTION
 
@@ -101,11 +101,12 @@ Enable default layout for the current container.
 
 == FILES
 
-=== i3.config
+=== ~/.i3/config
 
-When starting, i3 looks for i3.config in the current working directory and loads the
-configuration. At the moment, you can specify only the path to your favorite terminal
-emulator, the font and keybindings.
+When starting, i3 looks for ~/.i3/config and loads the configuration. If ~/.i3/config is not found,
+i3 tries /etc/i3/config. You can specify a custom path using the -c option.
+
+At the moment, you can specify only the path to your favorite terminal emulator, the font and keybindings.
 
 At the moment, you have to bind to keycodes (find them out via xev(1)).
 
@@ -182,7 +183,12 @@ export LC_TELEPHONE=de_DE.ISO8859-15
 export LC_MEASUREMENT=de_DE.ISO8859-15
 export LC_IDENTIFICATION=de_DE.ISO8859-15
 
-exec /usr/bin/i3
+# Enable core dumps in case something goes wrong
+ulimit -c unlimited
+
+# Start i3 and log to ~/.i3/logfile
+echo "Starting at $(date)" >> ~/.i3/logfile
+exec /usr/bin/i3 >> ~/.i3/logfile
 -------------------------------------------------------------
 
 == TODO
@@ -191,7 +197,6 @@ There is lots of stuff left to do. This release is to be considered as a technol
 Here is an overwiew of the most important points:
 
  * IPC
- * a command for toggling layouts/workspaces
  * floating
  * do something about applications which don’t use _NET_WM_STATE_FULLSCREEN (like xpdf)
 

diff --git a/src/mainx.c b/src/mainx.c
index 32dc77e5af4577411c97a9b617c2935958b5b23f..9927af6febd608b2963848209dd9a0dea2a2d7f2 100644 (file)
--- a/src/mainx.c
+++ b/src/mainx.c
@@ -18,11 +18,10 @@
 #include <assert.h>
 #include <limits.h>
 
-#include <xcb/xcb.h>
-
 #include <X11/XKBlib.h>
 #include <X11/extensions/XKB.h>
 
+#include <xcb/xcb.h>
 #include <xcb/xcb_wm.h>
 #include <xcb/xcb_aux.h>
 #include <xcb/xcb_event.h>
@@ -30,18 +29,18 @@
 #include <xcb/xcb_keysyms.h>
 #include <xcb/xcb_icccm.h>
 #include <xcb/xinerama.h>
-#include "data.h"
 
 #include "config.h"
-#include "queue.h"
-#include "table.h"
-#include "layout.h"
+#include "data.h"
 #include "debug.h"
 #include "handlers.h"
+#include "i3.h"
+#include "layout.h"
+#include "queue.h"
+#include "table.h"
 #include "util.h"
 #include "xcb.h"
 #include "xinerama.h"
-#include "i3.h"
 
 /* This is the path to i3, copied from argv[0] when starting up */
 char *application_path;
@@ -282,7 +281,8 @@ void manage_existing_windows(xcb_connection_t *conn, xcb_property_handlers_t *pr
 }
 
 int main(int argc, char *argv[], char *env[]) {
-        int i, screens;
+        int i, screens, opt;
+        char *override_configpath = NULL;
         xcb_connection_t *conn;
         xcb_property_handlers_t prophs;
         xcb_window_t root;
@@ -294,6 +294,17 @@ int main(int argc, char *argv[], char *env[]) {
 
         application_path = sstrdup(argv[0]);
 
+        while ((opt = getopt(argc, argv, "c:")) != -1) {
+                switch (opt) {
+                        case 'c':
+                                override_configpath = sstrdup(optarg);
+                                break;
+                        default:
+                                fprintf(stderr, "Usage: %s [-c configfile]\n", argv[0]);
+                                exit(EXIT_FAILURE);
+                }
+        }
+
         /* Initialize the table data structures for each workspace */
         init_table();
 
@@ -303,7 +314,7 @@ int main(int argc, char *argv[], char *env[]) {
         byChild = alloc_table();
         byParent = alloc_table();
 
-        load_configuration(NULL);
+        load_configuration(override_configpath);
 
         conn = xcb_connect(NULL, &screens);