On 27.04.2009, at 18:17, José L. Redrejo Rodríguez wrote:
El lun, 27-04-2009 a las 07:38 -0400, David T. Lewis escribió:
See http://bugs.squeak.org/view.php?id=7103
With these changes, sound works on 64-bit Linux ALSA and OSS for both 32-bit and 64-bit images. There are changes to Cross/plugins/SoundPlugin/ SoundPlugin.h so some updates will be required for win32 and Mac OS (minor changes to pass pointers properly).
Hi David. I think there are still some code that must be patched and you forgot to include in the patch in mantis. I've compiled the vm, patching the 3.10.4 sources and it still segfaults (using Debian Lenny + alsa).
I'm attaching the patch I've used (it should be the same patch you sent, but unified and all in one file, but it might help to know what is missed). The log is:
patching file platforms/Cross/plugins/SoundPlugin/SoundPlugin.h patching file platforms/unix/plugins/SoundPlugin/sqUnixSound.c patching file platforms/unix/vm-sound-ALSA/sqUnixSoundALSA.c patching file platforms/unix/vm-sound-MacOSX/sqUnixSoundMacOSX.c patching file platforms/unix/vm-sound-NAS/sqUnixSoundNAS.c patching file platforms/unix/vm-sound-null/sqUnixSoundNull.c patching file platforms/unix/vm-sound-OSS/sqUnixSoundOSS.c patching file platforms/unix/vm-sound-Sun/sqUnixSoundSun.c patching file platforms/unix/vm/SqSound.h
Regards José L. <sound64bits.dpatch>
You also need to regenerate the src/plugins/SoundPlugin.c file after applying the SoundPlugin-64bit-dtl.1.cs changeset.
- Bert -