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

Define SCMRevisionAction from SCMBinder

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      SCMBinder currently uses some hacks to save an SCM object in the build. Perhaps it would be better to use SCMRevisionAction, which was intended for this purpose.

        Attachments

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            Possible this would be helpful in resolving JENKINS-30222.

            Show
            jglick Jesse Glick added a comment - Possible this would be helpful in resolving JENKINS-30222 .
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            cps/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsScript.java
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMVar.java
            multibranch/src/main/resources/org/jenkinsci/plugins/workflow/multibranch/SCMVar/help.jelly
            multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java
            http://jenkins-ci.org/commit/workflow-plugin/4e9bd117797620e2f11c03f4053e8210d3086eac
            Log:
            [FIXED JENKINS-30531] Set SCMRevisionAction when we do a build, so client code can get access to this record.
            Has the happy side effect of fixing JENKINS-30222: scm as a global variable can be used unqualified from library code.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsScript.java multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMVar.java multibranch/src/main/resources/org/jenkinsci/plugins/workflow/multibranch/SCMVar/help.jelly multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java http://jenkins-ci.org/commit/workflow-plugin/4e9bd117797620e2f11c03f4053e8210d3086eac Log: [FIXED JENKINS-30531] Set SCMRevisionAction when we do a build, so client code can get access to this record. Has the happy side effect of fixing JENKINS-30222 : scm as a global variable can be used unqualified from library code.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            cps/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsScript.java
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMVar.java
            multibranch/src/main/resources/org/jenkinsci/plugins/workflow/multibranch/SCMVar/help.jelly
            multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java
            http://jenkins-ci.org/commit/workflow-plugin/46d63875dbaa218e9f23e9520cf5f1ed8f6a1c43
            Log:
            Merge pull request #239 from jglick/SCMRevisionAction-JENKINS-30531

            JENKINS-30531 Set SCMRevisionAction when we do a build

            Compare: https://github.com/jenkinsci/workflow-plugin/compare/59e6cf394ba7...46d63875dbaa

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsScript.java multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMVar.java multibranch/src/main/resources/org/jenkinsci/plugins/workflow/multibranch/SCMVar/help.jelly multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java http://jenkins-ci.org/commit/workflow-plugin/46d63875dbaa218e9f23e9520cf5f1ed8f6a1c43 Log: Merge pull request #239 from jglick/SCMRevisionAction- JENKINS-30531 JENKINS-30531 Set SCMRevisionAction when we do a build Compare: https://github.com/jenkinsci/workflow-plugin/compare/59e6cf394ba7...46d63875dbaa
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            cps/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsScript.java
            http://jenkins-ci.org/commit/workflow-cps-plugin/587f35da06082e0ee5c8f925d0877a63c0e123b0
            Log:
            [FIXED JENKINS-30531] Set SCMRevisionAction when we do a build, so client code can get access to this record.
            Has the happy side effect of fixing JENKINS-30222: scm as a global variable can be used unqualified from library code.
            Originally-Committed-As: 4e9bd117797620e2f11c03f4053e8210d3086eac

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsScript.java http://jenkins-ci.org/commit/workflow-cps-plugin/587f35da06082e0ee5c8f925d0877a63c0e123b0 Log: [FIXED JENKINS-30531] Set SCMRevisionAction when we do a build, so client code can get access to this record. Has the happy side effect of fixing JENKINS-30222 : scm as a global variable can be used unqualified from library code. Originally-Committed-As: 4e9bd117797620e2f11c03f4053e8210d3086eac
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMVar.java
            multibranch/src/main/resources/org/jenkinsci/plugins/workflow/multibranch/SCMVar/help.jelly
            multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java
            http://jenkins-ci.org/commit/workflow-multibranch-plugin/d8d9403029143715efee336d7094d46cef758d6f
            Log:
            [FIXED JENKINS-30531] Set SCMRevisionAction when we do a build, so client code can get access to this record.
            Has the happy side effect of fixing JENKINS-30222: scm as a global variable can be used unqualified from library code.
            Originally-Committed-As: 4e9bd117797620e2f11c03f4053e8210d3086eac

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMVar.java multibranch/src/main/resources/org/jenkinsci/plugins/workflow/multibranch/SCMVar/help.jelly multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java http://jenkins-ci.org/commit/workflow-multibranch-plugin/d8d9403029143715efee336d7094d46cef758d6f Log: [FIXED JENKINS-30531] Set SCMRevisionAction when we do a build, so client code can get access to this record. Has the happy side effect of fixing JENKINS-30222 : scm as a global variable can be used unqualified from library code. Originally-Committed-As: 4e9bd117797620e2f11c03f4053e8210d3086eac

              People

              Assignee:
              jglick Jesse Glick
              Reporter:
              jglick Jesse Glick
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: