This is old code, not new workaround code. Just a bad patch I guess. As I
said earlier - all that should have been added was the ant:copy lines.

> -----Original Message-----
> From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] 
> Sent: Monday, 25 August 2003 11:42 AM
> To: Maven Developers List
> Subject: Re: cvs commit: maven/src/plugins-build/jxr plugin.jelly
> 
> 
> [EMAIL PROTECTED] wrote on 23/08/2003 08:56:57 AM:
> 
> > bwalding    2003/08/22 15:56:57
> > 
> >   Modified:    src/plugins-build/jxr plugin.jelly
> >   Log:
> >   Now copies stylesheet into jxr test output
> > 
> [snip]
> >   +      <!-- XXX This is a dirty hack until the semantics of pom.
> > build.sourceDirectory are cleared up -->
> >   +      <j:set var="sourceDir" 
> value="${pom.build.sourceDirectory}"/> 
> >   +      <util:file var="sourceDirFile" name="${sourceDir}"/>
> >   + 
> >   +      <j:if test="${!sourceDirFile.isAbsolute()}">
> >   +        <j:set var="sourceDir" value="${basedir}/${sourceDir}"/>
> >   +      </j:if>
> 
> I thought we had it cleared up now? Afaik, pom.build.sourceDirectory 
> ALWAYS returns an absolute path.
> 
> Brett, Ben?
> 
> I'd rather we not propagate uncertainty if it's all working 
> as designed 
> now.
> --
> dIon Gillard, Multitask Consulting
> Blog:      http://blogs.codehaus.org/people/dion/
> 
> 
> 

Reply via email to