<div dir="ltr">Hi Kristof,<div><br></div><div>Thanks for the review. I've removed most of the docstring from the options and put a new sub section in the LNT documentation with a worked example.</div><div><br></div><div>> <span style="line-height:1.5">Don't we have directories with multiple tests in the same directory?</span></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5">Thanks for spotting this. This was completely wrong, but so was our handling of --only-test. I fixed that in r263443 and have committed the modified version of this patch in r263445.</span></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 1 Mar 2016 at 07:41 Kristof Beyls via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">kristof.beyls added inline comments.<br>
<br>
================<br>
Comment at: lnt/tests/test_suite.py:154-158<br>
@@ -153,1 +153,7 @@<br>
                          type='choice', default=[])<br>
+        group.add_option("", "--single-result", dest="single_result",<br>
+                         help=("only execute one test and apply "<br>
+                               "--single-result-predicate to calculate the "<br>
+                               "exit status. Do not submit to a LNT instance "<br>
+                               "or create a JSON report (useful for bisection)"))<br>
+        group.add_option("", "--single-result-predicate",<br>
----------------<br>
The help text doesn't indicate that this argument expects a fully-qualified test name.<br>
Also, it seems like this argument changes test execution - so is it best placed in the "Test Execution" group, or here in the "Output Options" group?<br>
<br>
All in all, the help text is a bit long. Maybe it would be better to shorten the help text here and write some more elaborate documentation with an example for this in one of the rst files under docs, so that how to use this becomes part of the LNT documentation?<br>
Should the documentation for the bisecter (which I guess lives in zorg) be updated to have an example making use of these options?<br>
<br>
================<br>
Comment at: lnt/tests/test_suite.py:230-232<br>
@@ -215,2 +229,5 @@<br>
<br>
+        if self.opts.single_result:<br>
+            self.opts.only_test = os.path.dirname(self.opts.single_result)<br>
+<br>
         opts.cmake = resolve_command_path(opts.cmake)<br>
----------------<br>
Don't we have directories with multiple tests in the same directory? e.g. test-suite/SingleSource/Regression/C<br>
Does this mean that at most a full directory can be specified, not a single test?<br>
Would using this to bisect a single test in a directory with multiple tests still work?<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D17717" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17717</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>