[Piglit] [PATCH 1/2] piglit-run.py: Fix Path resolution

Kenney Phillis kphillisjr at gmail.com
Mon Apr 29 08:50:14 PDT 2013


Path resolution should always be a resolved from relative path to absolute path.
This is to allow for distributed testing where you build a single copy of
piglit and then redistribute it over a network.
---
 piglit-run.py |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/piglit-run.py b/piglit-run.py
index 6d6ec77..13f9027 100755
--- a/piglit-run.py
+++ b/piglit-run.py
@@ -133,9 +133,8 @@ def main():
 
 	# If resume is requested attempt to load the results file
 	# in the specified path
+	resultsDir = path.realpath(args.resultsPath)
 	if args.resume is True:
-		resultsDir = path.realpath(args.resultsPath)
-
 		# Load settings from the old results JSON
 		old_results = core.loadTestResults(resultsDir)
 		profileFilename = old_results.options['profile']
@@ -148,7 +147,6 @@ def main():
 	# Otherwise parse additional settings from the command line
 	else:
 		profileFilename = args.testProfile
-		resultsDir = args.resultsPath
 
 	# Pass arguments into Environment
 	env = core.Environment(concurrent=args.concurrency,
-- 
1.7.9.5



More information about the Piglit mailing list