# HG changeset patch # User Jeff Hammel # Date 1296689434 28800 # Node ID c1c804aae3486f44a04e365f7a5299d4816aa0af # Parent 38fc07fb6aad08f9f98ee5dba184a1b8d6061f13 use a tuple not iterate over a string * also, try to find the firefox binary diff -r 38fc07fb6aad -r c1c804aae348 autobot/config.py --- a/autobot/config.py Wed Feb 02 15:15:38 2011 -0800 +++ b/autobot/config.py Wed Feb 02 15:30:34 2011 -0800 @@ -48,7 +48,7 @@ # get platform information platform = {} - for key in 'os': + for key in ('os',): if key in slave_dict: platform[key] = slave_dict.pop(key) else: diff -r 38fc07fb6aad -r c1c804aae348 autobot/process/factory.py --- a/autobot/process/factory.py Wed Feb 02 15:15:38 2011 -0800 +++ b/autobot/process/factory.py Wed Feb 02 15:30:34 2011 -0800 @@ -29,7 +29,7 @@ def __init__(self, platform, base_url=None): # must have the os specified! - assert platform.get('os'), "platform['os'] must be one of (linux, win, mac)" + assert platform.get('os'), "platform['os'] must be one of (linux, win, mac); you have %s" % platform # determine the (damn) url script = 'get-latest-tinderbox' @@ -69,6 +69,10 @@ WithProperties('%(firefox_bundle)s')], haltOnFailure=True )) + self.addStep(SetProperty(property='firefox', + command=['python', '-c', + 'import os; print os.path.abspath("firefox/firefox")'])) + else: raise NotImplementedError("FirefoxDownloader doesn't know how to work with your os: %s" % platform['os'])