added comment on missing version number in bootcd image names

[plcapi.git] / PLC / Methods / GetBootMedium.py

diff --git a/PLC/Methods/GetBootMedium.py b/PLC/Methods/GetBootMedium.py
index f050840..ba432c3 100644 (file)
--- a/PLC/Methods/GetBootMedium.py
+++ b/PLC/Methods/GetBootMedium.py
@@ -273,6 +273,8 @@ class GetBootMedium(Method):
 
         return (pldistro,fcdistro,arch)
 
+    # xxx Thierry : 5.2.1 build/version.txt for some reason is empty, that's why 
+    # the weird name with downloaded image filenames
     def bootcd_version (self):
         try:
             return file(self.BOOTCDDIR + "/build/version.txt").readline().strip()
@@ -317,7 +319,7 @@ class GetBootMedium(Method):
             # and this leads to the old file sitting in there forever
             # so, if the file is older than 5 minutes, we just trash
             grace=5
-            if os.path.exists(filename) and os.path.getmtime(filename)-time.time() >= (grace*60):
+            if os.path.exists(filename) and (time.time()-os.path.getmtime(filename)) >= (grace*60):
                 os.unlink(filename)
             if os.path.exists(filename):
                 raise PLCInvalidArgument, "Resulting file %s already exists - please try again in %d minutes"%\