Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1905)

Unified Diff: tests/lab/runner_constants.py

Issue 344013: Change from absolute paths to relative paths to allow tests,etc. to be locate... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/o3d/
Patch Set: Created 11 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/lab/run_lab_test.py ('k') | tests/lab/runner_util.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/lab/runner_constants.py
===================================================================
--- tests/lab/runner_constants.py (revision 30208)
+++ tests/lab/runner_constants.py (working copy)
@@ -29,7 +29,8 @@
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-"""Defines common O3D test runner constants.
+"""Defines common O3D test runner constants. This file determines paths to
+other O3D components relatively, so it must be placed in the right location.
"""
@@ -40,55 +41,57 @@
join = os.path.join
-if util.IsWindows():
- AUTO_PATH = r'C:\auto'
- PYTHON = r'C:\Python24\python.exe'
- if util.IsXP():
- HOME_PATH = r'C:\Documents and Settings\testing'
- else:
- HOME_PATH = r'C:\Users\testing'
+# Make sure OS is supported.
+if not util.IsWindows() and not util.IsMac() and not util.IsLinux():
+ print 'Only Windows, Mac, and Linux are supported.'
+ sys.exit(1)
-elif util.IsMac():
- AUTO_PATH = '/Users/testing/auto'
- PYTHON = 'python'
- HOME_PATH = '/Users/testing'
+# This path should be root/o3d/tests.
+TEST_PATH = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
+# This path should be root/o3d.
+O3D_PATH = os.path.dirname(TEST_PATH)
+# This path should be root, i.e., the checkout location.
+BASE_PATH = os.path.dirname(O3D_PATH)
-elif util.IsLinux():
- AUTO_PATH = '/home/testing/auto'
- PYTHON = 'python'
- HOME_PATH = '/home/testing'
-else:
- print 'Only Windows, Mac, and Linux are supported.'
+HOME = os.path.expanduser('~')
+if HOME == '~':
+ print 'Cannot find user home directory.'
sys.exit(1)
-O3D_PATH = join(AUTO_PATH, 'o3d')
-SCRIPTS_PATH = join(AUTO_PATH, 'scripts')
-RESULTS_PATH = join(AUTO_PATH, 'results')
-SOFTWARE_PATH = join(AUTO_PATH, 'software')
+if util.IsWindows():
+ PYTHON = r'C:\Python24\python.exe'
+else:
+ PYTHON = 'python'
+# Note: this path may or may not exist.
+RESULTS_PATH = join(TEST_PATH, 'results')
+
# Build directories.
if util.IsWindows():
- BUILD_PATH = join(O3D_PATH, 'o3d', 'build')
+ BUILD_PATH = join(O3D_PATH, 'build')
elif util.IsMac():
- BUILD_PATH = join(O3D_PATH, 'xcodebuild')
+ BUILD_PATH = join(BASE_PATH, 'xcodebuild')
else:
- BUILD_PATH = join(O3D_PATH, 'sconsbuild')
+ BUILD_PATH = join(BASE_PATH, 'sconsbuild')
if os.path.exists(join(BUILD_PATH, 'Debug')):
PRODUCT_DIR_PATH = join(BUILD_PATH, 'Debug')
+elif os.path.exists(join(BUILD_PATH, 'Release')):
+ PRODUCT_DIR_PATH = join(BUILD_PATH, 'Release')
else:
- PRODUCT_DIR_PATH = join(BUILD_PATH, 'Release')
+ print 'Cannot find Debug or Release folder in ' + BUILD_PATH
+ sys.exit(1)
# Plugin locations.
INSTALL_PATHS = []
if util.IsWindows():
- INSTALL_PATHS += [join(HOME_PATH, 'Application Data', 'Mozilla',
+ INSTALL_PATHS += [join(HOME, 'Application Data', 'Mozilla',
'plugins', 'npo3dautoplugin.dll')]
- INSTALL_PATHS += [join(HOME_PATH, 'Application Data', 'Google', 'O3D',
+ INSTALL_PATHS += [join(HOME, 'Application Data', 'Google', 'O3D',
'o3d_host.dll')]
elif util.IsMac():
INSTALL_PATHS += ['/Library/Internet Plug-Ins/O3D.plugin']
else:
- INSTALL_PATHS += [join(HOME_PATH, '.mozilla', 'plugins',
+ INSTALL_PATHS += [join(HOME, '.mozilla', 'plugins',
'libnpo3dautoplugin.so')]
« no previous file with comments | « tests/lab/run_lab_test.py ('k') | tests/lab/runner_util.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698