Summary: | dev-libs/xapian-bindings does not respect JAVACFLAGS and pulls php unconditionally | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Petteri Räty (RETIRED) <betelgeuse> |
Component: | [OLD] Java | Assignee: | Anthony Basile <blueness> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bugs, java, php-bugs |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Ebuild
Ebuild patch Patch to fix the issue Ebuild Ebuild patch xapian-bindings-1.2.7-r2.ebuild |
Description
Petteri Räty (RETIRED)
2007-04-28 23:20:53 UTC
Please try with dev-libs/xapian-bindings-1.0.16. (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. Why is this assigned to the Python team? Can we close it? 1.0.16 isn't in the tree anymore, for sure. 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. 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. 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 Attaching an ebuild (and its patch) and a patch to fix the issue. Ebuild also fixes other java eclass usage issues. Created attachment 286745 [details]
Ebuild
Created attachment 286747 [details, diff]
Ebuild patch
Created attachment 286749 [details, diff]
Patch to fix the issue
Created attachment 286751 [details]
Ebuild
Created attachment 286753 [details, diff]
Ebuild patch
Fixed unconditional php dependency as well. Thanks to olemarkus. 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 ) 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 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. Has this issue been addressed? If so, I'll close this. Both issues fixed. Thanks. |