You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@karaf.apache.org by "Christian Schneider (Updated) (JIRA)" <ji...@apache.org> on 2012/03/07 15:15:00 UTC

[jira] [Updated] (KARAF-526) add-features-to-repo: descriptor with file contains spaces on unix

     [ https://issues.apache.org/jira/browse/KARAF-526?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christian Schneider updated KARAF-526:
--------------------------------------

    Fix Version/s:     (was: 3.0.0)
    
> add-features-to-repo: descriptor with file contains spaces on unix
> ------------------------------------------------------------------
>
>                 Key: KARAF-526
>                 URL: https://issues.apache.org/jira/browse/KARAF-526
>             Project: Karaf
>          Issue Type: Bug
>          Components: karaf-feature, karaf-tooling
>    Affects Versions: 2.1.4
>         Environment: unix
>            Reporter: Alexey Markevich
>            Assignee: Jean-Baptiste Onofré
>             Fix For: 3.1.0
>
>   Original Estimate: 1m
>  Remaining Estimate: 1m
>
> Configuration:
> <groupId>org.apache.karaf.tooling</groupId>
> <artifactId>features-maven-plugin</artifactId>
>  <executions>
>   <execution>
>    <goals>
>     <goal>add-features-to-repo</goal>
>    </goals>
>    <configuration>
>     <descriptors>
>      <descriptor>file:${basedir}/target/classes/activemq-features.xml</descriptor>
> ${basedir} contains spaces at target PC. Result:
> Caused by: java.lang.IllegalArgumentException
> 	at java.net.URI.create(URI.java:842)
> 	at org.apache.karaf.tooling.features.AddFeaturesToRepoMojo.execute(AddFeaturesToRepoMojo.java:130)
> Looking into sources I found
> org.apache.karaf.tooling.features.MojoSupport.translateFromMaven()
> There is a check for windows and replacement spaces with %20.
> Simple fix:
>         } else if (uri.startsWith("file:")) {
>             if (System.getProperty("os.name").startsWith("Windows")) {
>                     String baseDir = uri.substring(5).replace('\\', '/').replaceAll(" ", "%20");
>                     String result = baseDir;
>                     if (baseDir.indexOf(":") > 0) {
>                             result = "file:///" + baseDir;
>                     }
>                     return result;
>             } else {
>                 return uri.replaceAll(" ", "%20");
>             }
>         }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira