Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 563372 - net-p2p/i2p-0.9.22 fails to build
Summary: net-p2p/i2p-0.9.22 fails to build
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Java team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-10-18 06:13 UTC by Yutao Yuan
Modified: 2015-10-29 17:07 UTC (History)
7 users (show)

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


Attachments
i2p-0.9.22_fix-paths.patch.out (i2p-0.9.22_fix-paths.patch.out,8.75 KB, text/plain)
2015-10-18 06:13 UTC, Yutao Yuan
Details
build log (i2p-0.9.22:20151018-055432.log,94.42 KB, text/plain)
2015-10-18 06:14 UTC, Yutao Yuan
Details
output of emerge --info '=net-p2p/i2p-0.9.22' (out,5.53 KB, text/plain)
2015-10-18 06:17 UTC, Yutao Yuan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yutao Yuan 2015-10-18 06:13:59 UTC
Created attachment 414812 [details]
i2p-0.9.22_fix-paths.patch.out

In install phase, patch "i2p-0.9.22_fix-paths.patch" failed.

Relevant file are attached.
Comment 1 Yutao Yuan 2015-10-18 06:14:59 UTC
Created attachment 414814 [details]
build log
Comment 2 Yutao Yuan 2015-10-18 06:17:05 UTC
Created attachment 414816 [details]
output of emerge --info '=net-p2p/i2p-0.9.22'
Comment 3 Patrice Clement gentoo-dev 2015-10-18 13:52:37 UTC
Indeed. Currently putting together and testing a fix.
Comment 4 Patrice Clement gentoo-dev 2015-10-18 14:20:01 UTC
>>> Install i2p-0.9.22-r1 into /var/tmp/portage/net-p2p/i2p-0.9.22-r1/image/ category net-p2p
 * Applying i2p-0.9.22_fix-paths.patch ...                                                                                                                             [ ok ]
>>> Completed installing i2p-0.9.22-r1 into /var/tmp/portage/net-p2p/i2p-0.9.22-r1/image/

ecompressdir: bzip2 -9 /usr/share/doc
ecompressdir: bzip2 -9 /usr/share/man

commit 9613ea0 (HEAD, master)
Author: Patrice Clement <monsieurp@gentoo.org>
Date:   Sun Oct 18 14:18:04 2015 +0000

    net-p2p/i2p: Add missing dependencies and mend wrong paths in patch. Fixes bug 563372.
    
    Package-Manager: portage-2.2.20.1
    Signed-off-by: Patrice Clement <monsieurp@gentoo.org>

 create mode 100644 net-p2p/i2p/i2p-0.9.22-r1.ebuild

Try again and report back please. Thank you.
Comment 5 Alex Efros 2015-10-18 23:34:43 UTC
Now it installs ok, but fail to start:

FATAL  | wrapper  | Unable to open configuration file: /usr/share/i2p/wrapper.config (No such file or directory)

Looks like this file exists in 0.9.18 but missed in 0.9.22, not sure why - either it wasn't installed by mistake in ebuild, or upstream change location of this config.

BTW, can you please add 0.9.18 back to portage until issues with 0.9.22 will be resolved?
Comment 6 Yutao Yuan 2015-10-21 12:40:41 UTC
In i2p-0.9.22-r1.ebuild, there is a commented line "doins blocklist.txt hosts.txt *.config". It is uncommented in 0.9.18. Uncommenting it seems to work now. I'm not sure why it got commented, though.
Comment 7 Patrice Clement gentoo-dev 2015-10-21 13:54:49 UTC
diff --git a/net-p2p/i2p/i2p-0.9.22-r1.ebuild b/net-p2p/i2p/i2p-0.9.22-r1.ebuild
index 7bcfaff..4a06383 100644
--- a/net-p2p/i2p/i2p-0.9.22-r1.ebuild
+++ b/net-p2p/i2p/i2p-0.9.22-r1.ebuild
@@ -85,7 +85,7 @@ src_install() {
        # Install main files and basic documentation
        exeinto ${i2p_home}
        insinto ${i2p_home}
-       #doins blocklist.txt hosts.txt *.config
+       doins blocklist.txt hosts.txt *.config
        doexe eepget i2prouter runplain.sh
        dodoc history.txt INSTALL-headless.txt LICENSE.txt
        doman man/*

commit 87f5e01 (HEAD, origin/master, origin/HEAD, master)
Author: Patrice Clement <monsieurp@gentoo.org>
Date:   Wed Oct 21 13:42:32 2015 +0000

    net-p2p/i2p: Uncomment a doins line. Fixes bug 563372.
    
    Package-Manager: portage-2.2.20.1
    Signed-off-by: Patrice Clement <monsieurp@gentoo.org>

Give -r1 another try please and report back. Thank you.
Comment 8 Yutao Yuan 2015-10-21 14:18:34 UTC
After cleaning out bcprov:0 with depclean, the build now fails with error "Package bcprov was not found!". It looks like it is having some problems finding the slotted bcprov:1.50.

The 0.9.18 version depends on >=bcprov-1.50, while the 0.9.22-r1 one depends on bcprov:1.50. I don't know what the correct dependency should be, but given that 0.9.18 works with >=brprov-1.50, it should probably stay the same as 0.9.18.
Comment 9 Patrice Clement gentoo-dev 2015-10-21 14:24:05 UTC
Emerge bcprov:1.50 like so:

# emerge -v bcprov:1.50

and then emerge i2p as usual. We (Java team) have recently cleaned up bc ebuilds (it was a real mess) and SLOTted them back. Sorry for the noise.
Comment 10 Yutao Yuan 2015-10-21 14:53:02 UTC
OK, it is working now. Thanks for your effort.
Comment 11 Patrice Clement gentoo-dev 2015-10-21 15:18:03 UTC
Thank you for reporting back. Marking this bug as FIXED.
Comment 12 Pavel Goran 2015-10-24 09:16:45 UTC
For me, emerging i2p-0.9.22-r1 still fails unless I have dev-java/bcprov:0 installed. Here is what I get:

!!! ERROR: Package bcprov was not found!
 * ERROR: net-p2p/i2p-0.9.22-r1::gentoo failed (compile phase):
 *   java-config --classpath=java-service-wrapper,jrobin,slf4j-api,tomcat-jstl-impl,tomcat-jstl-spec,bcprov failed

I have bcprov-1.50-r2 installed. (I tried with 1.50-r4 as well.)

With bcprov:0 installed, this doesn't happen.

I suppose this line from ebuild is incorrect:

EANT_GENTOO_CLASSPATH="java-service-wrapper,jrobin,slf4j-api,tomcat-jstl-impl,tomcat-jstl-spec,bcprov"

"bcprov" should be replaced with "bcprov-1.50".
Comment 13 Patrice Clement gentoo-dev 2015-10-24 09:21:07 UTC
diff --git a/net-p2p/i2p/i2p-0.9.22-r1.ebuild b/net-p2p/i2p/i2p-0.9.22-r1.ebuild
index 4a06383..c1b1f02 100644
--- a/net-p2p/i2p/i2p-0.9.22-r1.ebuild
+++ b/net-p2p/i2p/i2p-0.9.22-r1.ebuild
@@ -35,7 +35,7 @@ RDEPEND="${CDEPEND}
        >=virtual/jre-1.6"
 
 EANT_BUILD_TARGET="pkg"
-EANT_GENTOO_CLASSPATH="java-service-wrapper,jrobin,slf4j-api,tomcat-jstl-impl,tomcat-jstl-spec,bcprov"
+EANT_GENTOO_CLASSPATH="java-service-wrapper,jrobin,slf4j-api,tomcat-jstl-impl,tomcat-jstl-spec,bcprov-1.50"
 
 pkg_setup() {
        enewgroup i2p

commit c792399 (HEAD, master)
Author: Patrice Clement <monsieurp@gentoo.org>
Date:   Sat Oct 24 09:18:40 2015 +0000

    net-p2p/i2p: Correct bcprov version in classpath. Fixes bug 563372.
    
    Package-Manager: portage-2.2.20.1
    Signed-off-by: Patrice Clement <monsieurp@gentoo.org>

Give it another try.
Comment 14 Pavel Goran 2015-10-29 05:20:51 UTC
With the new version, it builds successfully. However, it builds just as fine without bcprov-1.50 in EANT_GENTOO_CLASSPATH at all. Why is it there in the first place?
Comment 15 Patrice Clement gentoo-dev 2015-10-29 17:07:32 UTC
Pavel, truth to be told, I didn't delve further into figuring out which dependencies are needed and which aren't. Glad to know it works without bcprov. We'll remove it for the next revbump.