well its disputed: http://en.wikipedia.org/wiki/Serial_comma
its probably better to reformat the message instead, e.g:
>sponsor-patch: Error: Unsupported builder specified: gsgs.
>sponsor-patch: Error: Supported builders: cowbuilder, cowbuilder-dist,
>pbuilder, pbuilder-dist, sbuild
maybe also get rid of the long if clause in get_builder too?
e.g. like this:
supported = {
"cowbuilder": lambda: Pbuilder("cowbuilder"),
"cowbuilder-dist": lambda: Pbuilderdist("cowbuilderdist"),
"pbuilder": lambda: Pbuilder(),
"pbuilder-dist": lambda: Pbuilderdist(),
"sbuild": lambda: Sbuild()
}
if builder in supported:
return supported[builder]()
Logger.error("Unsupported builder specified: %s." % builder)
Logger.error("Supported builders: %s" % ", ".join(sorted(supported.keys())))
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/728751
Title:
[sponsor-patch] support cowbuilder and cowbuilder-dist
--
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
