- more python 3.x fixes

[tld-builder.git] / TLD_Builder / request.py

diff --git a/TLD_Builder/request.py b/TLD_Builder/request.py
index 4ce68e33093d5f20ee430487c44eb0840b415c5e..efedbb96fb23fc59f3c3f8b55a5b6aaa9f77b512 100644 (file)
--- a/TLD_Builder/request.py
+++ b/TLD_Builder/request.py
@@ -59,7 +59,7 @@ def iso8601(ts, timezone='UTC'):
     return dt.isoformat()
 
 def is_blank(e):
-    return e.nodeType == Element.TEXT_NODE and string.strip(e.nodeValue) == ""
+    return e.nodeType == Element.TEXT_NODE and e.nodeValue.strip() == ""
 
 class Group:
     def __init__(self, e):
@@ -72,7 +72,7 @@ class Group:
         self.requester = ""
         self.max_jobs = 0
         self.requester_email = ""
-        self.flags = string.split(attr(e, "flags", ""))
+        self.flags = attr(e, "flags", "").split()
         for c in e.childNodes:
             if is_blank(c): continue
 
@@ -97,7 +97,7 @@ class Group:
             deps = []
             m[b.b_id] = b
             for dep in b.depends_on:
-                if m.has_key(dep):
+                if dep in m:
                     # avoid self-deps
                     if id(m[dep]) != id(b):
                         deps.append(m[dep])
@@ -110,7 +110,7 @@ class Group:
     def dump(self, f):
         f.write("group: %d (id=%s pri=%d)\n" % (self.no, self.id, self.priority))
         f.write("  from: %s\n" % self.requester)
-        f.write("  flags: %s\n" % string.join(self.flags))
+        f.write("  flags: %s\n" % self.flags.join())
         f.write("  time: %s\n" % time.asctime(time.localtime(self.time)))
         for b in self.batches:
             b.dump(f)
@@ -148,7 +148,7 @@ class Group:
             'requester': escape(self.requester),
             'priority': self.priority,
             'max_jobs': self.max_jobs,
-            'flags': string.join(self.flags)
+            'flags': self.flags.join()
         })
         f.write("<ol>\n")
         for b in self.batches:
@@ -162,7 +162,7 @@ class Group:
          <requester email='%s'>%s</requester>
          <time>%d</time>
          <priority>%d</priority>
-         <maxjobs>%d</maxjobs>\n""" % (self.id, self.no, string.join(self.flags),
+         <maxjobs>%d</maxjobs>\n""" % (self.id, self.no, self.flags.join(),
                     escape(self.requester_email), escape(self.requester),
                     self.time, self.priority, self.max_jobs))
         for b in self.batches:
@@ -206,7 +206,7 @@ class Batch:
         self.skip = []
         self.gb_id = ""
         self.b_id = attr(e, "id")
-        self.depends_on = string.split(attr(e, "depends-on"))
+        self.depends_on = attr(e, "depends-on").split()
         self.upgraded = True
 
         self.parse_xml(e)
@@ -233,7 +233,7 @@ class Batch:
             elif c.nodeName == "command":
                 self.spec = "COMMAND"
                 self.command = text(c).strip()
-                self.command_flags = string.split(attr(c, "flags", ""))
+                self.command_flags = attr(c, "flags", "").split()
             elif c.nodeName == "info":
                 self.info = text(c)
             elif c.nodeName == "kernel":
@@ -294,7 +294,7 @@ class Batch:
         builders = []
         for b in self.builders:
             builders.append("%s:%s" % (b, self.builders_status[b]))
-        f.write("    builders: %s\n" % string.join(builders))
+        f.write("    builders: %s\n" % builders.join())
 
     def is_command(self):
         return self.command != ""
@@ -401,7 +401,7 @@ class Batch:
                     'tooltip' : cgi.escape(tooltip, True),
             }
             + link_post)
-        f.write("%s]</small></li>\n" % string.join(builders))
+        f.write("%s]</small></li>\n" % builders.join())
 
     def rpmbuild_opts(self):
         """
@@ -441,7 +441,7 @@ class Batch:
         ignores = []
 
         # add php version based ignores
-        if self.defines.has_key('php_suffix'):
+        if 'php_suffix' in self.defines:
             # current version if -D php_suffix is present
             php_version = php_name_to_ver(self.defines['php_suffix'])
         else:
@@ -501,7 +501,7 @@ class Batch:
            <spec>%s</spec>
            <branch>%s</branch>
            <info>%s</info>\n""" % (self.b_id,
-                 string.join(list(map(lambda (b): b.b_id, self.depends_on))),
+                 list(map(lambda b: b.b_id, self.depends_on)).join(),
                  escape(self.src_rpm),
                  escape(' '.join(self.command_flags)), escape(self.command),
                  escape(self.spec), escape(self.branch), escape(self.info)))
@@ -516,7 +516,7 @@ class Batch:
         if self.defines:
             f.write("           %s\n" % self.defines_xml())
         for b in self.builders:
-            if self.builders_status_buildtime.has_key(b):
+            if b in self.builders_status_buildtime:
                 t = self.builders_status_buildtime[b]
             else:
                 t = "0"
@@ -578,7 +578,7 @@ class Notification:
         for r in q.requests:
             if r.kind == "group":
                 for b in r.batches:
-                    if self.batches.has_key(b.b_id):
+                    if b.b_id in self.batches:
                         b.builders_status[self.builder] = self.batches[b.b_id]
                         b.builders_status_time[self.builder] = time.time()
                         b.builders_status_buildtime[self.builder] = "0" #self.batches_buildtime[b.b_id]