<div dir="ltr">Hey<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 27, 2014 at 10:00 PM, Nathan Lynch <span dir="ltr"><<a href="mailto:nathan_lynch@mentor.com" target="_blank">nathan_lynch@mentor.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">An argument of "$2" was added to the prove command line in<br>
tests/run.sh by 68270f0f604e "Mi test: Basic test structure".  This is<br>
harmless since run.sh is never called with more than one argument, but<br></blockquote><div><br></div><div>The multiple arguments could be overcome by adding "" to the multiple test files list but as you said it is confusing and error prone.  </div><div><br></div><div>This actually can be used to debug test with a "-v". Or add any basic arguments to the prove command and was simply convenient at the time.If this actually break patch from OE ( I assume OE stand for OpenEmbedded and you are talking about this commit [1])  I'm okai with reverting the change I introduced.</div><div><br></div><div>Thanks</div><div>Cheers!</div><div><br></div><div>[1] <a href="https://github.com/openembedded/oe-core/commit/d0d184045e8034ddbc77549e681f187e12ab1d08">https://github.com/openembedded/oe-core/commit/d0d184045e8034ddbc77549e681f187e12ab1d08</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
it is confusing, and it breaks patches in OE that add support for </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
running tests on the target.<br>
<br>
Signed-off-by: Nathan Lynch <<a href="mailto:nathan_lynch@mentor.com">nathan_lynch@mentor.com</a>><br>
---<br>
 tests/run.sh | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/tests/run.sh b/tests/run.sh<br>
index ce51639f7626..c6c50fd9cff9 100755<br>
--- a/tests/run.sh<br>
+++ b/tests/run.sh<br>
@@ -19,4 +19,4 @@<br>
<br>
 [ -z "$1" ] && echo "Error: No testlist. Please specify a testlist to run." && exit 1<br>
<br>
-prove $2 --merge --exec '' - < $1<br>
+prove --merge --exec '' - < $1<br>
<span class=""><font color="#888888">--<br>
1.9.3<br>
<br>
<br>
_______________________________________________<br>
lttng-dev mailing list<br>
<a href="mailto:lttng-dev@lists.lttng.org">lttng-dev@lists.lttng.org</a><br>
<a href="http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev" target="_blank">http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Jonathan Rajotte Julien</div>
</div></div>