Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 242746 - [java-overlay] dev-java/icedtea6-1.3-r1 autoconf issues
Summary: [java-overlay] dev-java/icedtea6-1.3-r1 autoconf issues
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Java (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Java team
URL: http://icedtea.classpath.org/bugzilla...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-19 11:27 UTC by Alon Bar-Lev
Modified: 2008-10-20 20:45 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
icedtea6-1.3-autoconf.patch (icedtea6-1.3-autoconf.patch,1.49 KB, patch)
2008-10-19 11:28 UTC, Alon Bar-Lev
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.