From 15d0a4b27a0ebc29c9c518bed5b10130f8d23ce6 Mon Sep 17 00:00:00 2001 From: Armen Zambrano Gasparnian Date: Fri, 24 Jul 2015 11:03:03 -0400 Subject: [PATCH] Bug 1186987 - If file_path is None do not check if it is an executable. DONTBUILD. r=jlund --- testing/mozharness/mozharness/mozilla/vcstools.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/testing/mozharness/mozharness/mozilla/vcstools.py b/testing/mozharness/mozharness/mozilla/vcstools.py index 1259cabdd831..c7349291539b 100644 --- a/testing/mozharness/mozharness/mozilla/vcstools.py +++ b/testing/mozharness/mozharness/mozilla/vcstools.py @@ -49,10 +49,9 @@ class VCSToolsScript(VCSScript): if type(self.query_exe(vcs_tool)) is list: continue - if not self.is_exe(file_path): - self.critical("%s is not executable." % file_path) - if file_path is None: self.fatal("This machine is missing %s, if this is your " "local machine you can use --cfg " "developer_config.py" % vcs_tool) + elif not self.is_exe(file_path): + self.critical("%s is not executable." % file_path)