From 692f0124a34625f3296d97496b721973e1f951a2 Mon Sep 17 00:00:00 2001
From: Michael Stapelberg <michael@stapelberg.de>
Date: Mon, 10 Oct 2011 20:29:44 +0100
Subject: [PATCH] =?UTF-8?q?Don=E2=80=99t=20use=20a=20default=20setting=20f?=
 =?UTF-8?q?or=20'order'?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This will make i3status depend on a useful configuration file (so
/etc/i3status.conf has to be installed) to actually output anything.

However, it fixes a very common error: people edit their i3status.conf and
don’t get that the first order directive has to use =, not +=. Since order is
now empty by default, you can use += everywhere.
---
 i3status.c       | 5 ++++-
 i3status.conf    | 2 +-
 man/i3status.man | 2 +-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/i3status.c b/i3status.c
index a02360b..ca07931 100644
--- a/i3status.c
+++ b/i3status.c
@@ -253,7 +253,7 @@ int main(int argc, char *argv[]) {
         };
 
         cfg_opt_t opts[] = {
-                CFG_STR_LIST("order", "{ipv6,\"run_watch DHCP\",\"wireless wlan0\",\"ethernet eth0\",\"battery 0\",\"cpu_temperature 0\",load,time}", CFGF_NONE),
+                CFG_STR_LIST("order", "{}", CFGF_NONE),
                 CFG_SEC("general", general_opts, CFGF_NONE),
                 CFG_SEC("run_watch", run_watch_opts, CFGF_TITLE | CFGF_MULTI),
                 CFG_SEC("wireless", wireless_opts, CFGF_TITLE | CFGF_MULTI),
@@ -307,6 +307,9 @@ int main(int argc, char *argv[]) {
         if (cfg_parse(cfg, configfile) == CFG_PARSE_ERROR)
                 return EXIT_FAILURE;
 
+        if (cfg_size(cfg, "order") == 0)
+                die("Your 'order' array is empty. Please fix your config.\n");
+
         cfg_general = cfg_getsec(cfg, "general");
         if (cfg_general == NULL)
                 die("Could not get section \"general\"\n");
diff --git a/i3status.conf b/i3status.conf
index 2c90a27..ee0db5c 100644
--- a/i3status.conf
+++ b/i3status.conf
@@ -3,7 +3,7 @@ general {
         interval = 5
 }
 
-order  = "ipv6"
+order += "ipv6"
 order += "disk /"
 order += "run_watch DHCP"
 order += "run_watch VPN"
diff --git a/man/i3status.man b/man/i3status.man
index f9bcebf..b1564ff 100644
--- a/man/i3status.man
+++ b/man/i3status.man
@@ -51,7 +51,7 @@ general {
         interval = 5
 }
 
-order  = "ipv6"
+order += "ipv6"
 order += "disk /"
 order += "run_watch DHCP"
 order += "run_watch VPN"