Date: Thu, 8 Jan 2004 11:24:24 +0100 From: Bruno Van Den Bossche <bruno.van.den.bossche@pandora.be> To: Pete Fritchman <petef@absolutbsd.org> Cc: bug-followup@freebsd.org Subject: Re: ports/57956: New port: java/eclipse-pmd plugin Message-ID: <20040108112424.437359f8.bruno.van.den.bossche@pandora.be> In-Reply-To: <20040108041430.B0305177C6@sirius.firepipe.net> References: <20040107114226.5165aa4c.bruno.van.den.bossche@pandora.be> <20040108041430.B0305177C6@sirius.firepipe.net>
next in thread | previous in thread | raw e-mail | index | archive | help
Pete Fritchman <petef@absolutbsd.org> wrote: > * Wed, 07 Jan 2004 11:42:26 +0100 - Bruno Van Den Bossche: > | > http://www.freebsd.org/cgi/query-pr.cgi?pr=57956 > | The adjusted shar: > > Ok. A few more things I missed while looking through the first time: > > * Do we have to install stuff into ${PREFIX}/eclipse? How about > ${DATADIR}? With DATADIR= "${PREFIX}/eclipse" ? If not, I'm not sure what you mean. > * using ${CP} -R might have bad side effects. perhaps cpio, like some > other ports? I originally made the port using eclipse-gef as an example, which also uses ${CP} -R. Should the maintainer be notified? Changed, although I wasn't completely sure what variables for uid and gid had to be used. Some ports use an absolute path when defining CPIO. Is there a reason for this? Because doing so gives me a warning when running portlint on the port. > Other than that, we're looking good! Great, thanks! updated 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 XCPIO?= cpio --quiet -pdmu -R X XPLUGIN_DIR= "net.sourceforge.pmd.eclipse_${PORTVERSION}" XDATADIR= "${PREFIX}/eclipse" X XPLIST_SUB+= PLUGIN_DIR="${PLUGIN_DIR}" X Xdo-install: X @${MKDIR} ${DATADIR} X @${MKDIR} ${DATADIR}/plugins X @${MKDIR} ${DATADIR}/plugins/${PLUGIN_DIR} X cd ${WRKSRC}/${PLUGIN_DIR} && ${FIND} . \ X | ${CPIO} ${SHAREOWN}:${SHAREGRP} \ X ${DATADIR}/plugins/${PLUGIN_DIR} 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 Every creature has within him the wild, uncontrollable urge to punt.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040108112424.437359f8.bruno.van.den.bossche>