[jira] Commented: (VELOCITY-636) Remove (mandatory) dependency on ANTLR from POM

["Nathan Bubna (JIRA)" <de...@velocity.apache.org>]

    [ https://issues.apache.org/jira/browse/VELOCITY-636?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12643144#action_12643144 ] 

Nathan Bubna commented on VELOCITY-636:
---------------------------------------

any thoughts on this, Jarkko?  (if you're listening...)

> Remove (mandatory) dependency on ANTLR from POM
> -----------------------------------------------
>
>                 Key: VELOCITY-636
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-636
>             Project: Velocity
>          Issue Type: Bug
>          Components: Build, Engine
>    Affects Versions: 1.6-beta1
>            Reporter: Benjamin Bentmann
>
> A dependency on ANTLR 2.7.7 (BTW, the build.properties mentions version 2.7.5 in contrast) has been introduced in [r685386|http://svn.eu.apache.org/viewvc?view=rev&revision=685386]. The commit message ("add findbugs support to maven build") as well as the actual compilation indicates that this dependency is not an actual compile-time dependency of the engine. It's not clear to me why it was added in the first place, running "mvn site" succeeds without this dependency and the resulting FindBugs reports looks OK (ignoring the various issues it lists). For the sake of a proper class path in client applications of Velocity, this dependency should either be completely removed or at least marked optional (c.f. http://velocity.apache.org/engine/devel/jar-dependencies.html).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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