You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Jan Palmquist (JIRA)" <ji...@codehaus.org> on 2007/02/17 22:43:01 UTC

[jira] Created: (MCHECKSTYLE-70) Support for multiple source folders

Support for multiple source folders
-----------------------------------

                 Key: MCHECKSTYLE-70
                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-70
             Project: Maven 2.x Checkstyle Plugin
          Issue Type: Improvement
    Affects Versions: 2.1
            Reporter: Jan Palmquist
            Priority: Minor


It would be great if this plugin would support multiple source folders added by http://mojo.codehaus.org/build-helper-maven-plugin/ (or similar), and by default inspect sources from these folders instead of just ${project.build.sourceDirectory}. Correspondingly with respect to test sources if those are configured to be included.

There are other plugins available solving this problem (somehow), eg:
* http://mojo.codehaus.org/jdepend-maven-plugin/
* http://mojo.codehaus.org/findbugs-maven-plugin/

Maybe they can give some inspiration for how to make this possible?

-- 
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

        

[jira] Commented: (MCHECKSTYLE-70) Support for multiple source folders

Posted by "Taciano Tres (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHECKSTYLE-70?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=226774#action_226774 ] 

Taciano Tres commented on MCHECKSTYLE-70:
-----------------------------------------

Is this already solved?

> Support for multiple source folders
> -----------------------------------
>
>                 Key: MCHECKSTYLE-70
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-70
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Improvement
>    Affects Versions: 2.1
>            Reporter: Jan Palmquist
>            Priority: Minor
>
> It would be great if this plugin would support multiple source folders added by http://mojo.codehaus.org/build-helper-maven-plugin/ (or similar), and by default inspect sources from these folders instead of just ${project.build.sourceDirectory}. Correspondingly with respect to test sources if those are configured to be included.
> There are other plugins available solving this problem (somehow), eg:
> * http://mojo.codehaus.org/jdepend-maven-plugin/
> * http://mojo.codehaus.org/findbugs-maven-plugin/
> Maybe they can give some inspiration for how to make this possible?

-- 
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

        

[jira] Commented: (MCHECKSTYLE-70) Support for multiple source folders

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHECKSTYLE-70?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_122196 ] 

Dennis Lundberg commented on MCHECKSTYLE-70:
--------------------------------------------

Michael,

With the solution of MCHECKSTYLE-40, your problem should be solved. Please try the latest 2.2-SNAPSHOT version.

> Support for multiple source folders
> -----------------------------------
>
>                 Key: MCHECKSTYLE-70
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-70
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Improvement
>    Affects Versions: 2.1
>            Reporter: Jan Palmquist
>            Priority: Minor
>
> It would be great if this plugin would support multiple source folders added by http://mojo.codehaus.org/build-helper-maven-plugin/ (or similar), and by default inspect sources from these folders instead of just ${project.build.sourceDirectory}. Correspondingly with respect to test sources if those are configured to be included.
> There are other plugins available solving this problem (somehow), eg:
> * http://mojo.codehaus.org/jdepend-maven-plugin/
> * http://mojo.codehaus.org/findbugs-maven-plugin/
> Maybe they can give some inspiration for how to make this possible?

-- 
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

        

[jira] Commented: (MCHECKSTYLE-70) Support for multiple source folders

Posted by "Michael Tamm (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHECKSTYLE-70?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_91220 ] 

Michael Tamm commented on MCHECKSTYLE-70:
-----------------------------------------

src/main/resources should also be included a a source directory by default, and the default for the includes configuration should be "**/*.java,**/*.properties" - otherwise the <module name="Translation" /> (see http://checkstyle.sourceforge.net/config_misc.html#Translation) does not work.

> Support for multiple source folders
> -----------------------------------
>
>                 Key: MCHECKSTYLE-70
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-70
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Improvement
>    Affects Versions: 2.1
>            Reporter: Jan Palmquist
>            Priority: Minor
>
> It would be great if this plugin would support multiple source folders added by http://mojo.codehaus.org/build-helper-maven-plugin/ (or similar), and by default inspect sources from these folders instead of just ${project.build.sourceDirectory}. Correspondingly with respect to test sources if those are configured to be included.
> There are other plugins available solving this problem (somehow), eg:
> * http://mojo.codehaus.org/jdepend-maven-plugin/
> * http://mojo.codehaus.org/findbugs-maven-plugin/
> Maybe they can give some inspiration for how to make this possible?

-- 
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

        

[jira] Commented: (MCHECKSTYLE-70) Support for multiple source folders

Posted by "Jan Palmquist (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHECKSTYLE-70?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_87797 ] 

Jan Palmquist commented on MCHECKSTYLE-70:
------------------------------------------

Many other plugins have the same limitation, and after looking at the pmd-plugin, I found a way to solve my issue temporarily simply letting <sourceDirectory> point to the parent project src/main instead of pointing to src/main/java.
This, however, breaks the xref-links since the src/main is not the actual source folder.
In my case, I have an additional source folder src/main/java-generated holding generated source code which I wan't to be separated from the hand written source in order to delete it whenever I run mvn clean...

> Support for multiple source folders
> -----------------------------------
>
>                 Key: MCHECKSTYLE-70
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-70
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Improvement
>    Affects Versions: 2.1
>            Reporter: Jan Palmquist
>            Priority: Minor
>
> It would be great if this plugin would support multiple source folders added by http://mojo.codehaus.org/build-helper-maven-plugin/ (or similar), and by default inspect sources from these folders instead of just ${project.build.sourceDirectory}. Correspondingly with respect to test sources if those are configured to be included.
> There are other plugins available solving this problem (somehow), eg:
> * http://mojo.codehaus.org/jdepend-maven-plugin/
> * http://mojo.codehaus.org/findbugs-maven-plugin/
> Maybe they can give some inspiration for how to make this possible?

-- 
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

        

[jira] Commented: (MCHECKSTYLE-70) Support for multiple source folders

Posted by "Anthony Whitford (JIRA)" <ji...@codehaus.org>.
    [ https://jira.codehaus.org/browse/MCHECKSTYLE-70?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=277250#comment-277250 ] 

Anthony Whitford commented on MCHECKSTYLE-70:
---------------------------------------------

No, I don't see this solved...  I think the Checkstyle plugin should ask the MavenProject for its compileSourceRoots.  See:

http://maven.apache.org/ref/3.0.3/maven-core/apidocs/org/apache/maven/project/MavenProject.html#getCompileSourceRoots()

> Support for multiple source folders
> -----------------------------------
>
>                 Key: MCHECKSTYLE-70
>                 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-70
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Improvement
>    Affects Versions: 2.1
>            Reporter: Jan Palmquist
>            Priority: Minor
>
> It would be great if this plugin would support multiple source folders added by http://mojo.codehaus.org/build-helper-maven-plugin/ (or similar), and by default inspect sources from these folders instead of just ${project.build.sourceDirectory}. Correspondingly with respect to test sources if those are configured to be included.
> There are other plugins available solving this problem (somehow), eg:
> * http://mojo.codehaus.org/jdepend-maven-plugin/
> * http://mojo.codehaus.org/findbugs-maven-plugin/
> Maybe they can give some inspiration for how to make this possible?

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira