Bug 176419 – dev-libs/xapian-bindings does not respect JAVACFLAGS and pulls php unconditionally

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Petteri Räty (RETIRED) 2007-04-28 23:20:53 UTC]

* Incorrect bytecode version found
 * in file: java/built/xapian_jni.jar
 * See /var/tmp/portage/dev-libs/xapian-bindings-0.9.9/temp/class-version-verify.log for more details.

!!! ERROR: dev-libs/xapian-bindings-0.9.9 failed.
Call stack:

http://overlays.gentoo.org/proj/java/wiki/Migrating_packages#autotoolsmake

Ask the Java team for help if you need it. Basically it should be generating 1.3 bytecode with the current deps.

[Comment 1 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2009-09-15 00:43:10 UTC]

Please try with dev-libs/xapian-bindings-1.0.16.

[Comment 2 Petteri Räty (RETIRED) 2009-09-15 19:15:10 UTC]

(In reply to comment #1)
> Please try with dev-libs/xapian-bindings-1.0.16.
> 

There's nothing to fix this in the latest ebuild. 

[Comment 3 Dirkjan Ochtman (RETIRED) 2010-08-23 12:08:51 UTC]

Why is this assigned to the Python team? Can we close it? 1.0.16 isn't in the tree anymore, for sure.

[Comment 4 Johan Bergström 2010-08-23 14:06:47 UTC]

Xapian-bindings is a swig wrapper that comes in many flavors. Java herd: Please join in with bringing the java parts of this ebuild up to date.

[Comment 5 Dirkjan Ochtman (RETIRED) 2010-10-27 15:24:14 UTC]

Reassigning this to the Java herd, as it seems to be a problem with the Java bindings for xapian, and I really don't think we have the knowledge to fix this.

[Comment 6 Johan Bergström 2011-09-15 08:11:57 UTC]

Please test with latest ebuild in bug 383047. I don't get this QA warning at least – but I'm far from taking a place in the java herd so to speak :(

Thanks

[Comment 7 Serkan Kaba (RETIRED) 2011-09-17 04:59:32 UTC]

Attaching an ebuild (and its patch) and a patch to fix the issue. Ebuild also fixes other java eclass usage issues.

[Comment 8 Serkan Kaba (RETIRED) 2011-09-17 05:00:14 UTC]

Created attachment 286745 [details]
Ebuild

[Comment 9 Serkan Kaba (RETIRED) 2011-09-17 05:01:16 UTC]

Created attachment 286747 [details, diff]
Ebuild patch

[Comment 10 Serkan Kaba (RETIRED) 2011-09-17 05:01:51 UTC]

Created attachment 286749 [details, diff]
Patch to fix the issue

[Comment 11 Serkan Kaba (RETIRED) 2011-09-17 05:36:48 UTC]

Created attachment 286751 [details]
Ebuild

[Comment 12 Serkan Kaba (RETIRED) 2011-09-17 05:37:15 UTC]

Created attachment 286753 [details, diff]
Ebuild patch

[Comment 13 Serkan Kaba (RETIRED) 2011-09-17 05:40:40 UTC]

Fixed unconditional php dependency as well. Thanks to olemarkus.

[Comment 14 Johan Bergström 2011-09-17 07:27:05 UTC]

Hello,
thanks a bunch for looking at the java-part of these bindings. I will do my best to verify the patch.

(In reply to comment #13)
> Fixed unconditional php dependency as well. Thanks to olemarkus.

php dep was fixed in the newly committed -r1 , but thanks for spending time with it!
(  http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/dev-libs/xapian-bindings/xapian-bindings-1.2.7-r1.ebuild?revision=1.1&view=markup )

[Comment 15 Johan Bergström 2011-09-17 15:16:40 UTC]

Created attachment 286779 [details]
xapian-bindings-1.2.7-r2.ebuild

Here's a new version that takes the java patches into account. I chose to use sed over patch, as well as patching .in instead of .am / autoreconf. Finally, I made the python sed conditional. Please have a run with the new ebuild.

There's One More Thing I'd like java-people assistance with, and that's running java/SmokeTest.java with FEATURES=test. If you have suggestions for this (there's a shell script in java/ available), please post them.

Thanks

[Comment 16 Anthony Basile 2011-09-18 11:20:17 UTC]

(In reply to comment #15)
> Created attachment 286779 [details]
> xapian-bindings-1.2.7-r2.ebuild
> 
> Here's a new version that takes the java patches into account. I chose to use
> sed over patch, as well as patching .in instead of .am / autoreconf. Finally, I
> made the python sed conditional. Please have a run with the new ebuild.
> 
> There's One More Thing I'd like java-people assistance with, and that's running
> java/SmokeTest.java with FEATURES=test. If you have suggestions for this
> (there's a shell script in java/ available), please post them.
> 
> Thanks

In the tree, please test.

[Comment 17 Anthony Basile 2011-09-27 15:44:26 UTC]

Has this issue been addressed?  If so, I'll close this.

[Comment 18 Serkan Kaba (RETIRED) 2011-09-27 15:57:33 UTC]

Both issues fixed. Thanks.