Maven 1.x JDepend Plugin

more elements in jdepend classpath

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.6
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    1

Description

jdepend search for its jdepend.properties through classloader,
so it is good idea to add ${maven.build.dest} to classpath,
to enable setting properties per project.

You can then add
<preGoal name="maven-jdepend-plugin:report">
<copy failonerror="false" file="jdepend.properties" tofile="${maven.build.dest}/jdepend.properties"/>
</preGoal>

and you no longer have to have jdepend.properties in your home directory.

Patch:
— src/plugins-build/jdepend/plugin.jelly.orig Tue Jul 29 14:08:49 2003
+++ src/plugins-build/jdepend/plugin.jelly Tue Jul 29 14:09:10 2003
@@ -42,6 +42,7 @@
<ant:path refid="maven-classpath"/>
<ant:path refid="maven.dependency.classpath"/>
<ant:pathelement path="${plugin.getDependencyPath('jdepend')}"/>
+ <ant:pathelement path="${maven.build.dest}"/>
</ant:classpath>
</ant:java>

Activity

Hide
Norbert Pabis added a comment -

Sorry for patch in comment.
During adding the issue you cannot add patch, therefore I thought
it is impossible.

Show
Norbert Pabis added a comment - Sorry for patch in comment. During adding the issue you cannot add patch, therefore I thought it is impossible.
Hide
dion gillard added a comment -

Wouldn't it be better if the jdepend.properties was specified as a project property, rather than requiring a pre-goal?

Show
dion gillard added a comment - Wouldn't it be better if the jdepend.properties was specified as a project property, rather than requiring a pre-goal?
Hide
Norbert Pabis added a comment -

> Wouldn't it be better if the jdepend.properties was specified
> as a project property, rather than requiring a pre-goal?
Sounds cool for me. But it has to be in classpath anyway, because
jdepend works like this.

Show
Norbert Pabis added a comment - > Wouldn't it be better if the jdepend.properties was specified > as a project property, rather than requiring a pre-goal? Sounds cool for me. But it has to be in classpath anyway, because jdepend works like this.
Hide
Lukas Theussl added a comment -

Introduced maven.jdepend.properties property to specify a custom properties file.

Show
Lukas Theussl added a comment - Introduced maven.jdepend.properties property to specify a custom properties file.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: