You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by "Dave Sag (JIRA)" <ji...@codehaus.org> on 2005/10/10 15:05:11 UTC

[jira] Created: (MNG-1158) PMD is claiming my interface needs a constructor.

PMD is claiming my interface needs a constructor.
-------------------------------------------------

         Key: MNG-1158
         URL: http://jira.codehaus.org/browse/MNG-1158
     Project: Maven 2
        Type: Bug
  Components: maven-pmd-plugin  
    Versions: 2.0-beta-3    
 Reporter: Dave Sag
    Priority: Critical


When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.

this is crazy talk - Address.java is an interface class.

i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.



-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-1158?page=comments#action_51473 ] 

Brett Porter commented on MNG-1158:
-----------------------------------

Dave, you are welcome to work on this - I think it is creeping up the priority list, but there are a few other things on too.

One thing regarding the XML output - we probably want to build the dashboarding capability into the Maven reporting API in some respects, so I'm not sure that's the best way to go.

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag

>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Dave Sag (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-1158?page=comments#action_48410 ] 

Dave Sag commented on MNG-1158:
-------------------------------

haha probably yes.


> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag

>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MNG-1158?page=all ]

Brett Porter updated MNG-1158:
------------------------------

    Priority: Major  (was: Critical)
        type: Task  (was: Bug)
     Summary: PMD should use "sensible" default rulesets  (was: PMD is claiming my interface needs a constructor.)

we should probably match what the m1 plugin provides as a default.

currently, the only ruleset is "controversial.xml".

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag

>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Mark J. Titorenko (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MNG-1158?page=all ]

Mark J. Titorenko updated MNG-1158:
-----------------------------------

    Attachment: MNG-1158-maven-pmd-plugin.patch

Sorry, I didn't read the patch guidelines before attaching.  It's the same patch, but the diff is now performed from the correct place and it's following the naming scheme.

Apologies for the confusion.

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Closed: (MPMD-4) PMD should use "sensible" default rulesets

Posted by "mike perham (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MPMD-4?page=all ]
     
mike perham closed MPMD-4:
--------------------------

     Resolution: Fixed
    Fix Version: 2.0-beta-1

Added Locator patch.  Thanks Mark.  Changed default rulesets to the same as with the m1 plugin.

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MPMD-4
>          URL: http://jira.codehaus.org/browse/MPMD-4
>      Project: Maven 2.x Pmd Plugin
>         Type: Task

>     Reporter: Dave Sag
>      Fix For: 2.0-beta-1
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MPMD-4) PMD should use "sensible" default rulesets

Posted by "patrick OShea (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPMD-4?page=comments#action_54041 ] 

patrick OShea commented on MPMD-4:
----------------------------------

I've created http://jira.codehaus.org/browse/MPMD-8?rc=1 and uploaded a patch

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MPMD-4
>          URL: http://jira.codehaus.org/browse/MPMD-4
>      Project: Maven 2.x Pmd Plugin
>         Type: Task

>     Reporter: Dave Sag
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Mark J. Titorenko (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MNG-1158?page=all ]

Mark J. Titorenko updated MNG-1158:
-----------------------------------

    Attachment: Locator.java
                PmdReport.java.diff

I've attached a diff for the PmdReport class which requires the report to be supplied with a rulesetLocation configuration parameter.

Also attached is a duplicated version of the Locator class from the checkstyle plugin, which allows the ruleset to be stored either in the filesystem, as a classpath resource or on a web server. I imagine this kind of common functionality needs to be pushed up into the maven codebase somewhere, but don't have enough knowledge to suggest where!

This should be enough to fix the issue for this ticket - I don't believe there is a de facto "sensible" default ruleset and, instead, I think that anybody using this plugin should be choosing which rulesets to be using in their environment.

I think the XML output is a seperate issue for which an additional ticket should be opened.

I don't have developer access to the maven repository, so I'm hoping somebody can take this patch and apply it appropriately!


> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag
>  Attachments: Locator.java, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MPMD-4) PMD should use "sensible" default rulesets

Posted by "mike perham (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPMD-4?page=comments#action_55248 ] 

mike perham commented on MPMD-4:
--------------------------------

I have submitted MPMD-2 and it allows you to configure the rulesets used by PMD.

    <reporting>
        <plugins>
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-pmd-plugin</artifactId>
		<configuration>
			<rulesets>
				<ruleset>basic</ruleset>
				<ruleset>controversial</ruleset>
			</rulesets>
                </configuration>
            </plugin>
        </plugins>
    </reporting>

where the two rulesets are assumed to reside in the PMD standard location on the classpath: /rulesets/[name].xml.  Is this flexible enough to close this issue?

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MPMD-4
>          URL: http://jira.codehaus.org/browse/MPMD-4
>      Project: Maven 2.x Pmd Plugin
>         Type: Task

>     Reporter: Dave Sag
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Dave Sag (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-1158?page=comments#action_51617 ] 

Dave Sag commented on MNG-1158:
-------------------------------

Unfortunately I need the XML output so I can pipe that into QALab.  Am curently writing a QALab suite of plugins and reports for m2 so will attack PMD when I have this done.

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag

>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Dave Sag (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-1158?page=comments#action_50313 ] 

Dave Sag commented on MNG-1158:
-------------------------------

this error in the PMD plugin is a blocker for me.  also i need a way to get PMD to generate XML output.



> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag

>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MPMD-4) PMD should use "sensible" default rulesets

Posted by "John Allen (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPMD-4?page=comments#action_55258 ] 

John Allen commented on MPMD-4:
-------------------------------

Over the weekend i took this code and MPMD-8 patches and went through the pmd:pmd and pmd:execute Mojos refactoring and aligning them with the other report and analysis plugins I use (checkstyle, coberutra, pmd)

I would like to suggest that  the ruleSets config approach is adopted for both Mojos but modified to use the Locator lifted from checkstyle (MPMD-4) so that one is able to specify any number of rulesets, either from the pmd Jar or custom ones.

My current impl of both the pmd:pmd report Mojo and the pmd:execute Mojo are configured thus:

  				<configuration>
  					<ruleSets>
  						<ruleSet>D:\mine.xml</ruleSet>  					
  						<ruleSet>http:///localhost/basic.xml</ruleSet>
  						<ruleSet>/rulesets/basic.xml</ruleSet>
  						<ruleSet>/rulesets/controversial.xml</ruleSet>
  					</ruleSets>
  				</configuration>

Additionally I took the external report generation functionality from the execute mojo and added it to the report mojo. This is useful for integrating the output into other tools (CC, QALab) as perviously stated although i made it optional as the real item of interst is the generated site/pmd.html file.

I would like to respectfully ask that these changes be applied to your efforts before this plugin is released as its trivial to do and provides greater flexibility as well as functionality.

Note I can supply my code but as the plugin is trivial there doesnt seem much point.




> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MPMD-4
>          URL: http://jira.codehaus.org/browse/MPMD-4
>      Project: Maven 2.x Pmd Plugin
>         Type: Task

>     Reporter: Dave Sag
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-1158?page=comments#action_53215 ] 

Brett Porter commented on MNG-1158:
-----------------------------------

Patrick - can you submit a patch in a new JIRA issue?

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task

>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MPMD-4) PMD should use "sensible" default rulesets

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPMD-4?page=comments#action_55251 ] 

Brett Porter commented on MPMD-4:
---------------------------------

Mike - I think we need to do two things to close the issue:
- review the attached patches (sorry, I'd missed them) to see if they add anything over what is already available
- change the default ( I think you set it to still be controversial). Maybe look to the m1 plugin for guidance on a sensible default.

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MPMD-4
>          URL: http://jira.codehaus.org/browse/MPMD-4
>      Project: Maven 2.x Pmd Plugin
>         Type: Task

>     Reporter: Dave Sag
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "patrick OShea (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-1158?page=comments#action_52964 ] 

patrick OShea commented on MNG-1158:
------------------------------------

Hi, 
I have written a maven 2 plugin class that outputs the pmd data during a build in formats other than html.
Is there some where I could release this to?

Patrick

> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task

>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag
>  Attachments: Locator.java, MNG-1158-maven-pmd-plugin.patch, PmdReport.java.diff
>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MNG-1158) PMD should use "sensible" default rulesets

Posted by "Dave Sag (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-1158?page=comments#action_50314 ] 

Dave Sag commented on MNG-1158:
-------------------------------

Looking at the src of PMD's ant plugin it doesn't seem too hard to bring the maven 2 plugin up to speed.

http://cvs.sourceforge.net/viewcvs.py/pmd/pmd/src/net/sourceforge/pmd/ant/PMDTask.java?view=markup

specically I would like to add support for configurable rule sets, change the default to Basic from Controversial, and add support for other output formats.

I can't get onto SVN from in here due to a firewall issue, but could work on this from home if no-one else is working on it already.

cheers

dave


> PMD should use "sensible" default rulesets
> ------------------------------------------
>
>          Key: MNG-1158
>          URL: http://jira.codehaus.org/browse/MNG-1158
>      Project: Maven 2
>         Type: Task
>   Components: maven-pmd-plugin
>     Versions: 2.0-beta-3
>     Reporter: Dave Sag

>
>
> When I add a PMD report to my pom.xml the PMD report claims that my class 'Address' should have a constructor.
> this is crazy talk - Address.java is an interface class.
> i guess this is really a PMD bug but i find it hard to believe that such a well established tool as PMD would still be throwing up bugs like this.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org