<div dir="ltr">FFI used to BE part of the base image, and was specifically taken out on purpose. The purpose, as I remember it, was because it acted like a security hole for squeak as a squeaklet on the web (when you browse a page on a website that invokes Squeak, and then being able to, via FFI, invoke anything you want on the remote machine). So, for any image still working in that mode (such as the etoys derivative), it is important to leave out.<div>
<br></div><div>But, with the shiny new CI servers, couldn't we have an artifact built with FFI loaded?</div><div><br></div><div>Or, even neater, include FFI in the trunk, but have a switch/pragma in the image that says 'Don't ever load FFI' for those that don't want it, but want everything else loaded? So, for people that LIKE FFI, it is just part of Trunk; for those that have to avoid it, it isn't loaded; for the rest, they get whatever was setup when they got their image. (I'd be in the first group most of the time)</div>
<div><br></div><div>-cbc</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 14, 2013 at 2:30 PM, Frank Shearar <span dir="ltr"><<a href="mailto:frank.shearar@gmail.com" target="_blank">frank.shearar@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 14 November 2013 16:42, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>> wrote:<br>
> Hi Frank,<br>
><br>
><br>
> On Thu, Nov 14, 2013 at 1:59 AM, Frank Shearar <<a href="mailto:frank.shearar@gmail.com">frank.shearar@gmail.com</a>><br>
> wrote:<br>
>><br>
>> That's the source of the problem: FFI-Kernel uses FFIConstants, which<br>
>> is declared/defined in FFI-Pools.<br>
>><br>
>> Insert standard Frank rant of something called "Kernel" depending on<br>
>> other stuff. (Possibly insert standard Chris rant of a package<br>
>> containing a single class? :) )<br>
><br>
><br>
> The problem here is that the VM depends on FFIConstants also, and the VM<br>
> shouldn't depend on the rest of FFI. So FFIConstants does need to be on its<br>
> own, but could perhaps be called something different.<br>
<br>
</div>You know, I would be quite happy if we folded FFI into the base image.<br>
Seriously. For once I'd argue for _including_ something. I would<br>
gladly trade Morphic for FFI.<br>
<br>
OK, I'll chalk this up to That's How It Is, and move on. If you<br>
install from SqueakMap (as you should, and as I should have) you get<br>
the right load order.<br>
<span class="HOEnZb"><font color="#888888"><br>
frank<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
>> frank<br>
>><br>
>> On 12 November 2013 23:20, Chris Muller <<a href="mailto:ma.chris.m@gmail.com">ma.chris.m@gmail.com</a>> wrote:<br>
>> > I didn't see you loading FFI-Pools in advance. The issue you<br>
>> > encountered<br>
>> > may not have had anything to do with -eem.24.<br>
>> ><br>
>> > FYI -- FFI also has a "head" release on SqueakMap which documents how to<br>
>> > load it.<br>
>> ><br>
>> ><br>
>> > On Tue, Nov 12, 2013 at 4:50 PM, Frank Shearar <<a href="mailto:frank.shearar@gmail.com">frank.shearar@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> That pulls in the latest packages, right?<br>
>> >><br>
>> >> So that does work. But why does the reported version fail to load?<br>
>> >> That's a problem still.<br>
>> >><br>
>> >> (While this lets me get on with what I wanted to do - add<br>
>> >> #interleaving: to Xtreams - this is still a problem. But thanks for<br>
>> >> the workaround, Chris!)<br>
>> >><br>
>> >> frank<br>
>> >><br>
>> >> On 12 November 2013 20:55, Chris Muller <<a href="mailto:asqueaker@gmail.com">asqueaker@gmail.com</a>> wrote:<br>
>> >> > Installer new merge: #ffi.<br>
>> >> ><br>
>> >> > Worked for me..<br>
>> >> ><br>
>> >> > On Tue, Nov 12, 2013 at 2:45 PM, Frank Shearar<br>
>> >> > <<a href="mailto:frank.shearar@gmail.com">frank.shearar@gmail.com</a>><br>
>> >> > wrote:<br>
>> >> >> On 26 February 2013 18:36, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>><br>
>> >> >> wrote:<br>
>> >> >>><br>
>> >> >>><br>
>> >> >>> On Tue, Feb 26, 2013 at 1:51 AM, Frank Shearar<br>
>> >> >>> <<a href="mailto:frank.shearar@gmail.com">frank.shearar@gmail.com</a>><br>
>> >> >>> wrote:<br>
>> >> >>>><br>
>> >> >>>> (Installer monticello mc: (MCHttpRepository new location:<br>
>> >> >>>> '<a href="http://source.squeak.org/FFI'" target="_blank">http://source.squeak.org/FFI'</a>))<br>
>> >> >>>> install: 'FFI-Kernel-eem.24.mcz'. "There's a -tbn.25, but<br>
>> >> >>>> that's<br>
>> >> >>>> not important to this post"<br>
>> >> >>>><br>
>> >> >>>> fails with an MNU: ExternalFunction class >><br>
>> >> >>>> callingConventionFor:.<br>
>> >> >>>><br>
>> >> >>>> As far as I can see what's happening is this:<br>
>> >> >>>> * during the loading of the mcz ExternalFunction is defined,<br>
>> >> >>>> * a method is parsed (#XOpenDisplay, which has a pragma <cdecl:<br>
>> >> >>>> X11Display* ''XOpenDisplay'' (char*) module:''X11''>)<br>
>> >> >>>> * Parser >> externalFunctionDeclaration checks whether<br>
>> >> >>>> ExternalFunction is defined.<br>
>> >> >>>> * It is, so tries to evaluate `descriptorClass callingConvention:<br>
>> >> >>>> here` and boom, because ExternalFunction class >><br>
>> >> >>>> callingConventionFor: _has not been loaded yet_.<br>
>> >> >>><br>
>> >> >>><br>
>> >> >>> I thought we'd modified Monticello to load new methods first. Did<br>
>> >> >>> this not<br>
>> >> >>> get added to trunk?<br>
>> >> >><br>
>> >> >> Apparently not. It still happens with an up-to-date Squeak 4.5.<br>
>> >> >><br>
>> >> >> frank<br>
>> >> >><br>
>> >> >>>> I've seen this kind of issue with Helvetia code: sometimes you<br>
>> >> >>>> simply<br>
>> >> >>>> have to load class-side methods first.<br>
>> >> >>>><br>
>> >> >>>> Thoughts? Lukas worked around the issue with his Helvetia code by<br>
>> >> >>>> directly patching Monticello, and ripping out its "try to do<br>
>> >> >>>> atomic<br>
>> >> >>>> loading" mechanism.<br>
>> >> >>>><br>
>> >> >>>> frank<br>
>> >> >>>><br>
>> >> >>><br>
>> >> >>><br>
>> >> >>><br>
>> >> >>> --<br>
>> >> >>> best,<br>
>> >> >>> Eliot<br>
>> >> >>><br>
>> >> >>><br>
>> >> >>><br>
>> >> >><br>
>> >> ><br>
>> ><br>
>> ><br>
>><br>
><br>
><br>
><br>
> --<br>
> best,<br>
> Eliot<br>
<br>
</div></div></blockquote></div><br></div>