[Bug 59276] Update Checkstyle to version 6.17 needs a configuration change

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59276

Konstantin Kolinko <kn...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All

--- Comment #1 from Konstantin Kolinko <kn...@gmail.com> ---
(In reply to Konstantin Kolinko from comment #0)
> 
> A temporary backwards-compatible workaround may be to keep the property in
> both places (as setting the duplicate property on TreeWalker does not cause
> a runtime error with 6.17), but it is expected that the setter method will
> be removed from TreeWalker in some future release [4]
> 
> [4] https://github.com/checkstyle/checkstyle/issues/2883
> #2883 - Remove `cache` field from TreeWalker in Checkstyle 8.0

Such workaround is not possible. Checkstyle 6.15 does not have setCacheFile()
method on Walker. An attempt to run updated configuration with 6.15 results in

BUILD FAILED
\build.xml:586: Unable to create a Checker: configLocation
 {<...>\trunk\res\checkstyle\javax-checkstyle.xml}, classpath {null}.

without any further details provided by the error message.

I changed the configuration in Tomcat 9 in r1737903.

Tomcat 8, 8.5 are TODO.

Tomcat 7 is using Checkstyle 6.1.1 as running 6.2 requires Java 7.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org