Maven 2.x Checkstyle Plugin

If 'src/main/java' does not exist in a project then checkstyle skips the other source folders of the project

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.5
  • Fix Version/s: 2.10
  • Labels:
    None
  • Number of attachments :
    0

Description

I'm not sure whether this is related to MCHECKSTYLE-141.

If I check a test-project that contains only a folder src/test/java but not a folder src/main/java (even thoud the includeTestDir option is set) then checkstyle ignores the whole project:

[INFO] — maven-checkstyle-plugin:2.5:checkstyle (default-cli) @ de.faktorlogik.core.tests —
[INFO] Source directory does not exist - skipping report.
[INFO] ------------------------------------------------------------------------

Issue Links

supercedes

Bug - A problem which impairs or prevents the functions of the product. MCHECKSTYLE-153 Checkstyle doesn't run on projects containing only test classes

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ulli Hafner added a comment -

Version: 2.5

Show
Ulli Hafner added a comment - Version: 2.5
Hide
Permalink
Dennis Lundberg added a comment -

I have just applied 2 patches that could help solve this issue.
Can you please try 2.7-SNAPSHOT and see if i works for you?

Show
Dennis Lundberg added a comment - I have just applied 2 patches that could help solve this issue. Can you please try 2.7-SNAPSHOT and see if i works for you?
Hide
Permalink
Lukas Fryc added a comment -

Hi Dennis, I have ran into same issue and tried to use 2.7-SNAPSHOT,
but my build has failed with "Failed during checkstyle configuration: missing key 'cacheFile' in TreeWalker -> [Help 1]
"

Show
Lukas Fryc added a comment - Hi Dennis, I have ran into same issue and tried to use 2.7-SNAPSHOT, but my build has failed with "Failed during checkstyle configuration: missing key 'cacheFile' in TreeWalker -> [Help 1] "
Hide
Permalink
Dennis Lundberg added a comment -

Lukas,

It sounds as if your Checkstyle configuration file is incorrect.
Have you updated it to the changes that were made in Checkstyle 5.0?
See http://checkstyle.sourceforge.net/releasenotes.html for details.

Show
Dennis Lundberg added a comment - Lukas, It sounds as if your Checkstyle configuration file is incorrect. Have you updated it to the changes that were made in Checkstyle 5.0? See http://checkstyle.sourceforge.net/releasenotes.html for details.
Hide
Permalink
Sei Syvalta added a comment -

> "Failed during checkstyle configuration: missing key 'cacheFile' in TreeWalker

See http://jira.codehaus.org/browse/MCHECKSTYLE-159

Show
Sei Syvalta added a comment - > "Failed during checkstyle configuration: missing key 'cacheFile' in TreeWalker See http://jira.codehaus.org/browse/MCHECKSTYLE-159
Hide
Permalink
Dennis Lundberg added a comment -

Is this still an issue in version 2.8?

Show
Dennis Lundberg added a comment - Is this still an issue in version 2.8?
Hide
Permalink
Mark Hobson added a comment -

Yep, I've just encountered this in 2.9.1.

Show
Mark Hobson added a comment - Yep, I've just encountered this in 2.9.1.
Hide
Permalink
Peter Janes added a comment -

MCHECKSTYLE-185 (which I guess is a duplicate) also has a patch for this issue.

Show
Peter Janes added a comment - MCHECKSTYLE-185 (which I guess is a duplicate) also has a patch for this issue.
Hide
Permalink
Dennis Lundberg added a comment -

Fixed in r1438948.

Show
Dennis Lundberg added a comment - Fixed in r1438948 .

People

  • Assignee:
    Dennis Lundberg
    Reporter:
    Ulli Hafner
Vote (2)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: