<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 17, 2014 at 2:47 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Jean-Claude Beaudoin wrote:<br>
><br>
><br>
><br>
> On Mon, Mar 17, 2014 at 12:48 PM, Robert P. Goldman <<a href="mailto:rpgoldman@sift.info">rpgoldman@sift.info</a><br>
</div><div class="">> <mailto:<a href="mailto:rpgoldman@sift.info">rpgoldman@sift.info</a>>> wrote:<br>
><br>
> Random minor question:<br>
><br>
> In bundle.lisp why the replacement of 'defsystem with<br>
> 'asdf/defsystem:defsystem? Did something go wrong for you with the<br>
> former?<br>
><br>
><br>
> The "defsystem" symbol was interned in the wrong package, being read at<br>
> that point of the code into package "asdf/bundle". Package "asdf/bundle"<br>
> does<br>
> not use package "asdf/defsystem". All this resulting into a failure to<br>
> properly<br>
> expand the defsystem macro.<br>
<br>
</div>So, in other words, the bundle code was buggy before, right?</blockquote><div><br></div><div>You are quite right yourself.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I wonder<br>
why we didn't see this.... What piece of your code exercised this? I'd<br>
like to know so we can test.<br></blockquote><div><br></div><div>The bundle facility is a central part of the MKCL build process. A large<br></div><div>part of what MKCL puts in its "contrib" sub-directory is built using it.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class=""><br>
<br>
><br>
><br>
><br>
> Other than that, I didn't see anything in the diff.<br>
><br>
><br>
> I hope that you mean that you didn't see anything *wrong* in the diff.<br>
<br>
</div>You are quite right!<br>
<br>
I'll merge to master and tag this afternoon.<br>
<br>
Thanks,<br>
R<br>
<br>
<br>
</blockquote></div><br></div><div class="gmail_extra">P.S.: I took the liberty to CC the asdf-devel list since your initial "defsystem" question had too, I left the CC out of my reply by simply not paying enough attention to which reply button I clicked.on.<br>
<br></div></div>