[MCHECKSTYLE-225] headerLocation no longer sets checkstyle.header.file

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.12
• Fix Version/s: 2.12.1, 2.13
• Environment: JDK 7u51 on OS X with Maven 3.1.0
• Assignee: Robert Scholte
• Reporter: Diwaker Gupta
• Created: 17/Mar/14 11:34
• Updated: 12/Mar/15 20:01
• Resolved: 24/Apr/14 14:33

Description

We use a multi-module configuration as described at https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html

This morning I tried upgraded to checkstyle-plugin 2.12 and our builds started failing with:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.12:check (default-cli) on project common: Failed during checkstyle execution: Failed during checkstyle configuration: unable to read /path/to/common/target/checkstyle-checker.xml - unable to parse configuration stream - Property ${checkstyle.header.file} has not been set -> [Help 1]

Our config hasn't changed in almost two years. We are currently using v2.11 so this seems like a regression in 2.12

Issue Links

• is related to: MCHECKSTYLE-219 When no license is specified, ALv2 from maven-checkstyle-plugin is used
• relates to: MCHECKSTYLE-250 NPE on tying to load LICENSE.txt resource from non-jar plugin dependencies