• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • core
    • None
    • jenkins 1.537 (current master)

      Jenkins should support following variable expansions:

      JENKINS-19488 (3eea39648943583a530f47f69f93737266537d9d) and JENKINS-19926 (b95f2694529d22523701e8fe20f4f20b2f4d9fc4) did that in a too ad hoc way, and I'll re-implement it in more general way.

          [JENKINS-20280] Support nested variable expansion

          ikedam created issue -
          ikedam made changes -
          Link New: This issue is related to JENKINS-19488 [ JENKINS-19488 ]
          ikedam made changes -
          Link New: This issue is related to JENKINS-19926 [ JENKINS-19926 ]
          ikedam made changes -
          Link New: This issue is related to JENKINS-16255 [ JENKINS-16255 ]

          ikedam added a comment -

          ikedam added a comment - Sent a pull request: https://github.com/jenkinsci/jenkins/pull/986
          ikedam made changes -
          Link New: This issue is blocking JENKINS-16255 [ JENKINS-16255 ]

          Code changed in jenkins
          User: ikedam
          Path:
          core/src/main/java/hudson/EnvVars.java
          core/src/main/java/hudson/Launcher.java
          core/src/test/java/hudson/EnvVarsTest.java
          http://jenkins-ci.org/commit/jenkins/4204d22496a77c2ccc0bb9f959592a01e27fb54d
          Log:
          [FIX JENKINS-20280] Calculates the order to override variables. This is an improved fix for JENKINS-19488 and JENKINS-19926.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: ikedam Path: core/src/main/java/hudson/EnvVars.java core/src/main/java/hudson/Launcher.java core/src/test/java/hudson/EnvVarsTest.java http://jenkins-ci.org/commit/jenkins/4204d22496a77c2ccc0bb9f959592a01e27fb54d Log: [FIX JENKINS-20280] Calculates the order to override variables. This is an improved fix for JENKINS-19488 and JENKINS-19926 .

          Code changed in jenkins
          User: Kohsuke Kawaguchi
          Path:
          changelog.html
          core/src/main/java/hudson/EnvVars.java
          core/src/main/java/hudson/Launcher.java
          core/src/test/java/hudson/EnvVarsTest.java
          http://jenkins-ci.org/commit/jenkins/e68ec055fda2c9fcba7d7cf4347852bcdf3e1fa6
          Log:
          [FIXED JENKINS-20280]

          Merged pull request #986.

          Compare: https://github.com/jenkinsci/jenkins/compare/d954f22b4735...e68ec055fda2

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kohsuke Kawaguchi Path: changelog.html core/src/main/java/hudson/EnvVars.java core/src/main/java/hudson/Launcher.java core/src/test/java/hudson/EnvVarsTest.java http://jenkins-ci.org/commit/jenkins/e68ec055fda2c9fcba7d7cf4347852bcdf3e1fa6 Log: [FIXED JENKINS-20280] Merged pull request #986. Compare: https://github.com/jenkinsci/jenkins/compare/d954f22b4735...e68ec055fda2
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          dogfood added a comment -

          Integrated in jenkins_main_trunk #2984
          [FIX JENKINS-20280] Calculates the order to override variables. This is an improved fix for JENKINS-19488 and JENKINS-19926. (Revision 4204d22496a77c2ccc0bb9f959592a01e27fb54d)

          Result = SUCCESS
          kohsuke : 4204d22496a77c2ccc0bb9f959592a01e27fb54d
          Files :

          • core/src/main/java/hudson/EnvVars.java
          • core/src/test/java/hudson/EnvVarsTest.java
          • core/src/main/java/hudson/Launcher.java

          dogfood added a comment - Integrated in jenkins_main_trunk #2984 [FIX JENKINS-20280] Calculates the order to override variables. This is an improved fix for JENKINS-19488 and JENKINS-19926 . (Revision 4204d22496a77c2ccc0bb9f959592a01e27fb54d) Result = SUCCESS kohsuke : 4204d22496a77c2ccc0bb9f959592a01e27fb54d Files : core/src/main/java/hudson/EnvVars.java core/src/test/java/hudson/EnvVarsTest.java core/src/main/java/hudson/Launcher.java

            ikedam ikedam
            ikedam ikedam
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: