Update the variable after, not before, printing the warning, otherwise
the warning is bogus ("dir-template consists of more sub-paths (3)
than scan-dirs (3)", doh).  Also fix s/he/it/ - jack is not a person.

--- jack_checkopts.py~  2006-01-23 14:39:36.000000000 +0000
+++ jack_checkopts.py   2006-01-23 14:40:50.000000000 +0000
@@ -90,8 +90,8 @@
 
     # check dir_template and scan_dirs
     if len(cf['_dir_template'].split(os.path.sep)) > cf['_scan_dirs']:
+        warning("dir-template consists of more sub-paths (%i) than scan-dirs 
(%i). Jack may not find the workdir next time it is run. (Auto-raised)" % 
(len(cf['_dir_template'].split(os.path.sep)), cf['_scan_dirs']))
         cf.rupdate({'scan_dirs': {'val': 
len(cf['_dir_template'].split(os.path.sep))}}, "check")
-        warning("dir-template consists of more sub-paths (%i) than scan-dirs 
(%i). Jack may not find the workdir next time he is run. (Auto-raised)" % 
(len(cf['_dir_template'].split(os.path.sep)), cf['_scan_dirs']))
 
     # check for unsername
     if cf['username']['val'] == None:

-- 
Martin Michlmayr
http://www.cyrius.com/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to