Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-java
Navigation:
Lists: gentoo-java: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-java@g.o
From: Petteri R├Ąty <betelgeuse@g.o>
Subject: Re: Re: Installing Javadoc documentation with java-pkg_dohtml
Date: Wed, 21 Dec 2005 13:10:25 +0200
Ernst de Haan wrote:
>>While I'm at it, attached is the current version of my xmlenc ebuild.
>>As this is my very first Ebuild, feedback is highly appreciated.
> 
> 
> This time as a plain text .ebuild file. Comments are highly appreciated!
> 

Here we go.

> KEYWORDS="~x86 ~ppc ~sparc ~amd64"

Do you have all these arches to test with?
http://dev.gentoo.org/~plasmaroo/devmanual/keywording/

> src_compile() {
> 	local antflags="jar"
> 	use debug && antflags="-Djavac.debug=true ${antflags}"
> 	use jikes && antflags="-Dbuild.compiler=jikes ${antflags}"
> 	einfo "Ant flags: ${antflags}"

We don't einfo the antflags like this because there is nothing of
interest to the user. Feel free to use them for debugging but don't
leave them to the final ebuild.

> 	ant ${antflags} || die "Compilation failed."
> 	use doc && ( ant javadoc || die "Javadoc generation failed." )

Using parenthesis like this makes a subshell ( see man bash ). dieing
inside a subshell will not quit like you expect it to. Why not just add
javadoc to antflags?

> }
> 
> src_install() {
> 	java-pkg_dojar build/xmlenc.jar

java-pkd_dojar build/*.jar is probably more future proof so when version
bumping new jars are not missed.

Please note that these comments are only based on looking at the code. I
might find more if I ever get to actually testing it.

Thanks for the work so far.
http://www.gentoo.org/doc/en/ebuild-submit.xml

Regards,
Petteri
Attachment:
signature.asc (OpenPGP digital signature)
Replies:
Re: Re: Installing Javadoc documentation with java-pkg_dohtml
-- Ernst de Haan
References:
Installing Javadoc documentation with java-pkg_dohtml
-- Ernst de Haan
Re: Installing Javadoc documentation with java-pkg_dohtml
-- Ernst de Haan
Re: Installing Javadoc documentation with java-pkg_dohtml
-- Ernst de Haan
Navigation:
Lists: gentoo-java: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: Installing Javadoc documentation with java-pkg_dohtml
Next by thread:
Re: Re: Installing Javadoc documentation with java-pkg_dohtml
Previous by date:
Re: Installing Javadoc documentation with java-pkg_dohtml
Next by date:
Re: Re: Installing Javadoc documentation with java-pkg_dohtml


Updated Jun 17, 2009

Summary: Archive of the gentoo-java mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.