Bug 178383 - PATCH: allegro default cfg should use timidy wavetable patches
Summary: PATCH: allegro default cfg should use timidy wavetable patches
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: allegro
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jindrich Novy
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-01-19 22:36 UTC by Hans de Goede
Modified: 2013-07-02 23:13 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-01-25 10:58:02 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
allegrorc patch (251 bytes, patch)
2006-01-19 22:36 UTC, Hans de Goede
no flags Details | Diff

Description Hans de Goede 2006-01-19 22:36:26 UTC
Most allegro apps use the following code to initialise sound:
install_sound (DIGI_AUTODETECT, MIDI_AUTODETECT, NULL)

This however currently will fail on most motherboards with the FE-devel-CVS
allegro since most motherboards don't have a real midi chip these days (only
digital/wave out) and thus allegro tries to use the buildin softwavetable synth,
this however needs a patchset to work. Most apps now thus don't have sound
because they disable sound if this call fails.

By adding patches = /usr/share/timidity/timidity.cfg, and requiering timidity++,
which provides this files and the patches referenced therein, this problem can
be fixed. I've attached a patch against the default allegrorc with this fix.

Comment 1 Hans de Goede 2006-01-19 22:36:26 UTC
Created attachment 123460 [details]
allegrorc patch

Comment 2 Jindrich Novy 2006-01-24 12:37:20 UTC
Hans,

I tested it with allegro-4.2.0-4 on my rawhide box as well as with allegro-4.0.3
on my FC4 box.

I tried it with the following source:

#include <allegro.h>

int main() {
        allegro_init();
        install_sound (DIGI_AUTODETECT, MIDI_AUTODETECT, NULL);

        return 0;
}
END_OF_MAIN();

and everything works as expected, so no error message is reported by
install_sound on my i386 (Intel chipset).

What is the error you see?

Comment 3 Hans de Goede 2006-01-25 06:20:28 UTC
that shoud read:

if (install_sound (DIGI_AUTODETECT, MIDI_AUTODETECT, NULL) == -1)
   printf("error: %s\n", allegro_error);

Then I got an error that it couldnot find a suitable midi device or something
similar. I found out that it had soft wavetable support by doing a strace and
seeing that it couldnot open default.cfg (the default patches config name).


Comment 4 Jindrich Novy 2006-01-25 10:07:05 UTC
Ok. Sounds good to me. Sound initialization shows the error on my system as
well. alegrorc is now updated, thanks.

Comment 5 Hans de Goede 2006-01-25 10:24:42 UTC
Thanks, the timidity++ Requires should be a normal Require not a BuildRequire,
the timidity patches get used runtime.

Comment 6 Jindrich Novy 2006-01-25 10:58:02 UTC
allegro-4.2.0-5 is now built. I'll fix the timidity dependency in the next build.


Note You need to log in before you can comment on or make changes to this bug.