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

Jobs checked out from svn with pinned externals continuously run (with no changes in svn)

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      This is a regression - was fixed here: JENKINS-6209
      revision.txt does NOT contain the ": : p" when the 1.44 version is used.

      The previous version (1.39 in our case) correctly writes the ": : p"

        Attachments

          Issue Links

            Activity

            Hide
            ndeloof Nicolas De Loof added a comment -
            Show
            ndeloof Nicolas De Loof added a comment - https://github.com/jenkinsci/subversion-plugin/blob/master/src/main/java/hudson/scm/subversion/SubversionUpdateEventHandler.java#L72 only considers peg revision, so need to be improved to also support -r defined ones
            Hide
            yoichi Yoichi Nakayama added a comment -

            I've sent a pull request for this issue:
            https://github.com/jenkinsci/subversion-plugin/pull/51

            Show
            yoichi Yoichi Nakayama added a comment - I've sent a pull request for this issue: https://github.com/jenkinsci/subversion-plugin/pull/51
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Yoichi NAKAYAMA
            Path:
            src/main/java/hudson/scm/subversion/SubversionUpdateEventHandler.java
            http://jenkins-ci.org/commit/subversion-plugin/1b3c7dae85afa30b632507585ab1b67ba60363b3
            Log:
            [FIXED JENKINS-16533] Respect external revision

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Yoichi NAKAYAMA Path: src/main/java/hudson/scm/subversion/SubversionUpdateEventHandler.java http://jenkins-ci.org/commit/subversion-plugin/1b3c7dae85afa30b632507585ab1b67ba60363b3 Log: [FIXED JENKINS-16533] Respect external revision
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Yoichi NAKAYAMA
            Path:
            src/test/java/hudson/scm/SubversionSCMTest.java
            src/test/resources/hudson/scm/JENKINS-16533.zip
            http://jenkins-ci.org/commit/subversion-plugin/d0303e57720274a87f9bc5e59719b53305a0f5e1
            Log:
            add test for JENKINS-16533

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Yoichi NAKAYAMA Path: src/test/java/hudson/scm/SubversionSCMTest.java src/test/resources/hudson/scm/ JENKINS-16533 .zip http://jenkins-ci.org/commit/subversion-plugin/d0303e57720274a87f9bc5e59719b53305a0f5e1 Log: add test for JENKINS-16533
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Christoph Kutzinski
            Path:
            src/main/java/hudson/scm/subversion/SubversionUpdateEventHandler.java
            src/test/java/hudson/scm/SubversionSCMTest.java
            src/test/resources/hudson/scm/JENKINS-16533.zip
            http://jenkins-ci.org/commit/subversion-plugin/5ea622901e1ea04bdf9bc1bfc5b448844b6595ad
            Log:
            Merge pull request #51 from yoichi/master

            [FIXED JENKINS-16533] Respect external revision

            Compare: https://github.com/jenkinsci/subversion-plugin/compare/74b0f9ec5e23...5ea622901e1e

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Christoph Kutzinski Path: src/main/java/hudson/scm/subversion/SubversionUpdateEventHandler.java src/test/java/hudson/scm/SubversionSCMTest.java src/test/resources/hudson/scm/ JENKINS-16533 .zip http://jenkins-ci.org/commit/subversion-plugin/5ea622901e1ea04bdf9bc1bfc5b448844b6595ad Log: Merge pull request #51 from yoichi/master [FIXED JENKINS-16533] Respect external revision Compare: https://github.com/jenkinsci/subversion-plugin/compare/74b0f9ec5e23...5ea622901e1e

              People

              • Assignee:
                winotu Chris Z
                Reporter:
                mrysanek Michal Rysanek
              • Votes:
                12 Vote for this issue
                Watchers:
                19 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: