Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-25002

Using : as separator for tool location is not the best choice on Windows

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • swarm-plugin
    • None

      Using : as separator for tool location is not the best choice on Windows, as for example maven:c:\blahblah won't be parsed correctly.

          [JENKINS-25002] Using : as separator for tool location is not the best choice on Windows

          Nico Mommaerts created issue -

          Nico Mommaerts added a comment - - edited

          I tried to resolve this by using '=' as separator but then I bump into this issue with args4j: https://java.net/jira/browse/ARGS4J-24
          Gonna think about this tonight, any suggestions for a better, parseable format are always welcome.

          Nico Mommaerts added a comment - - edited I tried to resolve this by using '=' as separator but then I bump into this issue with args4j: https://java.net/jira/browse/ARGS4J-24 Gonna think about this tonight, any suggestions for a better, parseable format are always welcome.

          Daniel Beck added a comment -

          Only split first occurrence (http://docs.oracle.com/javase/1.5.0/docs/api/java/lang/String.html#split%28java.lang.String,%20int%29 )

          Does not support tool names with colon though, the other issue already discusses alternative formats.

          Daniel Beck added a comment - Only split first occurrence ( http://docs.oracle.com/javase/1.5.0/docs/api/java/lang/String.html#split%28java.lang.String,%20int%29 ) Does not support tool names with colon though, the other issue already discusses alternative formats.

          Nico Mommaerts added a comment -

          Agreed, that would be the quickest fix without breaking existing stuff (as far as someone besides myself is already using it).
          I'll fix it later today

          Nico Mommaerts added a comment - Agreed, that would be the quickest fix without breaking existing stuff (as far as someone besides myself is already using it). I'll fix it later today

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: nico.mommaerts
          Path:
          plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java
          http://jenkins-ci.org/commit/swarm-plugin/4081439807a289bfe74bc6d8e4c99d8374c407e4
          Log:
          JENKINS-25002 Limit the split function on 2 so that paths with : can be used

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: nico.mommaerts Path: plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java http://jenkins-ci.org/commit/swarm-plugin/4081439807a289bfe74bc6d8e4c99d8374c407e4 Log: JENKINS-25002 Limit the split function on 2 so that paths with : can be used

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Nico Mommaerts
          Path:
          plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java
          http://jenkins-ci.org/commit/swarm-plugin/c0e0fe33227af2c709563fb45d441df28fb13dbe
          Log:
          JENKINS-25002 Added a java comment explaining the use of the limit on the split function

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nico Mommaerts Path: plugin/src/main/java/hudson/plugins/swarm/PluginImpl.java http://jenkins-ci.org/commit/swarm-plugin/c0e0fe33227af2c709563fb45d441df28fb13dbe Log: JENKINS-25002 Added a java comment explaining the use of the limit on the split function
          Peter Jönsson made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Peter Jönsson made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          Peter Jönsson made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 158900 ] New: JNJira + In-Review [ 208104 ]

            mr_dfuse Nico Mommaerts
            mr_dfuse Nico Mommaerts
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: