Force configrc parsing only if it exists... why is this so hard to remember to do?

1 files changed, 11 insertions, 10 deletions

diff --git a/cbc/environment.py b/cbc/environment.py
index d0778f0..2603af2 100644
--- a/cbc/environment.py
+++ b/cbc/environment.py
@@ -30,16 +30,7 @@ class Environment(object):
         if 'CBC_HOME' in self.environ:
             self.cbchome = self.environ['CBC_HOME']
 
-        if os.path.exists(self.rcpath):
-            if os.path.isfile(self.rcpath):
-                self.configrc.read(self.rcpath)
-
-        if 'settings' in self.configrc.sections():
-            if 'path' in self.configrc['settings']:
-                self.cbchome = self.configrc['settings']['path']
-                if not self.cbchome:
-                    raise IncompleteEnv('.cbcrc empty path detected. Check: settings -> path')
-
+        # A few hard-coded defaults pertaining to the seldom-used internal web server
         self.configrc['cbc_cgi'] = {}
         self.configrc['cbc_cgi']['local_server'] = 'true'
         self.configrc['cbc_cgi']['local_port'] = '8888'
@@ -47,6 +38,16 @@ class Environment(object):
         self.configrc['cbc_cgi']['protocol'] = 'http'
         self.configrc['cbc_cgi']['url'] = '{0}://localhost:{1}'.format(self.configrc['cbc_cgi']['protocol'], self.configrc['cbc_cgi']['local_port'])
 
+        if os.path.exists(self.rcpath):
+            if os.path.isfile(self.rcpath):
+                self.configrc.read(self.rcpath)
+
+            if 'settings' in self.configrc.sections():
+                if 'path' in self.configrc['settings']:
+                    self.cbchome = self.configrc['settings']['path']
+                    if not self.cbchome:
+                        raise IncompleteEnv('.cbcrc empty path detected. Check: settings -> path')
+
         if self.cbchome is None:
             raise IncompleteEnv('CBC_HOME is undefined.')