Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 7 Jan 2004 11:42:26 +0100
From:      Bruno Van Den Bossche <bruno.van.den.bossche@pandora.be>
To:        Pete Fritchman <petef@FreeBSD.org>
Cc:        bruno.van.den.bossche@pandora.be
Subject:   Re: ports/57956: New port: java/eclipse-pmd plugin
Message-ID:  <20040107114226.5165aa4c.bruno.van.den.bossche@pandora.be>
In-Reply-To: <200401060647.i066lVKY084894@freefall.freebsd.org>
References:  <200401060647.i066lVKY084894@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Pete Fritchman <petef@FreeBSD.org> wrote:

> Synopsis: New port: java/eclipse-pmd plugin
> 
> State-Changed-From-To: open->feedback
> State-Changed-By: petef
> State-Changed-When: Tue Jan 6 00:47:05 CST 2004
> State-Changed-Why: 
> How about we make the version 1.1.2, and replace 1.1.2 with
> ${PORTVERSION}

Done.

> in the Makefile and %%VERSION%% (with a PLIST_SUB) in pkg-plist?

I've added a variable PLUGIN_DIR for use in the do-install subsection,
and the PLIST_SUB variable.
It seemed an appropriate change.

> http://www.freebsd.org/cgi/query-pr.cgi?pr=57956

I wasn't sure if I needed to just reply to this mail or submit a
follow-up to the PR so I just replied to this mail.  (Feel free to
point me in the right direction if I was wrong)


The adjusted shar:


# This is a shell archive.  Save it in a file, remove anything before
# this line, and then unpack it by entering "sh file".  Note, it may
# create directories; files and directories will be owned by you and
# have default permissions.
#
# This archive contains:
#
#	eclipse-pmd
#	eclipse-pmd/pkg-descr
#	eclipse-pmd/distinfo
#	eclipse-pmd/Makefile
#	eclipse-pmd/pkg-plist
#
echo c - eclipse-pmd
mkdir -p eclipse-pmd > /dev/null 2>&1
echo x - eclipse-pmd/pkg-descr
sed 's/^X//' >eclipse-pmd/pkg-descr << 'END-of-eclipse-pmd/pkg-descr'
XPMD scans Java source code and looks for potential problems like:
X
X    * Unused local variables
X    * Empty catch blocks
X    * Unused parameters
X    * Empty 'if' statements
X    * Duplicate import statements
X    * Unused private methods
X    * Classes which could be Singletons
X    * Short/long variable and method names
X
XWWW: http://pmd.sourceforge.net/
END-of-eclipse-pmd/pkg-descr
echo x - eclipse-pmd/distinfo
sed 's/^X//' >eclipse-pmd/distinfo << 'END-of-eclipse-pmd/distinfo'
XMD5 (eclipse/pmd-eclipse-bin-1.1.2.zip) = cd46079fc6c98e4bc6ff2586eb654353
END-of-eclipse-pmd/distinfo
echo x - eclipse-pmd/Makefile
sed 's/^X//' >eclipse-pmd/Makefile << 'END-of-eclipse-pmd/Makefile'
X# New ports collection makefile for:	eclipse-pmd
X# Date created:				October 3, 2003
X# Whom:					bruno.van.den.bossche@pandora.be
X#
X# $FreeBSD$
X#
X
XPORTNAME=	eclipse-pmd
XPORTVERSION=	1.1.2
XCATEGORIES=	java devel
XMASTER_SITES=	${MASTER_SITE_SOURCEFORGE}
XMASTER_SITE_SUBDIR=	pmd
XDISTNAME=	pmd-eclipse-bin-${PORTVERSION}
XDIST_SUBDIR=	eclipse
X
XMAINTAINER=	ports@FreeBSD.ORG
XCOMMENT=	PMD scans Java source code and looks for potential problems
X
XNO_BUILD=	yes
XNO_WRKSUBDIR=	yes
XUSE_ZIP=	yes
X
XPLUGIN_DIR=	"net.sourceforge.pmd.eclipse_${PORTVERSION}"
X
XPLIST_SUB+=	PLUGIN_DIR="${PLUGIN_DIR}"
X
Xdo-install:
X		@${MKDIR} ${PREFIX}/eclipse
X		@${MKDIR} ${PREFIX}/eclipse/plugins
X		@${MKDIR} ${PREFIX}/eclipse/plugins/${PLUGIN_DIR}
X		@${CP} -r ${WRKSRC}/${PLUGIN_DIR} ${PREFIX}/eclipse/plugins
X
X.include <bsd.port.mk>
END-of-eclipse-pmd/Makefile
echo x - eclipse-pmd/pkg-plist
sed 's/^X//' >eclipse-pmd/pkg-plist << 'END-of-eclipse-pmd/pkg-plist'
Xeclipse/plugins/%%PLUGIN_DIR%%/CHANGELOG.txt
Xeclipse/plugins/%%PLUGIN_DIR%%/LICENSE.txt
Xeclipse/plugins/%%PLUGIN_DIR%%/README.txt
Xeclipse/plugins/%%PLUGIN_DIR%%/RELNOTES.txt
Xeclipse/plugins/%%PLUGIN_DIR%%/TODO.txt
Xeclipse/plugins/%%PLUGIN_DIR%%/build.properties
Xeclipse/plugins/%%PLUGIN_DIR%%/doc.zip
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/error.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/file.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/info.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/pmd-icon-16.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/pmd-icon-32.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/prio_1.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/prio_2.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/prio_3.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/prio_4.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/prio_5.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/prj.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/remvio.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/icons/warn.gif
Xeclipse/plugins/%%PLUGIN_DIR%%/lib/commons-logging.jar
Xeclipse/plugins/%%PLUGIN_DIR%%/lib/jaxen-core-1.0-fcs.jar
Xeclipse/plugins/%%PLUGIN_DIR%%/lib/log4j-1.2.8.jar
Xeclipse/plugins/%%PLUGIN_DIR%%/lib/pmd-1.2.1.jar
Xeclipse/plugins/%%PLUGIN_DIR%%/lib/saxpath-1.0-fcs.jar
Xeclipse/plugins/%%PLUGIN_DIR%%/log4j.xml
Xeclipse/plugins/%%PLUGIN_DIR%%/messages.properties
Xeclipse/plugins/%%PLUGIN_DIR%%/nl/fr/messages.properties
Xeclipse/plugins/%%PLUGIN_DIR%%/plugin.properties
Xeclipse/plugins/%%PLUGIN_DIR%%/plugin.xml
Xeclipse/plugins/%%PLUGIN_DIR%%/plugin_fr.properties
Xeclipse/plugins/%%PLUGIN_DIR%%/pmd-eclipse.jar
Xeclipse/plugins/%%PLUGIN_DIR%%/toc.xml
Xshare/nls/POSIX
Xshare/nls/en_US.US-ASCII
X@dirrm eclipse/plugins/%%PLUGIN_DIR%%/nl/fr
X@dirrm eclipse/plugins/%%PLUGIN_DIR%%/nl
X@dirrm eclipse/plugins/%%PLUGIN_DIR%%/lib
X@dirrm eclipse/plugins/%%PLUGIN_DIR%%/icons
X@dirrm eclipse/plugins/%%PLUGIN_DIR%%
X@unexec rmdir %D/eclipse/plugins 2>/dev/null || true
X@unexec rmdir %D/eclipse/features 2>/dev/null || true
X@unexec rmdir %D/eclipse 2>/dev/null || true
END-of-eclipse-pmd/pkg-plist
exit


-- 
Bruno

Anthony's Law of Force:
	Don't force it; get a larger hammer.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040107114226.5165aa4c.bruno.van.den.bossche>