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

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

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

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

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          mr_dfuse 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.

          Show
          mr_dfuse 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.
          Hide
          Permalink
          danielbeck 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.

          Show
          danielbeck 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.
          Hide
          Permalink
          mr_dfuse 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

          Show
          mr_dfuse 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
          Hide
          Permalink
          scm_issue_link 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

          Show
          scm_issue_link 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
          Hide
          Permalink
          scm_issue_link 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

          Show
          scm_issue_link 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

            People

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

              Dates

              Created:
              Updated:
              Resolved: