<div dir="ltr"><br><br><div class="gmail_quote">On Fri, May 1, 2015 at 8:21 AM FUJIWARA Katsunori <<a href="mailto:foozy@lares.dti.ne.jp">foozy@lares.dti.ne.jp</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User FUJIWARA Katsunori <<a href="mailto:foozy@lares.dti.ne.jp" target="_blank">foozy@lares.dti.ne.jp</a>><br>
# Date 1430493303 -32400<br>
#      Sat May 02 00:15:03 2015 +0900<br>
# Branch stable<br>
# Node ID 7086e9cfe82c6c5c08ad634aa8cfb74fb67b01d7<br>
# Parent  ab9390165429b7891364c0d454f3ae3b04b00c7d<br>
tests: make tests with temporary environment setting portable<br>
<br>
With "dash" (as "/bin/sh" on Debian GNU/Linux), command execution in<br>
"ENV=val foo bar" style doesn't work as expect in test script files,<br>
if "foo" is user-defined function: it works fine, if "foo" is existing<br>
commands like "hg".<br>
<br>
09049042ab99 introduced tests for HGPLAIN and HGPLAINEXCEPT into<br>
test-revset.t, and all of them are in such style.<br>
<br>
This patch doesn't:<br>
<br>
  - add explicit unsetting for HGPLAIN and HGPLAINEXCEPT<br>
<br>
    they are already introduced by 09049042ab99<br>
<br>
  - write assignment and exporting in one line<br>
<br>
    "ENV=val; export ENV" for two or more environment variables in one<br>
    line causes failure of test-check-code-hg.t: it is recognized as<br>
    "don't export and assign at once" unfortunately.<br>
<br>
diff --git a/tests/test-revset.t b/tests/test-revset.t<br>
--- a/tests/test-revset.t<br>
+++ b/tests/test-revset.t<br>
@@ -1066,12 +1066,16 @@ aliases:<br>
     <fullreposet+ 0:9>><br>
   6<br>
<br>
-  $ HGPLAIN=1 try m<br>
+  $ HGPLAIN=1<br>
+  $ export HGPLAIN<br>
+  $ try m<br>
   ('symbol', 'm')<br>
   abort: unknown revision 'm'!<br>
   [255]<br>
<br>
-  $ HGPLAIN=1 HGPLAINEXCEPT=revsetalias try m<br>
+  $ HGPLAINEXCEPT=revsetalias<br>
+  $ export HGPLAINEXCEPT<br>
+  $ try m<br>
   ('symbol', 'm')<br>
   (func<br>
     ('symbol', 'merge')<br></blockquote><div><br></div><div>Also drop the "(for some reason HGPLAIN and HGPLAINEXCEPT can carry forward)" comment now? It seems quite expected after this patch.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -1097,14 +1101,18 @@ aliases:<br>
   <baseset+ [8]><br>
   8<br>
<br>
-  $ HGPLAIN=1 try 'p2(.)'<br>
+  $ HGPLAIN=1<br>
+  $ export HGPLAIN<br>
+  $ try 'p2(.)'<br>
   (func<br>
     ('symbol', 'p2')<br>
     ('symbol', '.'))<br>
   * set:<br>
   <baseset+ []><br>
<br>
-  $ HGPLAIN=1 HGPLAINEXCEPT=revsetalias try 'p2(.)'<br>
+  $ HGPLAINEXCEPT=revsetalias<br>
+  $ export HGPLAINEXCEPT<br>
+  $ try 'p2(.)'<br>
   (func<br>
     ('symbol', 'p2')<br>
     ('symbol', '.'))<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@selenic.com" target="_blank">Mercurial-devel@selenic.com</a><br>
<a href="http://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">http://selenic.com/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div></div>