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

Bug 184742

Summary: Include Sitemesh-2.3 into main tree
Product: Gentoo Linux Reporter: Jean-Noël Rivasseau (RETIRED) <elvanor>
Component: [OLD] JavaAssignee: Java team <java>
Status: RESOLVED INVALID    
Severity: normal    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Jean-Noël Rivasseau (RETIRED) gentoo-dev 2007-07-09 15:47:05 UTC
It is already in the java-experimental overlay, as well as all needed dependencies.

Reproducible: Always
Comment 1 Petteri Räty (RETIRED) gentoo-dev 2007-07-21 11:21:43 UTC
(In reply to comment #0)
> It is already in the java-experimental overlay, as well as all needed
> dependencies.
> 
> Reproducible: Always
> 

Some comments:
- Your RDEPEND and DEPEND have lots of common atoms. Just use something like COMMON_DEP and then have RDEPEND="${COMMON_DEP} <other stuff>"
- 	cd "${S}/src/java/com/opensymphony/module/sitemesh/html/tokenizer"
	epatch "${FILESDIR}/${P}-lexer.flex.patch"
  would be better to make this patch so that it applies from ${S}
- indentation is off. Run something like repoman 
- use java-ant_xml-rewrite in src_unpack
Comment 2 Jean-Noël Rivasseau (RETIRED) gentoo-dev 2007-07-23 10:11:31 UTC
I've cleaned up the ebuild as requested and commited the diffs. Sorry, as it was one of my first ebuilds.
Comment 3 Patrice Clement gentoo-dev 2015-01-25 11:07:20 UTC
Any update since 2007? Do we want this ebuild to make it into the tree? You know the drill: comment or close.
Comment 4 Patrice Clement gentoo-dev 2015-01-30 23:09:01 UTC
HOMEPAGE dead. We have to clean this off the java experimental overlay. I've just asked in #gentoo-infra for write access to it.
Comment 5 Patrice Clement gentoo-dev 2015-01-31 14:27:25 UTC
* commit 25747ca (HEAD, origin/master, origin/HEAD, master)
| Author: Patrice Clement <monsieurp@gentoo.org>
| Date:   Fri Jan 30 23:57:42 2015 +0100
|
|     HOMEPAGE dead. Removing it from the overlay.