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

Shell Task on Windows Slave Uses Incorrect /bin/sh

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • None

      At https://github.com/hudson/hudson/blob/master/core/src/main/java/hudson/tasks/Shell.java#L139 the Shell task attempts to detect Windows, and if so, use "sh" instead of "/bin/sh". The problem is that this detection takes place on the Master, instead of on the Slave that is actually going to run the task. If this were fixed to detect correctly, lots of people's workarounds of globally setting the shell as plain "sh" could go away.

          [JENKINS-8449] Shell Task on Windows Slave Uses Incorrect /bin/sh

          Code changed in jenkins
          User: Seiji Sogabe
          Path:
          changelog.html
          core/src/main/java/hudson/tasks/Shell.java
          http://jenkins-ci.org/commit/jenkins/415a2729a54bc65c696e06d804bffd658d16b5ea
          Log:
          [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Seiji Sogabe Path: changelog.html core/src/main/java/hudson/tasks/Shell.java http://jenkins-ci.org/commit/jenkins/415a2729a54bc65c696e06d804bffd658d16b5ea Log: [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          dogfood added a comment -

          dogfood added a comment - Integrated in jenkins_main_trunk #620

          dogfood added a comment -

          Integrated in jenkins_main_trunk #734
          [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          Kohsuke Kawaguchi : 6d8fd5450a823f0c7abe528bce6a36fd81d2ad5f
          Files :

          • core/src/main/java/hudson/tasks/Shell.java

          dogfood added a comment - Integrated in jenkins_main_trunk #734 [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave. Kohsuke Kawaguchi : 6d8fd5450a823f0c7abe528bce6a36fd81d2ad5f Files : core/src/main/java/hudson/tasks/Shell.java

          Code changed in jenkins
          User: Seiji Sogabe
          Path:
          changelog.html
          core/src/main/java/hudson/tasks/Shell.java
          http://jenkins-ci.org/commit/jenkins/415a2729a54bc65c696e06d804bffd658d16b5ea
          Log:
          [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Seiji Sogabe Path: changelog.html core/src/main/java/hudson/tasks/Shell.java http://jenkins-ci.org/commit/jenkins/415a2729a54bc65c696e06d804bffd658d16b5ea Log: [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          Code changed in jenkins
          User: Seiji Sogabe
          Path:
          core/src/main/java/hudson/tasks/Shell.java
          http://jenkins-ci.org/commit/jenkins/6d8fd5450a823f0c7abe528bce6a36fd81d2ad5f
          Log:
          [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Seiji Sogabe Path: core/src/main/java/hudson/tasks/Shell.java http://jenkins-ci.org/commit/jenkins/6d8fd5450a823f0c7abe528bce6a36fd81d2ad5f Log: [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          Code changed in jenkins
          User: Seiji Sogabe
          Path:
          core/src/main/java/hudson/tasks/Shell.java
          http://jenkins-ci.org/commit/jenkins/6d8fd5450a823f0c7abe528bce6a36fd81d2ad5f
          Log:
          [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Seiji Sogabe Path: core/src/main/java/hudson/tasks/Shell.java http://jenkins-ci.org/commit/jenkins/6d8fd5450a823f0c7abe528bce6a36fd81d2ad5f Log: [FIXED JENKINS-8449] Use correct shell interpreter on Windows slave.

            Unassigned Unassigned
            ccutrer Cody Cutrer
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: