1 |
This is in regards to bug 35412, |
2 |
http://bugs.gentoo.org/show_bug.cgi?id=35412 |
3 |
|
4 |
To anyone who maintains an ebuild that is linked against alsa-lib; |
5 |
With alsa-lib-1.0 there was a change in the api; the library is still |
6 |
binary compatible, but it needs 2 defintions prior to the |
7 |
(sys|alsa)/asoundlib.h include for compilation to not crash out. |
8 |
|
9 |
The following code pretty much summs up what is needed; the version |
10 |
check is debatable how useful it is, since those definitions don't |
11 |
appear to affect 0.9.* in any way. |
12 |
|
13 |
#include <alsa/version.h> |
14 |
#if SND_LIB_MAJOR==1 |
15 |
#define ALSA_PCM_OLD_HW_PARAMS_API |
16 |
#define ALSA_PCM_NEW_HW_PARAMS_API |
17 |
#endif |
18 |
#include <alsa/asoundlib.h> |
19 |
|
20 |
If the original src included sys/asoundlib (the old, deprecated |
21 |
location), it's debatable whether to have it point at the new location, |
22 |
or leave it at the old (and leave out the <alsa/version.h>/snd_lib_major |
23 |
conditional). If it's using the old location, the upstream dev should |
24 |
be pestered about it if it hasn't changed in the latest version... |
25 |
|
26 |
Please take a quick scan through this-> |
27 |
http://bugs.gentoo.org/attachment.cgi?id=21912&action=view . It's a |
28 |
quick scan of packages that have media-lib/alsa-lib as a dependency. I |
29 |
haven't stripped out versi |
30 |
|
31 |
All packages that relied on the 0.9.* api ought to be affected by this. |
32 |
Thanks, |
33 |
~brian |