<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 3, 2014 at 11:31 PM, Robert P. Goldman <span dir="ltr"><<a href="mailto:rpgoldman@sift.info" target="_blank">rpgoldman@sift.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">Jean-Claude Beaudoin wrote:<br>
><br>
> On Mon, Nov 3, 2014 at 4:16 PM, Robert P. Goldman <<a href="mailto:rpgoldman@sift.info">rpgoldman@sift.info</a><br>
</span><span class="">> <mailto:<a href="mailto:rpgoldman@sift.info">rpgoldman@sift.info</a>>> wrote:<br>
><br>
><br>
>     This morning I pulled an update from the mkcl git repo, rebuilt on my<br>
>     Linux Mint machine, and retested with the latest ASDF.  All the tests<br>
>     that completed seemed to complete successfully, but test-program.script<br>
>     hung, instead of completing.<br>
><br>
>     ...<br>
><br>
>     The block of the test that fails is "test program-op", "test image-op"<br>
>     seems to complete successfully.<br>
><br>
>     This seems odd to me, since the result of the image test is a standalone<br>
>     program: it doesn't need to be loaded into mkcl:<br>
><br>
>     ...<br>
><br>
>     So seems like the program-op should be able to do exactly what the<br>
>     image-op does, and work correctly.<br>
><br>
><br>
> I just tried to have as clean a run at it as I can reasonably have it.<br>
> So, from a freshly created directory, I did:<br>
><br>
> git clone git://<a href="http://common-lisp.net/projects/mkcl/mkcl.git" target="_blank">common-lisp.net/projects/mkcl/mkcl.git</a><br>
</span>> <<a href="http://common-lisp.net/projects/mkcl/mkcl.git" target="_blank">http://common-lisp.net/projects/mkcl/mkcl.git</a>><br>
<span class="">><br>
> then built it with: (cd mkcl; configure; cd src; make install-local)<br>
<br>
</span>Hm.  I did a slightly different set of steps:<br>
<br>
cd mkcl<br>
./configure<br>
make<br>
sudo make install<br>
<br>
but these look like they should be equivalent (since I didn't set MKCL).<br>
<br></blockquote><div><br></div><div>It should be OK, it is just less self-contained in its effects.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
> Followed by:<br>
><br>
> git clone git://<a href="http://common-lisp.net/projects/asdf/asdf.git" target="_blank">common-lisp.net/projects/asdf/asdf.git</a> <br>
<span class="">><br>
> cd asdf; make t l=mkcl<br>
><br>
> This resulted in:<br>
><br>
> -#---------------------------------------<br>
> Using /home/jean-claude/tmp/ASDF/mkcl/src/bin/mkcl<br>
> Ran 56 tests:<br>
>   55 passing and 1 failing<br>
> failing test(s):  test-undeferred-warnings.script<br>
> -#---------------------------------------<br>
><br>
> To view full results and failures, try the following command:<br>
>      less -p ABORTED build/results/mkcl-test.text<br>
> make: *** [test-lisp] Error 1<br>
> jean-claude@mars><br>
><br>
> The last line just here above is my normal prompt, so no hang.<br>
> As you see I do not reproduce the problem you mentioned.<br>
><br>
> BTW, what happens in test-undeferred-warnings seems to be a syntax issue but I am still unclear about it.<br>
<br>
</span>What happens here is that there is a function with undeclared local<br>
variables, but ASDF is not getting a warning from COMPILE-FILE, so it<br>
mistakenly believes the build is successful.<br>
<br>
I don't think I wrote this test properly, though.  In a message crossing<br>
with this one, Faré has limited its applicability, but I'm not sure I<br>
understand the change. Surely that system should not build successfully?<br>
<br></blockquote><div><br></div><div>I did a minor modification to test-undeferred-warnings.script and it passes now.<br></div><div>Here is what "git diff" says:<br><br>diff --git a/test/test-undeferred-warnings.script b/test/test-undeferred-warnings.script<br>index 810bcd2..bf1cfdb 100644<br>--- a/test/test-undeferred-warnings.script<br>+++ b/test/test-undeferred-warnings.script<br>@@ -10,12 +10,12 @@<br> (def-test-system :undefined-variables<br>   :components ((:file "fun-with-undefined-locals")))<br> <br>-(errors #+(or allegro clozure) compile-file-error<br>+(errors #+(or allegro clozure mkcl) compile-file-error<br>         #+(or cmu scl) null<br>         #+sbcl compile-failed-error<br>         (let ((*compile-file-warnings-behaviour* :error))<br>           (load-system :undefined-variables)))<br>-(errors #+(or allegro clozure) compile-file-error<br>+(errors #+(or allegro clozure mkcl) compile-file-error<br>         #+(or cmu scl) null<br>         #+sbcl compile-failed-error<br>         (let ((*compile-file-warnings-behaviour* :warning))<br><br><br></div><div>Maybe a #-(or <all-of-the-above>) should be added after the set of #+ to make sure that macro "errors" always gets the right number of arguments.<br><br></div><div>Cheers,<br><br>JCB<br><br></div><div> </div></div></div></div>