Are you sure? To me it looks like the sole purpose of
unifyPathSeparator() is to, well, harmonize the path separator.

Here's the jira http://jira.codehaus.org/browse/MJAVADOC-326

I'm not sure where to add a test for this.

--Erik

On Tue, Sep 13, 2011 at 02:25, Vincent Siveton <[email protected]> wrote:
> I think unifyPathSeparator method should take care of this directly
>
> Vincent
>
> 2011/9/12 Erik Lindblad <[email protected]>:
>> The docletPath parameter is concatenated incorrectly (no separator
>> between last docletArtifact and docletPath).
>>
>>
>> Here's a fix:
>>
>> Index: src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java
>> ===================================================================
>> --- src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java   
>>    (revision
>> 1169905)
>> +++ src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java   
>>    (working
>> copy)
>> @@ -2728,6 +2728,8 @@
>>
>>         if ( !StringUtils.isEmpty( docletPath ) )
>>         {
>> +            if(!path.toString().endsWith(":") ||
>> !path.toString().endsWith(";"))
>> +                path.append( ":" );
>>             path.append( JavadocUtil.unifyPathSeparator( docletPath ) );
>>         }
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to