Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 242746

Summary: [java-overlay] dev-java/icedtea6-1.3-r1 autoconf issues
Product: Gentoo Linux Reporter: Alon Bar-Lev <alon.barlev>
Component: [OLD] JavaAssignee: Java team <java>
Status: RESOLVED FIXED    
Severity: normal CC: gnu_andrew, rb6
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
URL: http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=224
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: icedtea6-1.3-autoconf.patch

Description Alon Bar-Lev 2008-10-19 11:27:18 UTC
Hello,

Due to invalid AC_ARG_ENABLE, --disable-X for some keys actually enables them.

Attached is a fix.

Thanks!
Comment 1 Alon Bar-Lev 2008-10-19 11:28:10 UTC
Created attachment 169072 [details, diff]
icedtea6-1.3-autoconf.patch
Comment 2 Petteri Räty (RETIRED) gentoo-dev 2008-10-19 11:56:05 UTC
(In reply to comment #1)
> Created an attachment (id=169072) [edit]
> icedtea6-1.3-autoconf.patch
> 

please submit this to the upstream bug tracker
http://icedtea.classpath.org/bugzilla/
Comment 3 Alon Bar-Lev 2008-10-19 12:55:50 UTC
Done.
Please add this patch so that people can merge current ebuild.
Comment 4 R Bar-On 2008-10-19 13:22:08 UTC
I have a different error on configure.  I have -pulseaudio set and don't have PA installed.  On configure, icedtea6-1.3-r1 fails with:

"checking for pulseaudio... no
configure: error: "pulseaudio was not found.""

If the use has -pulseaudio, I think the pulseaudio check should be disabled.
Comment 5 Alon Bar-Lev 2008-10-19 17:44:01 UTC
(In reply to comment #4)
> I have a different error on configure.  I have -pulseaudio set and don't have
> PA installed.  On configure, icedtea6-1.3-r1 fails with:
> 
> "checking for pulseaudio... no
> configure: error: "pulseaudio was not found.""
> 
> If the use has -pulseaudio, I think the pulseaudio check should be disabled.
> 

This is the same issue, use the attached patch to solve it.
Comment 6 R Bar-On 2008-10-19 18:32:25 UTC
It is not the same issue.  Applying the patch does not resolve it.
Comment 7 Alon Bar-Lev 2008-10-19 18:44:22 UTC
It is.
The ebuild use --disable-pulse-java but the autoconf actually enables it.
You should apply the patch to the ebuild src_unpack() phase before the eautoreconf.
Comment 8 R Bar-On 2008-10-19 18:51:05 UTC
ok, tell me what I'm doing wrong:

I do ebuild icedtea6-1.3-r1 unpack, go into the work directory, apply the patch, then run ebuild icedtea6-1.3-r1 configure, then ebuild icedtea6-1.3-r1 compile.  It fails at the same point I mentioned.
Comment 9 Alon Bar-Lev 2008-10-19 18:59:09 UTC
This is not the way.

Copy the icedtea6 directory to your overlay, should be something like:
/usr/local/portage/dev-java/icedtea6

Make sure that the following is available at /etc/make.conf:
PORTDIR_OVERLAY="/usr/local/portage"

Modify icedtea6-1.3-r1.ebuild, add:
epatch "${FILESDIR}/${P}-autoconf.patch"

Just before:
eautoreconf || die "failed to regenerate autoconf infrastructure"

Run:
ebuild icedtea6-1.3-r1.ebuild digest

And you are ready to go.

java: It would be nicer if you do this...
Comment 10 R Bar-On 2008-10-19 19:18:42 UTC
ok, works now :)

thanks!

Java people: please add this to the overlay so that the ebuild actually works...
Comment 11 Petteri Räty (RETIRED) gentoo-dev 2008-10-20 20:45:28 UTC
Committed.