Patch: fix alsa configury for systems with multilibs

Patch: fix alsa configury for systems with multilibs

[Anthony Green]

[-- Attachment #1: Type: text/plain, Size: 169 bytes --]

It's not enough to test that the alsa headers exist.  We also need to
make sure the libraries are there.  This was a problem for x86-64 linux.

Ok for everywhere?

AG



[-- Attachment #2: alsa.patch --]
[-- Type: text/x-patch, Size: 911 bytes --]

2005-12-01  Anthony Green  <green@redhat.com>

	* configure.ac: Make sure we have an alsa library in addition to
	the headers.  This extra test is required for systems with
	multilibs.
	* configure: Rebuilt.


Index: libjava/classpath/configure.ac
===================================================================
--- libjava/classpath/configure.ac	(revision 107822)
+++ libjava/classpath/configure.ac	(working copy)
@@ -112,7 +112,9 @@
                 no) COMPILE_ALSA=no ;;
                 *) COMPILE_ALSA=yes ;;
               esac],
-	      [AC_CHECK_HEADERS([alsa/asoundlib.h],COMPILE_ALSA=yes,COMPILE_ALSA=no)])
+	      [AC_CHECK_HEADERS([alsa/asoundlib.h],
+				[AC_CHECK_LIB([asound], [snd_seq_open], COMPILE_ALSA=yes,COMPILE_ALSA=no)],
+				COMPILE_ALSA=no)])
 AM_CONDITIONAL(CREATE_ALSA_LIBRARIES, test "x${COMPILE_ALSA}" = xyes)
 
 dnl -----------------------------------------------------------

Re: [cp-patches] Patch: fix alsa configury for systems with multilibs

[Mark Wielaard]

[-- Attachment #1: Type: text/plain, Size: 550 bytes --]

Hi Anthony,

On Thu, 2005-12-01 at 08:59 -0800, Anthony Green wrote:
> It's not enough to test that the alsa headers exist.  We also need to
> make sure the libraries are there.
>
> 2005-12-01  Anthony Green  <green@redhat.com>
>
>        * configure.ac: Make sure we have an alsa library in addition to
>        the headers.  This extra test is required for systems with
>        multilibs.
>        * configure: Rebuilt.
>
> Ok for everywhere?

Looks fine to me. (Note that configure isn't in CVS for classpath.)

Thanks,

Mark

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: Patch: fix alsa configury for systems with multilibs

[Tom Tromey]

>>>>> "Anthony" == Anthony Green <green@redhat.com> writes:

Anthony> It's not enough to test that the alsa headers exist.  We also
Anthony> need to make sure the libraries are there.  This was a
Anthony> problem for x86-64 linux.
Anthony> Ok for everywhere?

We talked on irc, but just for the record -- yes please, especially
the 4.1 branch.

Tom