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

Bug 129511

Summary: commons-pool 1.3 bump
Product: Gentoo Linux Reporter: Chris Bainbridge (RETIRED) <chrb>
Component: New packagesAssignee: Java team <java>
Status: RESOLVED FIXED    
Severity: enhancement CC: alonbl
Priority: High    
Version: 2005.1   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 139215    
Attachments: commons-pool-1.3.ebuild.diff
commons-pool-1.3.ebuild.diff

Description Chris Bainbridge (RETIRED) gentoo-dev 2006-04-10 12:16:39 UTC
--- commons-pool-1.2.ebuild     2005-11-05 11:35:31.000000000 +0000
+++ commons-pool-1.3.ebuild     2006-04-10 20:14:20.000000000 +0100
@@ -18,6 +18,7 @@
 SLOT="0"
 KEYWORDS="amd64 ppc ppc64 sparc x86"
 IUSE="jikes junit doc"
+S="${WORKDIR}/${P}-src"
 
 src_unpack() {
        unpack ${A}
@@ -37,7 +38,7 @@
 }
 
 src_install() {
-       java-pkg_dojar dist/${PN}.jar
+       java-pkg_dojar dist/${P}.jar
        dodoc README.txt
        dohtml STATUS.html PROPOSAL.html
Comment 1 Josh Nichols (RETIRED) gentoo-dev 2006-04-10 12:20:30 UTC
By using ${P}.jar, that introduces installs a jar with the version in it. It is then difficult to pull jars from this package without having to know the version. Instead, java-pkg_newjar ${P}.jar ${PN}.jar should be used.

Alternatively, the build.xml could be tweaked (if it hasn't already been) to let you definea property to control the name of the build jar, and then add the appropriate value to antflags.
Comment 2 Alon Bar-Lev (RETIRED) gentoo-dev 2006-07-04 11:10:26 UTC
Created attachment 90871 [details, diff]
commons-pool-1.3.ebuild.diff

Updated patch.
Solves J2SE 1.5 as well.
Comment 3 Alon Bar-Lev (RETIRED) gentoo-dev 2006-07-05 10:00:19 UTC
Created attachment 90979 [details, diff]
commons-pool-1.3.ebuild.diff

Build using new build environment.
Comment 4 Petteri Räty (RETIRED) gentoo-dev 2007-01-16 18:01:40 UTC
(In reply to comment #3)
> Created an attachment (id=90979) [edit]
> commons-pool-1.3.ebuild.diff
> 
> Build using new build environment.
> 

Version bump added with support for src_test and source use flag support. apicheck  reported 100% so should be fine in the same slot.