<div dir="ltr">Thanks for the feedback.  I tried to factor the emails out in a way that tried to balance<div>the number of submissions and ease of either merging or just handcoding the</div><div>changes.  I will dig a bit more into the basis-spline.</div><div><br></div><div>In the future I will focus patches on one issue at a time.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 6, 2015 at 10:38 AM, Liam Healy <span dir="ltr"><<a href="mailto:lhealy@common-lisp.net" target="_blank">lhealy@common-lisp.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks. I have modified and applied #1 and #2, which should make it<br>
into the next QL release. #3 will have to wait; GSLL is behind even<br>
later versions of GSL 1.x.<br>
<br>
For patch #2, I have added the use of trivial-features to standardize<br>
the different platforms, so I use only #+windows now for any<br>
windows-based platform.<br>
<br>
In the future, it would be helpful if you could not mix together<br>
unrelated changes into a single patch, and defer sending patches until<br>
you have figured out exactly where the problem is and how to fix it.<br>
For example, I did not apply the removal of basis-spline, because that<br>
works on other platforms, so this patch would gratuitously remove<br>
capability that functions for many people.<br>
<br>
Thanks again,<br>
Liam<br>
<div class="HOEnZb"><div class="h5"><br>
On Sun, Nov 22, 2015 at 4:47 PM, Mirko Vukovic <<a href="mailto:mirko.vukovic@gmail.com">mirko.vukovic@gmail.com</a>> wrote:<br>
> For GSLL I am attaching two git patches that accomplish several things:<br>
> - Loading of GSL that is part of MSYS2 on Windows<br>
> - Removed dependency on Osicat library<br>
> - Enable loading of GSL 2.0 by fixing a bit of code in nonlinear least<br>
> squares fitter.<br>
> - Disabled loading of basis-spline.lisp<br>
> - Cosmetic fixes.<br>
><br>
> I could not figure out the problem with basis-spline.  I thus removed it.<br>
> There may be a much better solution than that.<br>
><br>
> I checked the above on latest CCL and SBCL on Windows 7.<br>
><br>
> Warning: the two commits required some git surgery (splitting a commit into<br>
> two).  I hope that I did not introduce additional problems with the splits.<br>
><br>
> For Antik I am attaching one patch that fixes the readtable problem on CCL:<br>
> (named-readtables:defreadtable :antik<br>
>   (:merge :standard)<br>
>   #-ccl(:macro-char #\# :dispatch))<br>
><br>
> Without the patch all #... reader macros (CL and Antik) were deleted.  With<br>
> this patch both CL and Antik's reader macros work in CCL.  I also tried the<br>
> patch with SBCL, and I don't see a need for it.  But since the macro-char<br>
> redefinition did not hurt in SBCL, I left it in.<br>
><br>
><br>
> Mirko<br>
</div></div></blockquote></div><br></div>