massfile: Fix wrong variable name and make pylint a little bit happier.

This commit is contained in:
Benjamin Drung 2010-12-23 12:17:20 +01:00
parent 121bc50341
commit 2279efb2cf

View File

@ -29,7 +29,9 @@ import sys
import email import email
from ubuntutools.config import UDTConfig from ubuntutools.config import UDTConfig
from ubuntutools.lp.libsupport import get_launchpad, translate_api_web, translate_web_api from ubuntutools.lp.libsupport import (get_launchpad,
translate_api_web,
translate_web_api)
def read_config(): def read_config():
instructions_file = open("instructions") instructions_file = open("instructions")
@ -84,14 +86,18 @@ def file_bug(config, launchpad):
summary = config["subject"].replace("$pack", config["sourcepackage"]) summary = config["subject"].replace("$pack", config["sourcepackage"])
description = config["text"].replace("$pack", config["sourcepackage"]) description = config["text"].replace("$pack", config["sourcepackage"])
product_url = "%subuntu/+source/%s" %(launchpad._root_uri, config["sourcepackage"]) product_url = "%subuntu/+source/%s" % \
tags = filter(None, map(lambda t: t.strip("\n").strip(), config["tags"].split(","))) (launchpad._root_uri, config["sourcepackage"])
tags = filter(None, map(lambda t: t.strip("\n").strip(),
config["tags"].split(",")))
bug = launchpad.bugs.createBug(description=description, title=summary, bug = launchpad.bugs.createBug(description=description, title=summary,
target=product_url, tags=tags) target=product_url, tags=tags)
print "Successfully filed bug %i: %s" %(bug.id, translate_api_web(bug.self_link)) print "Successfully filed bug %i: %s" % \
(bug.id, translate_api_web(bug.self_link))
subscribers = filter(None, map(lambda t: t.strip("\n").strip(), config["subscribers"].split(","))) subscribers = filter(None, map(lambda t: t.strip("\n").strip(),
config["subscribers"].split(",")))
for sub in subscribers: for sub in subscribers:
subscribe_url = "%s~%s" % (launchpad._root_uri, sub) subscribe_url = "%s~%s" % (launchpad._root_uri, sub)
bug.subscribe(person=subscribe_url) bug.subscribe(person=subscribe_url)
@ -111,14 +117,16 @@ def file_bug(config, launchpad):
task.assignee = assignee_url task.assignee = assignee_url
task.lp_save() task.lp_save()
except: except:
"Bug for '%s' was not filed." % config["sourcepackage"] print >> sys.stderr, "Bug for '%s' was not filed." % \
config["sourcepackage"]
def read_buglist(url, launchpad): def read_buglist(url, launchpad):
if not url: if not url:
return set() return set()
if len(url.split("?", 1)) == 2: if len(url.split("?", 1)) == 2:
# search options not supported, because there is no mapping web ui options <-> API options # search options not supported, because there is no mapping
# web ui options <-> API options
print >> sys.stderr, "Options in url are not supported, url: %s" % url print >> sys.stderr, "Options in url are not supported, url: %s" % url
sys.exit(1) sys.exit(1)
@ -148,7 +156,7 @@ def main():
p.add_option('--no-conf', p.add_option('--no-conf',
dest='no_conf', default=False, action='store_true', dest='no_conf', default=False, action='store_true',
help="Don't read config files or environment variables") help="Don't read config files or environment variables")
opts, args = p.parse_args() options, args = p.parse_args()
udtconfig = UDTConfig(options.no_conf) udtconfig = UDTConfig(options.no_conf)
if options.lpinstance is None: if options.lpinstance is None:
options.lpinstance = udtconfig.get_value('LPINSTANCE') options.lpinstance = udtconfig.get_value('LPINSTANCE')
@ -169,4 +177,3 @@ def main():
if __name__ == '__main__': if __name__ == '__main__':
main() main()