[ http://jira.codehaus.org/browse/MSITE-170?page=comments#action_79854 ] 
            
Dennis Lundberg commented on MSITE-170:
---------------------------------------

Using the most recent nightly snapshot of Jakarta Velocity I did this:
- Install the snapshot of Velocity 1.5 into my local repo
- Update pom.xml for plexus-components/plexus-velocity to use velocity-1.5-dev, 
update commons-logging to 3.1 and add a dependency on commons-lang-2.1
- Install plexus-velocity 1.1.4-SNAPSHOT into local repo
- Update pom.xml of doxia-siterenderer to use plexus-velocity 1.1.4-SNAPSHOT
- Install doxia-siterenderer-1.0-alpha-9-SNAPSHOT into local repo
- Install maven-site-plugin-2.0-SNAPSHOT built from source into local repo
- Run 'mvn site' on any project.

The errors about "too many arguments to macro" no longer appears. There are 
however quite a lot of warnings, that has not been there before :(

> [ERROR] VM #displayTree: error : too few arguments to macro
> -----------------------------------------------------------
>
>                 Key: MSITE-170
>                 URL: http://jira.codehaus.org/browse/MSITE-170
>             Project: Maven 2.x Site Plugin
>          Issue Type: Bug
>    Affects Versions: 2.0
>            Reporter: fabrizio giustina
>
> when running mvn site:site a couple of "too few arguments to macro" always 
> pop up. This is extremely bad in terms of user experience, and we should find 
> a way to remove these logs:
> [ERROR] VM #displayTree: error : too few arguments to macro. Wanted 2 got 0
> [ERROR] VM #menuItem: error : too few arguments to macro. Wanted 1 got 0
> These errors are due to a known velocity bug related to the use of recursive 
> macros:
> http://issues.apache.org/bugzilla/show_bug.cgi?id=13623
> http://mail-archives.apache.org/mod_mbox/jakarta-velocity-user/200402.mbox/[EMAIL
>  PROTECTED]
> Recursive macros are defined in 
> org/apache/maven/doxia/siterenderer/resources/default-site.vm in the doxia 
> site-renderer component.
> Logging is handled in the plexus velocity component.
> This velocity bug is still open in velocity and no patch will be available 
> anytime soon. In the meanwhile we should try to handle this situation in some 
> way by filtering out messages or removing the use of recursive macros (very 
> hard, they are used to print out the site tree)... or switching to a better 
> templating engine like freemarker.
> This issue could probably be addressed in the plexus velocity component or in 
> the doxia site renderer (given that waiting for a bugfixed velocity release 
> is not an option). I'm anyway assigning it to the site plugin since it's 
> where users see these logs coming from and where probably users could open 
> similar issues.
> Any suggestion on how to dirty-patch this?

-- 
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
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to