Looks good. Once the Findbugs POMs are rectified you could drop explicitly naming coreplugin.
--
Regards,
Garvin LeClaire
garvin.leclaire@xxxxxxxxx
On 10/6/06, Fabian Ritzmann <ritzmann@xxxxxxxxxxxxxxxxxxxxx> wrote:
Garvin LeClaire wrote:
> That may not work as the Findbugs POM does not declare its dependency
> on the coreplugin while coreplugin shows a dependency on findbugs.
> This is wrong and should be reversed. It gave me fits when trying to
> get the minimum dependencies listed in the plugin POM and let the
> transitive dependency mechanism of Maven do the rest.
I see. I think I'll go for a "compromise":
<dependencies pathId="
findbugs.classpath"
useScope="runtime">
<dependency groupId="net.sourceforge.findbugs"
artifactId="findbugs"
version="
1.0.0"/>
<dependency groupId="net.sourceforge.findbugs"
artifactId="coreplugin"
version="1.0.0"/>
</dependencies>
Fabian
> On 10/6/06, *Fabian Ritzmann* <ritzmann@xxxxxxxxxxxxxxxxxxxxx
> <mailto:ritzmann@xxxxxxxxxxxxxxxxxxxxx
>> wrote:
>
> Garvin LeClaire wrote:
> > Yes there is a POM issue in the repository. If you add the
> following
> > you should be fine.
> >
> > <dependency>
> > <groupId>net.sourceforge.findbugs </groupId>
> > <artifactId>coreplugin</artifactId>
> > <version>1.0.0</version>
> > </dependency>
> >
> > The POM for findbugs is incorrect. The coreplugin defines some
> of the
> > dependencies that are missing from the findbugs POM.
>
> Even better, I just replaced the entire mess by this:
>
> <dependencies pathId="findbugs.classpath"
> useScope="runtime">
> <dependency groupId="net.sourceforge.findbugs
"
> artifactId="findbugs"
> version="1.0.0"/>
> </dependencies>
>
> Thanks, Maven works so much better when your POM has the right
> dependencies. :-)
>
> > Once again this is all working in the FingBugs Maven plugin at
> > http://mojo.codehaus.org <
http://mojo.codehaus.org>
>
> I had already wondered how it could have worked with that dependency
> mess. I'm using the Maven Antlib, i.e. the build scripts are all
> Ant and
> it uses the Maven repository to download external libraries on
> demand.
>
> Fabian
>
>
>
| 