Bug 53011 - Ant Jasper task fails after one error
Summary: Ant Jasper task fails after one error
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 6
Classification: Unclassified
Component: Jasper (show other bugs)
Version: 6.0.35
Hardware: All All
: P2 enhancement with 5 votes (vote)
Target Milestone: default
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-31 15:01 UTC by Nick Williams
Modified: 2017-05-10 19:58 UTC (History)
1 user (show)



Attachments
Patch to add an option "failFast" (4.41 KB, text/plain)
2014-04-24 06:56 UTC, Marc Pompl
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Williams 2012-03-31 15:01:46 UTC
The Ant task for Jasper JSP compiling (JspC) will fail (if failOnError = true) after a single error, without looking for other errors.

This can be very challenging for large projects. For example, we have 2,487 JSPs that need compiling. If some part of the Java code is changed that breaks hundreds of JSPs (which, admittedly, means a developer didn't do his job correctly, but still), the continuous integration build for our project fails (as it should), but the log only shows one of the errors.

The Ant task should have a new property, "failAfterAllErrors," that makes it log all compile failures as errors but continue compiling until every JSP compilation has been attempted.

This is similar to how the javac Ant task works. It logs ALL of the compile errors, and only after having tried to compile everything, it fails the build.

Thoughts?

I am interested in possibly contributing code for this feature. I will email the developers' list later about it.
Comment 1 Christopher Schultz 2012-04-02 21:20:00 UTC
+1 to the idea, but I'd like to see an implementation that is more in line with similar tasks like <javac> and <junit>: they use "failOnError" (defaults to 'true' to get current behavior) and "errorProperty". If you set failOnError=false and errorProperty=jasperFailed, then you can test for jasperFailed and emit any kind of warning/error you'd like from your ant target.
Comment 2 Nick Williams 2012-04-03 22:06:04 UTC
Note: While I don't disagree with your suggestion, Chris, and we can certainly discuss it further, that's not how Ant's javac works. Javac failonerror defaults to true, and when it's set to true, the behavior is to try compiling everything and wait until all compile errors have happened before failing. In fact, as far as I can tell, there is actually no way to make it behave like JspC (fail after the first error).

Relevant to 53011, 53012, 53031 and 53032: Per discussions with Chris Schultz in the users list, we agree that it would probably make more sense to separate the Ant task from the JspC class and place a new Ant task in the o.a.catalina.ant package (possibly o.a.catalina.ant.jasper?). Research about the state of the task needs to be completed. Further discussions about this will take place in the developer's list, and then these issues will be updated when a consensus is reached.
Comment 3 Christopher Schultz 2012-04-03 22:11:01 UTC
(In reply to comment #2)
> Javac failonerror defaults to true, and when it's set to true, the behavior
> is to try compiling everything and wait until all compile errors have
> happened before failing. In fact, as far as I can tell, there is actually
> no way to make it behave like JspC (fail after the first error).

Fair enough. Perhaps a third setting to make it fail-fast (to emulate current behavior).
Comment 4 Marc Pompl 2014-04-24 06:56:59 UTC
Created attachment 31555 [details]
Patch to add an option "failFast"

Appended is a patch that adds an option "failFast". Default is "false", so the build does not fail at the first compile error, but after the last JSP has been parsed using the following message:
"Failed to compile JSPS: org.apache.jasper.JasperException: Aborted due to compilation failures!"

Additionally, a tiny statistic will be printed.
"JSP-compilations: 465, Errors: 20, Time elapsed: 60,777 sec"
Comment 5 Marc Pompl 2014-04-24 06:58:15 UTC
(In reply to Marc Pompl from comment #4)
> Patch to add an option "failFast"

See corresponding issue for the maven-jspc-plugin https://github.com/Jasig/jspc-maven-plugin/issues/17
Comment 6 Mark Thomas 2017-05-10 19:58:11 UTC
I have applied a variation of this patch (mainly to account for code changes) to 9.0.x and it will be included in 9.0.0.M22 onwards.