mirror of
https://github.com/mozilla/gecko-dev.git
synced 2025-02-03 04:27:41 +00:00
Bug 1083279 - Follow-up: address review comments. r=gbrown
Pushed without committing the changes addressing gbrown's review comments. --HG-- extra : rebase_source : 4df8be9befc251aa8652baa62da81d1637a6a86d
This commit is contained in:
parent
de50edfb55
commit
e2aa678126
@ -622,6 +622,12 @@ class ReftestOptions(OptionParser):
|
||||
self.add_option("--appname",
|
||||
action = "store", type = "string", dest = "app",
|
||||
help = "absolute path to application, overriding default")
|
||||
# Certain paths do not make sense when we're cross compiling Fennec. This
|
||||
# logic is cribbed from the example in
|
||||
# python/mozbuild/mozbuild/mach_commands.py.
|
||||
defaults['appname'] = build_obj.get_binary_path() if \
|
||||
build_obj and build_obj.substs['MOZ_BUILD_APP'] != 'mobile/android' else None
|
||||
|
||||
self.add_option("--extra-profile-file",
|
||||
action = "append", dest = "extraProfileFiles",
|
||||
default = [],
|
||||
@ -641,6 +647,8 @@ class ReftestOptions(OptionParser):
|
||||
action = "store", type = "string", dest = "utilityPath",
|
||||
help = "absolute path to directory containing utility "
|
||||
"programs (xpcshell, ssltunnel, certutil)")
|
||||
defaults["utilityPath"] = build_obj.bindir if \
|
||||
build_obj and build_obj.substs['MOZ_BUILD_APP'] != 'mobile/android' else None
|
||||
|
||||
self.add_option("--total-chunks",
|
||||
type = "int", dest = "totalChunks",
|
||||
@ -722,14 +730,6 @@ class ReftestOptions(OptionParser):
|
||||
dest = "extraPrefs", metavar = "PREF=VALUE",
|
||||
help = "defines an extra user preference")
|
||||
|
||||
|
||||
# Certain paths do not make sense when we're cross compiling Fennec. This
|
||||
# logic is cribbed from the example in
|
||||
# python/mozbuild/mozbuild/mach_commands.py.
|
||||
if build_obj and build_obj.substs['MOZ_BUILD_APP'] != 'mobile/android':
|
||||
defaults['appname'] = build_obj.get_binary_path() if build_obj else None
|
||||
defaults["utilityPath"] = build_obj.bindir if build_obj else None
|
||||
|
||||
self.set_defaults(**defaults)
|
||||
|
||||
def verifyCommonOptions(self, options, reftest):
|
||||
|
Loading…
x
Reference in New Issue
Block a user