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.