Log to gtest-parallel-logs/ subdir. (#46)
This changes the default logdir from /tmp/gtest-parallel to
/tmp/gtest-parallel-logs. The primary motivation is that that we do this by
appending gtest-parallel-logs (to /tmp by default), so that if a user provides a
directory that is already in use, we wouldn't clear that directory out
completely (--output_dir=Docs/ was previously dangerous, as we'd nuke Docs/*).
Now instead we'd only clear out a subdirectory that we create and own ourselves.
Fixes #44.
diff --git a/gtest_parallel.py b/gtest_parallel.py
index 5e50c33..274fb44 100755
--- a/gtest_parallel.py
+++ b/gtest_parallel.py
@@ -603,8 +603,11 @@
usage = 'usage: %prog [options] binary [binary ...] -- [additional args]')
parser.add_option('-d', '--output_dir', type='string',
- default=os.path.join(tempfile.gettempdir(), "gtest-parallel"),
- help='output directory for test logs')
+ default=tempfile.gettempdir(),
+ help='Output directory for test logs. Logs will be '
+ 'available under gtest-parallel-logs/, so '
+ '--output_dir=/tmp will results in all logs being '
+ 'available under /tmp/gtest-parallel-logs/.')
parser.add_option('-r', '--repeat', type='int', default=1,
help='Number of times to execute all the tests.')
parser.add_option('--retry_failed', type='int', default=0,
@@ -640,6 +643,11 @@
'case in parallel.')
(options, binaries) = parser.parse_args()
+ # Append gtest-parallel-logs to log output, this is to avoid deleting user
+ # data if an user passes a directory where files are already present. If a
+ # user specifies --output_dir=Docs/, we'll create Docs/gtest-parallel-logs
+ # and clean that directory out on startup, instead of nuking Docs/.
+ options.output_dir = os.path.join(options.output_dir, 'gtest-parallel-logs')
if binaries == []:
parser.print_usage()
