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

Malformed PATH environment variable on Windows slave (unix syntax used)

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      PATH without the plugin:

      Path=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;...

      PATH with the plugin:

      Path=C:\Users\developer\jenkins\tools\Custom_tool\Maven_3.0.4_for_non-maven_projects_\apache-maven-3.0.4:/C:\Users\developer\jenkins\tools\Custom_tool\Maven_3.0.4_for_non-maven_projects_\apache-maven-3.0.4/bin:C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;...

        Attachments

          Issue Links

          links to

          Web Link PR 20

            Activity

            Descending order - Click to sort in ascending order
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            src/main/java/com/cloudbees/jenkins/plugins/customtools/CustomToolInstallWrapper.java
            http://jenkins-ci.org/commit/customtools-plugin/6f337351a7450d1a29139d9045805a762fae51e5
            Log:
            Merge pull request #20 from jglick/JENKINS-14515-pathsep

            [FIXED JENKINS-14515] Incorrect path separator used on Win slave from Unix master

            Compare: https://github.com/jenkinsci/customtools-plugin/compare/8b09f8bbff45...6f337351a745

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: src/main/java/com/cloudbees/jenkins/plugins/customtools/CustomToolInstallWrapper.java http://jenkins-ci.org/commit/customtools-plugin/6f337351a7450d1a29139d9045805a762fae51e5 Log: Merge pull request #20 from jglick/ JENKINS-14515 -pathsep [FIXED JENKINS-14515] Incorrect path separator used on Win slave from Unix master Compare: https://github.com/jenkinsci/customtools-plugin/compare/8b09f8bbff45...6f337351a745
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            src/main/java/com/cloudbees/jenkins/plugins/customtools/CustomToolInstallWrapper.java
            http://jenkins-ci.org/commit/customtools-plugin/6698c8f0cd931bdd823d9e8c6325daf14f7103d7
            Log:
            [FIXED JENKINS-14515] toEnvVars must respect the slave platform when creating a new EnvVars object.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/com/cloudbees/jenkins/plugins/customtools/CustomToolInstallWrapper.java http://jenkins-ci.org/commit/customtools-plugin/6698c8f0cd931bdd823d9e8c6325daf14f7103d7 Log: [FIXED JENKINS-14515] toEnvVars must respect the slave platform when creating a new EnvVars object.
            Hide
            Permalink
            oleg_nenashev Oleg Nenashev added a comment -

            The functionality has been broken in https://github.com/jenkinsci/customtools-plugin/commit/1ed68b4a65642113f4a61da08aa5f2e4da5d5b4b (0.4.1). I'll fix the regression

            Show
            oleg_nenashev Oleg Nenashev added a comment - The functionality has been broken in https://github.com/jenkinsci/customtools-plugin/commit/1ed68b4a65642113f4a61da08aa5f2e4da5d5b4b (0.4.1). I'll fix the regression
            Hide
            Permalink
            jglick Jesse Glick added a comment -

            Behavior may have been changed, but not actually fixed in 0.4.2. You now get

            Path=C:\slave\tools\something;C:\slave\tools\something\bin;:C:\WINDOWS\system32;…

            Note the : separating the path addition from the original path, causing the first entry in the original path to be unusable.

            Show
            jglick Jesse Glick added a comment - Behavior may have been changed, but not actually fixed in 0.4.2. You now get Path=C:\slave\tools\something;C:\slave\tools\something\bin;:C:\WINDOWS\system32;… Note the : separating the path addition from the original path, causing the first entry in the original path to be unusable.
            Hide
            Permalink
            oleg_nenashev Oleg Nenashev added a comment -

            Resolved issue in the version 0.3

            Show
            oleg_nenashev Oleg Nenashev added a comment - Resolved issue in the version 0.3
            5 older comments

              People

              Assignee:
              oleg_nenashev Oleg Nenashev
              Reporter:
              korotkov Konstantin Korotkov
              Votes:
              1 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: