<div dir="ltr">I think those are already planned to be taken. So lets see if the tests pass in the next build.<div><br></div><div>Robby</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 24, 2013 at 9:55 PM, Sam Tobin-Hochstadt <span dir="ltr">&lt;<a href="mailto:samth@ccs.neu.edu" target="_blank">samth@ccs.neu.edu</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If we just take the latter 3 commits that Vincent mentioned, it should<br>
fix the (serial) tests. Those are just TR test fixes, and can&#39;t break<br>
anything else.<br>
<br>
Sam<br>
<br>
On Wed, Jul 24, 2013 at 10:54 PM, Robby Findler<br>
<div class="HOEnZb"><div class="h5">&lt;<a href="mailto:robby@eecs.northwestern.edu">robby@eecs.northwestern.edu</a>&gt; wrote:<br>
&gt; I don&#39;t think we should take that commit. Is it possible to get a version of<br>
&gt; the TR tests that either don&#39;t run in parallel or run without passing<br>
&gt; keywords across place channels?<br>
&gt;<br>
&gt; Robby<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; On Wed, Jul 24, 2013 at 9:41 PM, Sam Tobin-Hochstadt &lt;<a href="mailto:samth@ccs.neu.edu">samth@ccs.neu.edu</a>&gt;<br>
&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; I will say that I&#39;m not 100% happy with just assuming that TR works in<br>
&gt;&gt; the absence of a clean test run.  Is there a reason not to include<br>
&gt;&gt; these fixes?<br>
&gt;&gt;<br>
&gt;&gt; Sam<br>
&gt;&gt;<br>
&gt;&gt; On Wed, Jul 24, 2013 at 5:15 PM, Robby Findler<br>
&gt;&gt; &lt;<a href="mailto:robby@eecs.northwestern.edu">robby@eecs.northwestern.edu</a>&gt; wrote:<br>
&gt;&gt; &gt; Thanks.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Robby<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; On Wed, Jul 24, 2013 at 3:19 PM, Vincent St-Amour &lt;<a href="mailto:stamourv@ccs.neu.edu">stamourv@ccs.neu.edu</a>&gt;<br>
&gt;&gt; &gt; wrote:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; The problems I&#39;ve observed were only with the tests.<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; The first problem I reported was caused by a bug/limitation in places,<br>
&gt;&gt; &gt;&gt; which was fixed by 733907474190da499a1782b230086170c5b87643. It was<br>
&gt;&gt; &gt;&gt; preventing the TR test suite from running properly.<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; The others were bugs in the TR tests, not in TR itself.<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; TR itself should be fine for the release.<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Vincent<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; At Wed, 24 Jul 2013 15:03:08 -0500,<br>
&gt;&gt; &gt;&gt; Robby Findler wrote:<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; [1  &lt;text/plain; UTF-8 (7bit)&gt;]<br>
&gt;&gt; &gt;&gt; &gt; Vincent: can you clarify what the status of TR &amp; the release is,<br>
&gt;&gt; &gt;&gt; &gt; please?<br>
&gt;&gt; &gt;&gt; &gt; Are there problems only with tests, or were there problems elsewhere<br>
&gt;&gt; &gt;&gt; &gt; too?<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; Thanks,<br>
&gt;&gt; &gt;&gt; &gt; Robby<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; On Tue, Jul 23, 2013 at 9:11 AM, Vincent St-Amour<br>
&gt;&gt; &gt;&gt; &gt; &lt;<a href="mailto:stamourv@ccs.neu.edu">stamourv@ccs.neu.edu</a>&gt;wrote:<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; The TR tests still fail when using a single place.<br>
&gt;&gt; &gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; The following commits should be included in the release:<br>
&gt;&gt; &gt;&gt; &gt; &gt; 069ff59a4bd6a988a5670c7e4dd38c1dbbe12ec0<br>
&gt;&gt; &gt;&gt; &gt; &gt; 0e7940ab4943600e6f5c8f13ce7ee13e8af9a8f5<br>
&gt;&gt; &gt;&gt; &gt; &gt; ab5075bc762356f86bb7dfd34dac8d24ada1a140<br>
&gt;&gt; &gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; Vincent<br>
&gt;&gt; &gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; At Mon, 22 Jul 2013 17:49:20 -0400,<br>
&gt;&gt; &gt;&gt; &gt; &gt; Vincent St-Amour wrote:<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; At Mon, 22 Jul 2013 15:13:28 -0400,<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; Ryan Culpepper wrote:<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; Checklist items for the v5.3.6 release<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;    (using the v5.3.5.900 release candidate build)<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; Search for your name to find relevant items, reply when you<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; finish<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; an<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; item (please indicate which item/s is/are done).  Also, if you<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; have any<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; commits that should have been picked, make sure that the<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; changes<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; are<br>
&gt;&gt; &gt;&gt; &gt; &gt; in.<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; Important: new builds are created without announcement, usually<br>
&gt;&gt; &gt;&gt; &gt; &gt; whenever<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; I pick a few commits.  If you need to commit changes, please<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; make<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; sure<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; you tell me to pick it into the release branch.<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; --&gt; Release candidates are at<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; --&gt;   <a href="http://pre.racket-lang.org/release/installers" target="_blank">http://pre.racket-lang.org/release/installers</a><br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; The &quot;Linux/i386/Ubuntu Precise&quot; installer is 64 bits, which is<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; wrong.<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; * Sam Tobin-Hochstadt &lt;<a href="mailto:samth@ccs.neu.edu">samth@ccs.neu.edu</a>&gt;,<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;     Vincent St-Amour &lt;<a href="mailto:stamourv@ccs.neu.edu">stamourv@ccs.neu.edu</a>&gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;    - Match Tests<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;    - Typed Racket Tests<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;    - Typed Racket Updates: update HISTORY<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;    (updates should show v5.3.6 as the most current version;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; email<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; me<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;    to pick the changes when they&#39;re done, or tell me if there<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; are<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt; no<br>
&gt;&gt; &gt;&gt; &gt; &gt; such<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; &gt;    changes.)<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; I get the following error when runing the TR tests:<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; place-channel-put: value not allowed in a message<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;   value: &#39;#:methods<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;   ...<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; Is 733907474190da499a1782b230086170c5b87643 missing from the<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; release?<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; Now running the tests without places.<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; &gt; &gt; Vincent<br>
&gt;&gt; &gt;&gt; &gt; &gt; _________________________<br>
&gt;&gt; &gt;&gt; &gt; &gt;   Racket Developers list:<br>
&gt;&gt; &gt;&gt; &gt; &gt;   <a href="http://lists.racket-lang.org/dev" target="_blank">http://lists.racket-lang.org/dev</a><br>
&gt;&gt; &gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; [2  &lt;text/html; UTF-8 (quoted-printable)&gt;]<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; _________________________<br>
&gt;&gt; &gt;   Racket Developers list:<br>
&gt;&gt; &gt;   <a href="http://lists.racket-lang.org/dev" target="_blank">http://lists.racket-lang.org/dev</a><br>
&gt;&gt; &gt;<br>
&gt;<br>
&gt;<br>
</div></div></blockquote></div><br></div>