Stupidest bug in history.
authorStephen Soltesz <soltesz@cs.princeton.edu>
Thu, 19 Jun 2008 19:12:02 +0000 (19:12 +0000)
committerStephen Soltesz <soltesz@cs.princeton.edu>
Thu, 19 Jun 2008 19:12:02 +0000 (19:12 +0000)
rstrip is a character oriented function, that nukes all characters in the
argument, therefore, an attribute named "comon_exec.acl" will strip all the
right most, 'a', 'c', 'l', and '.' characters.  The result is 'comon_exe' b/c
of the trailing 'c'.

Ugg.

vsys.py

diff --git a/vsys.py b/vsys.py
index a429760..a2a6b9f 100644 (file)
--- a/vsys.py
+++ b/vsys.py
@@ -61,7 +61,7 @@ def touchAcls():
     for (root, dirs, files) in os.walk(VSYSBKEND):
         for file in files:
             if file.endswith(".acl"):
-                acls.append(file.rstrip(".acl"))
+                acls.append(file.replace(".acl", ""))
             else:
                 scripts.append(file)
     for new in (Set(scripts) - Set(acls)):
@@ -101,7 +101,7 @@ def parseAcls():
         for file in files:
             if file.endswith(".acl"):
                 f = open(root+"/"+file,"r+")
-                scriptname = file.rstrip(".acl")
+                scriptname = file.replace(".acl", "")
                 scriptacls[scriptname] = []
                 for slice in f.readlines():  
                     scriptacls[scriptname].append(slice.rstrip())