[ 
https://issues.apache.org/jira/browse/MWRAPPER-114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17882635#comment-17882635
 ] 

Edward Woolhouse commented on MWRAPPER-114:
-------------------------------------------

The above comment
{code:java}
The proposed fix does not actually fix the reported issue.
Relative paths, such as `distributionUrl=apache-maven-3.8.8-bin.zip` does not 
have a `file` schema nor use a full local path (as per the tests provided).
Or did I miss something? {code}
was withdrawn and deleted with a message along the lines of 'I've tested it, it 
does work after all'

The bot helpfully copied the first comment, but not the retraction. Is anyone 
able ot push this over the line? I'm happy to address any issues people have 
with it.

> Maven wrapper relative distributionUrl does not work on Windows
> ---------------------------------------------------------------
>
>                 Key: MWRAPPER-114
>                 URL: https://issues.apache.org/jira/browse/MWRAPPER-114
>             Project: Maven Wrapper
>          Issue Type: Bug
>          Components: Maven Wrapper Jar
>    Affects Versions: 3.2.0
>         Environment: Windows
>            Reporter: Edward Woolhouse
>            Priority: Normal
>
> If Maven Wrapper is used with a relative distributionUrl on Windows then an 
> error message indicating invalid path characters is shown.
>  
> {code:java}
> # Directory structure 
> ./mvnw
> ./mvnw.cmd
> ./.mvn/wrapper/maven-wrapper.jar
> ./.mvn/wrapper/maven-wrapper.properties
> ...{code}
> {code:java}
> # maven-wrapper.properties
> distributionUrl=apache-maven-3.8.8-bin.zip
> wrapperUrl=maven-wrapper.jar{code}
> When run
> {code:java}
> ./mvnw
> ...
> java.nio.file.InvalidPathException: Illegal char <:> at index 2: 
> /C:/some-path/apache-maven-3.8.8.zip {code}
> This is caused by PathAssembler.java:getBaseName unnecessarilly stringifying 
> the distibution URL. The fix is as follows
> {code:java}
>     private String getBaseName(URI distUrl) {
> -       return Paths.get(distUrl.getPath()).getFileName().toString();
> +       return Paths.get(distUrl).getFileName().toString();
>     }{code}
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to