All kvm modules that can be used as stand alone programs were
updated to use #!/usr/bin/python instead of #!/usr/bin/env python,
complying with the rest of the autotest code base. As suggested
by Martin, common.py was added. With this, the stand alone
programs will be able to use the autotest library namespace and
choose the best python interpreter available in the system.

Signed-off-by: Lucas Meneghel Rodrigues <l...@redhat.com>
---
 client/tests/kvm/common.py           |    8 ++++++++
 client/tests/kvm/fix_cdkeys.py       |    3 ++-
 client/tests/kvm/kvm_config.py       |    4 +++-
 client/tests/kvm/make_html_report.py |    5 +++--
 client/tests/kvm/stepeditor.py       |    4 ++--
 client/tests/kvm/stepmaker.py        |    4 +++-
 6 files changed, 21 insertions(+), 7 deletions(-)
 create mode 100644 client/tests/kvm/common.py

diff --git a/client/tests/kvm/common.py b/client/tests/kvm/common.py
new file mode 100644
index 0000000..ce78b85
--- /dev/null
+++ b/client/tests/kvm/common.py
@@ -0,0 +1,8 @@
+import os, sys
+dirname = os.path.dirname(sys.modules[__name__].__file__)
+client_dir = os.path.abspath(os.path.join(dirname, "..", ".."))
+sys.path.insert(0, client_dir)
+import setup_modules
+sys.path.pop(0)
+setup_modules.setup(base_path=client_dir,
+                    root_module_name="autotest_lib.client")
diff --git a/client/tests/kvm/fix_cdkeys.py b/client/tests/kvm/fix_cdkeys.py
index 4f7a824..7a821fa 100755
--- a/client/tests/kvm/fix_cdkeys.py
+++ b/client/tests/kvm/fix_cdkeys.py
@@ -1,5 +1,6 @@
-#!/usr/bin/env python
+#!/usr/bin/python
 import shutil, os, sys
+import common
 
 """
 Program that replaces the CD keys present on a KVM autotest configuration file.
diff --git a/client/tests/kvm/kvm_config.py b/client/tests/kvm/kvm_config.py
index 40f16f1..13fdac2 100755
--- a/client/tests/kvm/kvm_config.py
+++ b/client/tests/kvm/kvm_config.py
@@ -1,4 +1,6 @@
+#!/usr/bin/python
 import re, os, sys, StringIO
+import common
 from autotest_lib.client.common_lib import error
 
 """
@@ -356,7 +358,7 @@ class config:
                 # (inside an exception or inside subvariants)
                 if restricted:
                     e_msg = "Using variants in this context is not allowed"
-                    raise error.AutotestError()
+                    raise error.AutotestError(e_msg)
                 if self.debug and not restricted:
                     self.__debug_print(indented_line,
                                      "Entering variants block (%d dicts in"
diff --git a/client/tests/kvm/make_html_report.py 
b/client/tests/kvm/make_html_report.py
index 6aed39e..e69367b 100755
--- a/client/tests/kvm/make_html_report.py
+++ b/client/tests/kvm/make_html_report.py
@@ -1,4 +1,7 @@
 #!/usr/bin/python
+import os, sys, re, getopt, time, datetime, commands
+import common
+
 """
 Script used to parse the test results and generate an HTML report.
 
@@ -7,8 +10,6 @@ Script used to parse the test results and generate an HTML 
report.
 @author: Dror Russo (dru...@redhat.com)
 """
 
-import os, sys, re, getopt, time, datetime, commands
-
 
 format_css="""
 html,body {
diff --git a/client/tests/kvm/stepeditor.py b/client/tests/kvm/stepeditor.py
index 9669200..e7794ac 100755
--- a/client/tests/kvm/stepeditor.py
+++ b/client/tests/kvm/stepeditor.py
@@ -1,6 +1,6 @@
-#!/usr/bin/env python
+#!/usr/bin/python
 import pygtk, gtk, os, glob, shutil, sys, logging
-import ppm_utils
+import common, ppm_utils
 pygtk.require('2.0')
 
 """
diff --git a/client/tests/kvm/stepmaker.py b/client/tests/kvm/stepmaker.py
index 2b7fd54..8f16ffd 100644
--- a/client/tests/kvm/stepmaker.py
+++ b/client/tests/kvm/stepmaker.py
@@ -1,8 +1,10 @@
-#!/usr/bin/env python
+#!/usr/bin/python
 import pygtk, gtk, gobject, time, os, commands
+import common
 from autotest_lib.client.common_lib import error
 import kvm_utils, logging, ppm_utils, stepeditor
 pygtk.require('2.0')
+
 """
 Step file creator/editor.
 
-- 
1.6.2.2

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to