And the actual grubby path set is not valid.
Signed-off-by: Cleber Rosa <[email protected]>
---
client/tools/boottool.py | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/client/tools/boottool.py b/client/tools/boottool.py
index 41ca1db..299593f 100755
--- a/client/tools/boottool.py
+++ b/client/tools/boottool.py
@@ -834,6 +834,16 @@ class Grubby(object):
args = []
args.append(self.path)
+ if self.path is None:
+ self.log.error('grubby executable currently set to None. this is '
+ 'a serious error condition')
+
+ if self.path is not None and not os.path.exists(self.path):
+ self.log.error('grubby executable does not exist: "%s"', self.path)
+ if not os.access(self.path, os.R_OK | os.X_OK):
+ self.log.error('insufficient permissions (read and execute) '
+ 'for grubby executable: "%s"', self.path)
+
# If a bootloader has been detected, that is, a mode has been set,
# it's passed as the first command line argument to grubby
if include_bootloader and self.bootloader is not None:
--
1.7.10.4
_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
