Message: A new issue has been created in JIRA.
--------------------------------------------------------------------- View the issue: http://jira.codehaus.org/secure/ViewIssue.jspa?key=MPCHECKSTYLE-17 Here is an overview of the issue: --------------------------------------------------------------------- Key: MPCHECKSTYLE-17 Summary: Exception handling always reports missing license file Type: Bug Status: Unassigned Priority: Major Original Estimate: Unknown Time Spent: Unknown Remaining: Unknown Project: maven-checkstyle-plugin Versions: 2.2 Assignee: Reporter: Dale King Created: Thu, 26 Feb 2004 1:58 PM Updated: Thu, 26 Feb 2004 1:58 PM Environment: Don't think it matters, but WIN2K Description: I kept trying to figure out why I couldn't get Checkstyle to generate a report. I see it start to run and then all it tells me is the warning that I don't have a license file. That is not true as I do have an empty license file. The problem was that I had an error in my Checkstyle XML file. I tried to comment out some checks and accidentally was nesting XML comments, which is illegal. So checkstyle was generating an exception and the plugin caught that there was an exception. It's logic tries to check if the exception was due to the lack of a header file. It has the following line: <j:when test="${exception.indexOf('unable to load header file') != '-1'}"> It appears that this test is always evaluating to true. I modified the jelly script to print the content of the exception and this is what the exception was: org.apache.commons.jelly.JellyTagException: file:/C:/Documents and Settings/kingd/.maven/plugins/maven-checkstyle-plugin-2.2/:124:63: <ant:checkstyle> Unable to create a Checker: unable to parse C:\Documents and Settings\kingd\My Documents\Eclipse Workspace\NobleWare Utilities\checkstyle.xml - The string "--" is not permitted within comments.:68:13 I don't know enough about JEXL to diagnose what was wrong. I suspect that the problem is that -1 is in quotes so is taken to be the string "-1" not the number -1. I tried removing the quotes but got a parse error from JEXL. Does JEXL perhaps not support unary minus? I was able to get it to work by changing the check to: <j:when test="${exception.indexOf('unable to load header file') >= 0}"> --------------------------------------------------------------------- JIRA INFORMATION: 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 If you want more information on JIRA, or have a bug to report see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
