Ben, I suggested a while back a findbugs plugin I had written, and was told that because of the lgpl licensing issues, it needed to go elsewhere. So it is currently living at maven-plugins.sf.net. I have talked to the developers, and they are evaluating switching to an ASF friendly license that would allow findbugs to be run in the core.
Now, while some people may think that if 1 findbugs plugin is good, then 2 must be better, I take the approach that we should merge the best idea's out of both of our efforts! Do you want to review what I did and figure out how to take the best of both? Also, if the lgpl think isn't an issue, I would prefer to have the plugin be part of maven core to increase usage. Eric Pugh > -----Original Message----- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > Sent: Saturday, August 30, 2003 1:15 AM > To: [EMAIL PROTECTED] > Subject: cvs commit: maven/src/plugins-build/findbugs plugin.jelly > > > bwalding 2003/08/29 16:15:15 > > Modified: src/plugins-build/findbugs plugin.jelly > Log: > Was using incorrect sourcepath variable > > Revision Changes Path > 1.2 +4 -4 maven/src/plugins-build/findbugs/plugin.jelly > > Index: plugin.jelly > =================================================================== > RCS file: /home/cvs/maven/src/plugins-build/findbugs/plugin.jelly,v > retrieving revision 1.1 > retrieving revision 1.2 > diff -u -r1.1 -r1.2 > --- plugin.jelly 29 Aug 2003 01:55:48 -0000 1.1 > +++ plugin.jelly 29 Aug 2003 23:15:15 -0000 1.2 > @@ -21,10 +21,10 @@ > > <findbugs home="${maven.build.dir}/findbugshome" > output="text" > - debug="false"><!-- > - outputFile="${maven.build.dir}/findbugs.xml"--> > - <sourcePath path="${maven.compile.src.set}" /> > - <class location="${maven.build.dest}" /> > + debug="false" > + outputFile="${maven.build.dir}/findbugs.xml"> > + <sourcePath path="${pom.build.sourceDirectory}"/> > + <class location="${maven.build.dest}"/> > > <j:forEach var="lib" items="${pom.artifacts}"> > <auxClasspath path="${maven.repo.local}/${lib.urlPath}"/> > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
