67718 2004-10-15 14:16 0000 glob2-0.8.9 dies on startup due to badly written sound mixer function 2004-10-15 21:13:16 0000 1 1 1 Unclassified Gentoo Linux Games unspecified All Linux RESOLVED FIXED P2 normal --- 1 tetromino@gmail.com games@gentoo.org tetromino@gmail.com 2004-10-15 14:16:04 0000 Glob2-0.8.9 (and earlier versions, if the forum is to be believed) dies on startup unless esd is active (maybe it also works with arts, I am not sure). The problem is that in SoundMixer.cpp, the mixer thinks that since it asked for an SDL_AudioSpec to supply the mixer callback with 128k of data at a time, then SDL_AudioSpec will faithfully obey. As a matter of fact, SDL_AudioSpec does behave as expected when the mixer is around, but when playing to a raw alsa device, it only gives the callback function 2k of data at a time. This triggers a spurious assert() - the mixer callback has no features whatsoever that depend on an exact numerical value of the length of the data stream, but the authors of glob2 had deciced to practice some paranoid defensive programming... Reproducible: Always Steps to Reproduce: 1. make sure sound dameons aren't active 2. emerge glob2 3. $ glob2 Actual Results: Toolkit : Initialized : Graphic Context created Toolkit : Screen set to 640x480 at 0 bpp in window glob2: SoundMixer.cpp:70: void mixaudio(void*, Uint8*, int): Assertion `nsamples == 8192*8' failed. Aborted Expected Results: glob2 successfully runs tetromino@gmail.com 2004-10-15 14:18:46 0000 Created an attachment (id=41915) glob2-0.8.9.soundmixer.patch Patch to change the assertion to something more sensible. Also, typo above: change "SDL_AudioSpec does behave as expected when the mixer is around" to "SDL_AudioSpec does behave as expected when the sound daemon is around" tetromino@gmail.com 2004-10-15 14:19:56 0000 Created an attachment (id=41916) glob2-0.8.9.ebuild to use the patch above mr_bones_@gentoo.org 2004-10-15 20:22:07 0000 Let's have the output of emerge --info please. mr_bones_@gentoo.org 2004-10-15 21:13:16 0000 Turned down the assert checking. Upstream will pick this up for the next release. Thanks for the bug report. Resync and remerge for the fix. Since it works for me as is and for most people, I imagine, there's no rev bump. 41915 2004-10-15 14:18 0000 glob2-0.8.9.soundmixer.patch glob2-0.8.9.soundmixer.patch text/plain ZGlmZiAtciAtdWIgZ2xvYjItMC44Ljkvc3JjL1NvdW5kTWl4ZXIuY3BwIGdsb2IyLTAuOC45LmZp eGF1ZGlvL3NyYy9Tb3VuZE1peGVyLmNwcAotLS0gZ2xvYjItMC44Ljkvc3JjL1NvdW5kTWl4ZXIu Y3BwCTIwMDQtMDktMTQgMDQ6MDE6MzUuMDAwMDAwMDAwIC0wNDAwCisrKyBnbG9iMi0wLjguOS5m aXhhdWRpby9zcmMvU291bmRNaXhlci5jcHAJMjAwNC0xMC0xNSAxNjo0MjowMi45NTc3MjYzMTIg LTA0MDAKQEAgLTY3LDcgKzY3LDcgQEAKIAogCWFzc2VydChtaXhlci0+YWN0VHJhY2sgPj0gMCk7 CiAJYXNzZXJ0KG1peGVyLT5tb2RlICE9IFNvdW5kTWl4ZXI6Ok1PREVfU1RPUFBFRCk7Ci0JYXNz ZXJ0KG5zYW1wbGVzID09IFNBTVBMRV9DT1VOVF9QRVJfU0xJQ0UpOworCWFzc2VydChuc2FtcGxl cyk7CiAKIAlpZiAobWl4ZXItPm1vZGUgPT0gU291bmRNaXhlcjo6TU9ERV9FQVJMWV9DSEFOR0Up CiAJewo= 41916 2004-10-15 14:19 0000 glob2-0.8.9.ebuild to use the patch above glob2-0.8.9.ebuild text/plain IyBDb3B5cmlnaHQgMTk5OS0yMDA0IEdlbnRvbyBGb3VuZGF0aW9uCiMgRGlzdHJpYnV0ZWQgdW5k ZXIgdGhlIHRlcm1zIG9mIHRoZSBHTlUgR2VuZXJhbCBQdWJsaWMgTGljZW5zZSB2MgojICRIZWFk ZXI6IC92YXIvY3Zzcm9vdC9nZW50b28teDg2L2dhbWVzLXN0cmF0ZWd5L2dsb2IyL2dsb2IyLTAu OC45LmVidWlsZCx2IDEuMSAyMDA0LzEwLzE1IDA2OjMyOjQ0IG1yX2JvbmVzXyBFeHAgJAoKaW5o ZXJpdCBmbGFnLW8tbWF0aWMgZ2FtZXMKCkRFU0NSSVBUSU9OPSJzdGF0ZSBvZiB0aGUgYXJ0IFJl YWwgVGltZSBTdHJhdGVneSAoUlRTKSBnYW1lIgpIT01FUEFHRT0iaHR0cDovL3d3dy55c2Fnb29u LmNvbS9nbG9iMi8iClNSQ19VUkk9Imh0dHA6Ly93d3cueXNhZ29vbi5jb20vZ2xvYjIvZGF0YS8k e1B9LnRhci5neiIKCkxJQ0VOU0U9IkdQTC0yIgpTTE9UPSIwIgpLRVlXT1JEUz0ieDg2IH5wcGMg fmFtZDY0IgpJVVNFPSIiCgpERVBFTkQ9InZpcnR1YWwvbGliYwoJdmlydHVhbC9vcGVuZ2wKCT49 bWVkaWEtbGlicy9saWJzZGwtMS4yLjAKCW1lZGlhLWxpYnMvbGlicG5nCgltZWRpYS1saWJzL3Nk bC1uZXQKCW1lZGlhLWxpYnMvc2RsLWltYWdlCgltZWRpYS1saWJzL2xpYnZvcmJpcwoJPW1lZGlh LWxpYnMvZnJlZXR5cGUtMioKCXN5cy1saWJzL3psaWIiCgpzcmNfdW5wYWNrKCkgewoJdW5wYWNr ICR7QX0KCWNkICIke1N9IgoJZXBhdGNoICR7RklMRVNESVJ9LyR7UE59LTAuOC45LnNvdW5kbWl4 ZXIucGF0Y2gKCXNlZCAtaSBcCgkJLWUgInMvZGVmYXVsdExhbmd1YWdlID0gMS9kZWZhdWx0TGFu Z3VhZ2UgPSAwLyIgc3JjL1NldHRpbmdzLmNwcCBcCgkJfHwgZGllICJzZWQgZmFpbGVkIgp9Cgpz cmNfY29tcGlsZSgpIHsKCSMgY29tbWVudCBmcm9tIGJ1ZyAjNjQxNTAgdG8gZml4IGNvbXBpbGUg aXNzdWUuCglmaWx0ZXItZmxhZ3MgLU8/CgkjLi9jb25maWd1cmUgYXNzdW1lcyB0aGF0IHZvcmJp cyB3aWxsIGJlIGluc3RhbGxlZCB1bmRlciBQUkVGSVggYnVnICM0NjM1MgoJZWdhbWVzY29uZiBc CgkJLS13aXRoLXZvcmJpcz0vdXNyIFwKCQl8fCBkaWUKCWVtYWtlIHx8IGRpZSAiZW1ha2UgZmFp bGVkIgp9CgpzcmNfaW5zdGFsbCgpIHsKCW1ha2UgREVTVERJUj0iJHtEfSIgaW5zdGFsbCB8fCBk aWUgIm1ha2UgaW5zdGFsbCBmYWlsZWQiCglkb2RvYyBBVVRIT1JTIFJFQURNRSBUT0RPCglwcmVw Z2FtZXNkaXJzCn0K