You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2023/03/14 15:20:00 UTC

[jira] [Commented] (MNG-7131) maven.config doesn't handle arguments with spaces in them

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

ASF GitHub Bot commented on MNG-7131:
-------------------------------------

gnodet merged PR #819:
URL: https://github.com/apache/maven-mvnd/pull/819




> maven.config doesn't handle arguments with spaces in them
> ---------------------------------------------------------
>
>                 Key: MNG-7131
>                 URL: https://issues.apache.org/jira/browse/MNG-7131
>             Project: Maven
>          Issue Type: Bug
>          Components: Command Line
>    Affects Versions: 3.6.3
>            Reporter: Stefan Oehme
>            Assignee: Michael Osipov
>            Priority: Minor
>             Fix For: 3.9.0, 4.0.0-alpha-2, 4.0.0
>
>
> I tried putting a path with a space in the maven.config like so:
> {code:java}
> "-Dmy.path=/some/path with/a/space"
> {code}
> When asking for the my.path property inside Maven, I get the path with a trailing quote
> {code:java}
> /some/path with/a/space"
> {code}
> The file is parsed by [splitting it by whitespace|https://github.com/apache/maven/blob/c3cf29438e3d65d6ee5c5726f8611af99d9a649a/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java#L386]. I suggest replacing that code with the much better [CommandLineUtils.translateCommandline(argLine)|https://github.com/codehaus-plexus/plexus-utils/blob/85a4a3c534752919dd97d757d4ef0fdb6d40eb8c/src/main/java/org/codehaus/plexus/util/cli/CommandLineUtils.java#L412]



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