[Vm-dev] Setting up a VMMaker image

[David T. Lewis]

On Fri, Nov 20, 2009 at 03:23:46PM +0100, Henrik Johansen wrote:
> 
> Hi David!
> I tried this on latest Pharo 1.1 core (11050), ran into a couple of problems, posting them here for information.
> 
> - One package contained method definitions : 
>   CanvasCharacterScanner>>cr
>   CanvasCharacterScanner>>paddedSpace
>   CanvasCharacterScanner>>tab
> This class is in MorphicExtras, which is not in Pharo-core anymore. I assume it's safe for those using it to hit proceed.
>  

Probably this is safe, but it's worth checking to see which class is making that reference
to be sure.

> - B3DEnginePlugin class>>translateInDirectory: directory doInlining: inlineFlag
> 	contains:
> inject: 0 into: [:tS :cl |
> 		tS := tS max: cl timeStamp]
> 
> Raises a Syntax error in Pharo, since you can't store into blockArgs, and either way that store should be unnecessary, right?
> 

This needs to be fixed in VMMaker, thanks.


> This might be an Pharo-specific MC bug. but after fixing the source in the popup and hitting ctrl-s, loading proceeds, but when MCMultiPackageLoader tries to install the method, selector and compiledMethod is nil, as if the modifed source had not been accepted... 
> To fix, you can change method source in debugger manually, then remake the MethodAddition and make it become the failed one before proceeding, but it's a lot less convenient.
> Might be a good idea to check if the same happens when encountering a syntax error while loading in Squeak as well :)
> 
> Finally, I get a warning when initializing SlangBrowser, since CodeHolder does not respond to addEditContentsOption:.  (I assume this just means some menu-options won't be accessible in Pharo?)
> 

Just take SlangBrowser out of your configuration. It is not needed for VM building.

> Other than that, after fixing a bug in CrLfStream and loading the Sound package from Pharo repository (+ fixing a loading intialization issue with that...)) it seemed to generate sources just fine for all plugins! :D
> 

Good! Thanks for the updates.