[ http://jira.codehaus.org/browse/MCHECKSTYLE-25?page=comments#action_54561 
] 

fabrizio giustina commented on MCHECKSTYLE-25:
----------------------------------------------

I am committing changes to trunk, AFAIK there are no branches for plugins 
actually.
The current version of CheckstyleReport.java has this code:

    public boolean canGenerateReport()
    {
        // TODO: would be good to scan the files here
        return super.canGenerateReport() && sourceDirectory.exists();
    }

which prevents the plugin from running when the source file is missing, without 
any error, but only if the plugin is run as a report.
I guess you are running checkstyle:check out of the site generation if you 
still get errors... I just checked in a fix wich prevents errors to be thrown 
also when the plugin is executed directly

> Do not execute checkstyle when the project has no sources
> ---------------------------------------------------------
>
>          Key: MCHECKSTYLE-25
>          URL: http://jira.codehaus.org/browse/MCHECKSTYLE-25
>      Project: Maven 2.x Checkstyle Plugin
>         Type: Improvement

>     Versions: 2.0-beta-2
>     Reporter: Vincent Massol
>     Assignee: Brett Porter

>
>
> It's nice to be able to share the checkstyle config in a parent pom. However 
> doing so leads to an error:
> C:\dev\cargo\trunk>mvn clean install
> [...]
> [INFO] 
> ----------------------------------------------------------------------------
> [INFO] Building Cargo Core
> [INFO]    task-segment: [clean, install]
> [INFO] 
> ----------------------------------------------------------------------------
> [...]
> [INFO] Preparing checkstyle:check
> [...]
> [INFO] [checkstyle:checkstyle]
> [INFO] 
> ----------------------------------------------------------------------------
> [ERROR] BUILD ERROR
> [INFO] 
> ----------------------------------------------------------------------------
> [INFO] An error has occurred in Checkstyle report generation.
> Embedded error: No source directory to process for checkstyle
> [INFO] 
> ----------------------------------------------------------------------------
> [INFO] For more information, run Maven with the -e switch
> [INFO] 
> ----------------------------------------------------------------------------
> [INFO] Total time: 19 seconds
> [INFO] Finished at: Fri Dec 30 19:40:19 CET 2005
> [INFO] Final Memory: 8M/15M
> [INFO] 
> ----------------------------------------------------------------------------

-- 
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to