You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@buildr.apache.org by croes <gi...@git.apache.org> on 2017/10/17 08:14:35 UTC

[GitHub] buildr pull request #29: java command: escape classpath entries in the pathi...

GitHub user croes opened a pull request:

    https://github.com/apache/buildr/pull/29

    java command: escape classpath entries in the pathing jar manifest

    Hi,
    
    I noticed that the java command fails when using long classpaths on windows, while the classpath also has entries that contain spaces.
    
    The issue is that the manifest in the pathing jar uses space as a separator between classpath entries, and entries aren't URI-encoded. A classpath entry with a space in it is incorrectly treated as 2 entries.
    
    PR is for an older revision, as we use buildr 1.4 internally. It should be easy enough to merge though, as it's a one-liner fix.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/luciad/buildr luciad_jar_classpath_url_encoding

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/buildr/pull/29.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #29
    
----
commit b6520a7801dbbc4d61b78e5f03545392efddff60
Author: Glenn Croes <gl...@gmail.com>
Date:   2017-10-16T11:00:48Z

    java command: escape classpath entries in the pathing jar manifest

----


---

[GitHub] buildr issue #29: java command: escape classpath entries in the pathing jar ...

Posted by realityforge <gi...@git.apache.org>.

Github user realityforge commented on the issue:

    https://github.com/apache/buildr/pull/29
  
    Thanks - fix merged


---

[GitHub] buildr issue #29: java command: escape classpath entries in the pathing jar ...

Posted by croes <gi...@git.apache.org>.

Github user croes commented on the issue:

    https://github.com/apache/buildr/pull/29
  
    JIRA: https://issues.apache.org/jira/browse/BUILDR-733


---

[GitHub] buildr pull request #29: java command: escape classpath entries in the pathi...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/buildr/pull/29


---