From c7749aeaeafa217c838b65a3848548647aa06e89 Mon Sep 17 00:00:00 2001
From: Daniel Narvaez <dwnarvaez@gmail.com>
Date: Wed, 28 Nov 2012 01:47:32 +0000
Subject: Generalize a bit how the test command is run

We should be able to use the same logic for other services.
---
diff --git a/commands/common.py b/commands/common.py
index a5dd056..b26e192 100644
--- a/commands/common.py
+++ b/commands/common.py
@@ -13,16 +13,22 @@ from devbot import distro
 def setup():
     config.load_plugins()
 
+    relocatable = "SUGAR_BUILDBOT" in os.environ
+    logs_dir = os.path.join(base_dir, "logs")
+    install_dir = os.path.join(base_dir, "install")
+
     config.set_devbot_dir(os.path.join(base_dir, "devbot"))
     config.set_config_dir(os.path.join(base_dir, "config"))
-    config.set_install_dir(os.path.join(base_dir, "install"),
-                           relocatable="SUGAR_BUILDBOT" in os.environ)
+    config.set_install_dir(install_dir, relocatable=relocatable)
     config.set_source_dir(os.path.join(base_dir, "source"))
     config.set_build_dir(os.path.join(base_dir, "build"))
     config.set_commands_dir(os.path.join(base_dir, "commands"))
-    config.set_logs_dir(os.path.join(base_dir, "logs"))
     config.set_home_dir(os.path.join(base_dir, "home"))
     config.set_prefs_path(os.path.join(base_dir, "prefs"))
+    config.set_logs_dir(logs_dir)
+
+    os.environ["SB_LOGS_DIR"] = logs_dir
+    os.environ["SB_LIBEXEC_DIR"] = config.libexec_dir
 
     version = distro.get_distro_info().system_version
 
diff --git a/commands/helpers/test b/commands/helpers/test
index a2ea227..4037a0d 100755
--- a/commands/helpers/test
+++ b/commands/helpers/test
@@ -1,9 +1,8 @@
 #!/bin/sh
 
-helpersdir=$SUGAR_BUILD_DIR/install/libexec/sugar-runner
-testsdir=$SUGAR_BUILD_DIR/tests
-logsdir=$SUGAR_BUILD_DIR/logs
-display=`$helpersdir/find-free-display`
+libexecdir=$SB_LIBEXEC_DIR
+logsdir=$SB_LOGS_DIR
+display=`$libexecdir/sugar-runner/find-free-display`
 resolution=1024x768x16
 
 xvfb=`which Xvfb`
@@ -17,12 +16,12 @@ export SUGAR_LOGGER_LEVEL=debug
 export GTK_MODULES=gail:atk-bridge
 export SUGAR_PROFILE=uitests
 
-xinit $helpersdir/xinitrc -- $xserver &
+xinit $libexecdir/sugar-runner/xinitrc -- $xserver &
 xinitpid=$!
 
 sleep 5
 
-DISPLAY=$display python -u $testsdir/shell.py
+DISPLAY=$display python -u $1
 result=$?
 
 kill $xinitpid
diff --git a/commands/test b/commands/test
index ff4f107..a0707e0 100755
--- a/commands/test
+++ b/commands/test
@@ -9,6 +9,5 @@ from devbot import config
 
 common.setup()
 
-os.environ["SUGAR_BUILD_DIR"] = os.path.join(common.base_dir)
-
-run.run([os.path.join(common.helpers_dir, "test")])
+run.run([os.path.join(common.helpers_dir, "test"),
+         os.path.join(common.base_dir, "tests", "shell.py")])
diff --git a/devbot/config.py b/devbot/config.py
index 7b9978c..a264ccb 100644
--- a/devbot/config.py
+++ b/devbot/config.py
@@ -18,6 +18,7 @@ lib_dir = None
 share_dir = None
 bin_dir = None
 etc_dir = None
+libexec_dir = None
 home_dir = None
 dep_files = None
 module_files = None
@@ -113,6 +114,7 @@ def set_install_dir(dir, relocatable=False):
     global bin_dir
     global etc_dir
     global lib_dir
+    global libexec_dir
 
     install_dir = dir
     _ensure_dir(install_dir)
@@ -125,6 +127,7 @@ def set_install_dir(dir, relocatable=False):
 
     bin_dir = os.path.join(prefix_dir, "bin")
     etc_dir = os.path.join(prefix_dir, "etc")
+    libexec_dir = os.path.join(prefix_dir, "libexec")
 
     if distro.get_distro_info().use_lib64:
         lib_dir = os.path.join(prefix_dir, "lib64")
--
cgit v0.9.1