X-Git-Url: https://git.tld-linux.org/?a=blobdiff_plain;f=TLD_Builder%2Frequest.py;h=7638bf2f73b8701d87cf5a1682d9e1eb2e213d9e;hb=99c8ec16ca741caf94419fa7976525744ce9aed1;hp=039cf76627cc7821edac766ec19a419a4d8e937a;hpb=92949e17161a53bfb4a60099e22b58ad63abce4c;p=tld-builder.git diff --git a/TLD_Builder/request.py b/TLD_Builder/request.py index 039cf76..7638bf2 100644 --- a/TLD_Builder/request.py +++ b/TLD_Builder/request.py @@ -11,6 +11,7 @@ import urllib import cgi import pytz import tempfile +import subprocess import util import log @@ -59,7 +60,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 +73,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 +98,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 +111,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" % ' '.join(self.flags)) f.write(" time: %s\n" % time.asctime(time.localtime(self.time))) for b in self.batches: b.dump(f) @@ -148,7 +149,7 @@ class Group: 'requester': escape(self.requester), 'priority': self.priority, 'max_jobs': self.max_jobs, - 'flags': string.join(self.flags) + 'flags': ' '.join(self.flags) }) f.write("
%sflags: [%s]" % (self.command, ' '.join(self.command_flags)) else: cmd = "/usr/bin/git ls-remote --heads git://git.tld-linux.org/packages/%s 1>/dev/null 2>&1" % (self.spec[:-5]) - r = call(cmd, shell=True) + r = subprocess.call(cmd, shell=True) if r == 0: package_url = "http://git.tld-linux.org/?p=packages/%(package)s.git;a=blob;f=%(spec)s;hb=%(branch)s" % { 'spec': urllib.quote(self.spec), @@ -378,8 +379,8 @@ class Batch: bld = lin_ar.split('-') tree_name = '-'.join(bld[:-1]) tree_arch = '-'.join(bld[-1:]) - link_pre = "" \ - % (urllib.quote(tree_name), urllib.quote(tree_arch), urllib.quote(bl_name), urllib.quote(rid)) + link_pre = "" \ + % (urllib.quote(tree_name), urllib.quote(tree_arch), urllib.quote(bl_name), urllib.quote(rid)) link_post = "" def ftime(s): @@ -401,7 +402,7 @@ class Batch: 'tooltip' : cgi.escape(tooltip, True), } + link_post) - f.write("%s]\n" % string.join(builders)) + f.write("%s]\n" % ' '.join(builders)) def rpmbuild_opts(self): """ @@ -441,7 +442,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 +502,7 @@ class Batch: