maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Arnaud HERITIER" <aherit...@gmail.com>
Subject Re: [vote] [m1] release xdoc-plugin-1.10
Date Mon, 12 Jun 2006 22:22:56 GMT
I'm not what were the cases when the empty function failed.
I remember that it didn't work on empty lists.
It seems that there was also a problem with if the parameter is null.
I don't know if we can have the same error somewhere else.

Arnaud

On 6/13/06, Lukas Theussl <ltheussl@apache.org> wrote:
>
> Right on,
>
> I replaced
>
> <j:if test="${!empty(context.getVariable('maven.xdoc.distributionUrl'))}">
>
> by
>
> <j:if test="${context.getVariable('maven.xdoc.distributionUrl') != null
> and context.getVariable('maven.xdoc.distributionUrl') != ''}">
>
> and now it works also with m1.0.2. The point is: the empty function is
> used all over the place in the xdoc plugin (plugin.jelly, site.jsl,
> various templates), do I have to replace it everywhere?
>
> -Lukas
>
>
> Arnaud HERITIER wrote:
> > In the Jelly release bundled in maven 1.0.x there was some bugs in the
> > empty() function.
> >
> > cheers
> >
> > Arnaud
> >
> > On 6/12/06, Lukas Theussl <ltheussl@apache.org> wrote:
> >
> >>
> >>
> >>
> >> > I just did a "maven clean site" and I still get the "Downloads" link
> >> > when building it using the 1.10-SNAPSHOT version.
> >> >
> >>
> >> Ok, I can reproduce it now using maven-1.0.2, I don't see it with
> >> m1.1-beta-3. Certainly a jelly issue, I'll look into it.
> >>
> >> -Lukas
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> >> For additional commands, e-mail: dev-help@maven.apache.org
> >>
> >>
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message