Yes, the patch looks good.

/Robert

On 01/18/2012 06:38 PM, Kelly O'Hair wrote:
Patch looks fine to me.

-kto

On Jan 13, 2012, at 9:54 AM, mark.reinh...@oracle.com wrote:

2012/1/13 9:33 -0800, michael.j.silvers...@gmail.com:
I am getting the following error when building openjdk 7.0 on ubuntu 11.10
But I already have the package libasound2-dev installed.
This is 7110396.  You need this patch:

diff --git a/make/javax/sound/jsoundalsa/Makefile 
b/make/javax/sound/jsoundalsa/Makefile
--- a/make/javax/sound/jsoundalsa/Makefile
+++ b/make/javax/sound/jsoundalsa/Makefile
@@ -65,7 +65,7 @@
        $(MIDIFILES_export) \
        $(PORTFILES_export)

-LDFLAGS += -lasound
+OTHER_LDLIBS += -lasound

CPPFLAGS += \
        -DUSE_DAUDIO=TRUE \

This is a tiny revision of the patch I originally proposed [1].
If someone could review this then I'll merge and push it.

- Mark


[1] http://mail.openjdk.java.net/pipermail/build-dev/2011-November/005188.html


--
Oracle
Robert Ottenhag | Senior Member of Technical Staff
Phone: +46850630961 | Fax: +46850630911 | Mobile: +46707106161
Oracle Java HotSpot Virtual Machine
ORACLE Sweden | Folkungagatan 122 | SE-116 30 Stockholm

Oracle Svenska AB, KronborgsgrÀnd 17, S-164 28 KISTA, reg.no. 556254-6746

Green Oracle

Oracle is committed to developing practices and products that help protect the 
environment
--

Reply via email to