New VMMaker/svn release
David T. Lewis
lewis at mail.msen.com
Sun Mar 12 17:35:49 UTC 2006
Tim,
Some belated followup on this. The patches for sqMemoryAccess.h do
not seem to have been committed to SVN as of SVN 1352. I'm attaching
a copy of this patch along with the others that I currently apply
to the platforms tree in order to do a Unix VM build.
Unrelated to the patch I sent for sqMemoryAccess.h, but affecting
the same source file, you had asked if the oopForPointer macro was
defined wrong and I had replied that I thought that it did look
wrong. However, I just got around to trying a build with the macro
defined as:
#define oopForPointer(ptr) ((sqInt)((ptr) - sqMemoryBase))
and this does *not* work (blows up with a compiler error). So I
don't know what is right here, but you should not make that change
until somebody figures it out.
For reference, here is a summary of the patches (attached) that I
am currently using. Two patches are required, and the other two
are nice to have.
platforms-unix-vm-sqUnixCharConv.c.diff - Provide missing
sqGetFilenameFromString() function. This function must be present
in order to build a unix VM.
platforms-Cross-vm-sqMemoryAccess.h.diff - Missing memory access
macros. Required for Linux build.
platforms-unix-vm-sqUnixExternalPrims.c.diff - Provide meaningful
error message if e.g. vm-display-X11 fails to load for some reason.
Nice to have but not required for successful VM build.
platforms-unix-config-config.h.in.diff - Define #SQAIO_H. Required
for OSPP only, nice to have.
I'll note also that I cannot build an MPEG plugin for Unix with
the current sources, but I assume this a new issue related to
John's recent upgrades and I have not looked into it yet.
Dave
On Tue, Jan 03, 2006 at 01:34:35PM -0800, tim Rowledge wrote:
>
> On 29-Dec-05, at 12:00 PM, David T. Lewis wrote:
> >
> > Attachments (in zip file):
> >
> > platforms-win32-plugins-FilePlugin-sqWin32FilePrims.c.diff - Change
> > Win32
> > FilePlugin to use session ID from the interpreter rather than
> > generate its
> > own value (for consistency, and also so as not to break OSPP
> > for Win32).
> OK, that is up to andreas - I couldn't commit it if I wanted to.
> >
> > platforms-Cross-vm-sqMemoryAccess.h.diff - Add missing memory
> > access macros
> > (the inline functions are complete, but some of the
> > corresponding macros
> > are missing).
> OK, made the changes. Am I hallucinating or should we really make the
> #define oopForPointer(ptr) ((sqInt)(ptr))
> read
> #define oopForPointer(ptr) ((sqInt)(ptr-sqMemoryBase))
> instead?
>
>
> >
> > platforms-unix-vm-sqUnixExternalPrims.c.diff - Provide meaningful
> > console
> > error message if an external module (e.g. vm-display-X11) fails
> > to load.
> That's one for Ian.
> >
> > platforms-unix-config-config.h.in.diff - Add a definition to unix/
> > config/config.in
> > to #define SQAIO_H "sqaio.h" due to renaming aio.h to sqaio.h.
> > Permits OSPP
> > and AIO plugins to be backward compatible with older source
> > trees. (Note
> > to Tim: RiscOS uses a copy of aio.h in its platform tree,
> > should sqaio.h
> > be moved to Cross?)
> Well, maybe it should. I suppose if it is used by two or more
> platforms it makes a reasonable candidate for Cross. It is another
> posixy rather than ansi-ish file though.
>
>
> tim
> --
> tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
> Never write software that anthropomorphizes the machine. They hate that.
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vmPatchesForSVN1352.zip
Type: application/x-zip-compressed
Size: 3101 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20060312/b6daf36a/vmPatchesForSVN1352.bin
More information about the Vm-dev
mailing list