• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Critical Critical
    • subversion-plugin
    • None
    • Platform: All, OS: All

      I have a project which is triggered by subversion changes. It has 3 repository
      URLs specified. Update is de-selected.

      Since I upgraded to 1.286, checkouts are not being triggered. What appears to be
      the case is this:

      Previously, a change in any of the repositories would trigger a checkout. In
      1.286, it appears that a checkout is only triggered if a change in the LAST of
      the listed repositories is detected.

      This has been observed by another user as well (except that he used update). He
      reported that downgrading to 1.285 fixed the problem.

          [JENKINS-3168] SVN polling broken in 1.286

          Alan Harder added a comment -
              • Issue 3183 has been marked as a duplicate of this issue. ***

          Alan Harder added a comment - Issue 3183 has been marked as a duplicate of this issue. ***

          cutterslade added a comment -

          I have also noticed this issue, although with slightly different circumstances.

          Several of my projects use the svn:externals property to link two projects, and
          several branches from each together. After upgrading from 1.285 to 1.287,
          builds are not being triggered by changes to the external branches. The SVN
          Polling log indicates that only the top level (the URL configured for the build)
          is being checked, before the upgrade, the polling log would include revision
          numbers of each of the external branches.

          cutterslade added a comment - I have also noticed this issue, although with slightly different circumstances. Several of my projects use the svn:externals property to link two projects, and several branches from each together. After upgrading from 1.285 to 1.287, builds are not being triggered by changes to the external branches. The SVN Polling log indicates that only the top level (the URL configured for the build) is being checked, before the upgrade, the polling log would include revision numbers of each of the external branches.

          Code changed in hudson
          User: : cliffano
          Path:
          trunk/hudson/extras/firefox-extension/buildmonitor/src/chrome/content/managers/buildmonitor-link-mgr.js
          trunk/hudson/extras/firefox-extension/buildmonitor/src/chrome/content/managers/buildmonitor-pref-mgr.js
          trunk/hudson/extras/firefox-extension/buildmonitor/src/defaults/preferences/buildmonitor.js
          http://fisheye4.cenqua.com/changelog/hudson/?cs=15958
          Log:
          JENKINS-3168 Added Hudson /rss pattern check. This makes 'Add Link To Hudson Build Monitor' menu only appears on Hudson rss feed links.
          In case someone wants to enable all links, this can be done by setting config value hudson.identifyrsspattern to false via about:config .

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : cliffano Path: trunk/hudson/extras/firefox-extension/buildmonitor/src/chrome/content/managers/buildmonitor-link-mgr.js trunk/hudson/extras/firefox-extension/buildmonitor/src/chrome/content/managers/buildmonitor-pref-mgr.js trunk/hudson/extras/firefox-extension/buildmonitor/src/defaults/preferences/buildmonitor.js http://fisheye4.cenqua.com/changelog/hudson/?cs=15958 Log: JENKINS-3168 Added Hudson /rss pattern check. This makes 'Add Link To Hudson Build Monitor' menu only appears on Hudson rss feed links. In case someone wants to enable all links, this can be done by setting config value hudson.identifyrsspattern to false via about:config .

          cliffano added a comment -

          I'm sorry, I mistyped the issue number. The above check-in comment was meant for
          issue 3169.

          cliffano added a comment - I'm sorry, I mistyped the issue number. The above check-in comment was meant for issue 3169.

          Alan Harder added a comment -

          looks like "return changesFound;" is inside the for-loop checking each
          location.. so pollChanges returns after checking the first item.

          Alan Harder added a comment - looks like "return changesFound;" is inside the for-loop checking each location.. so pollChanges returns after checking the first item.

          Code changed in hudson
          User: : mindless
          Path:
          trunk/hudson/main/core/src/main/java/hudson/scm/SubversionSCM.java
          trunk/hudson/main/test/src/test/java/hudson/scm/SubversionSCMTest.java
          trunk/www/changelog.html
          http://fisheye4.cenqua.com/changelog/hudson/?cs=15981
          Log:
          [FIXED JENKINS-3168] "return changesFound;" was inside for-loop for multiple locations,
          so pollChanges always returned after checking only first location.
          Also fixed form field validator to handle unix newlines.

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : mindless Path: trunk/hudson/main/core/src/main/java/hudson/scm/SubversionSCM.java trunk/hudson/main/test/src/test/java/hudson/scm/SubversionSCMTest.java trunk/www/changelog.html http://fisheye4.cenqua.com/changelog/hudson/?cs=15981 Log: [FIXED JENKINS-3168] "return changesFound;" was inside for-loop for multiple locations, so pollChanges always returned after checking only first location. Also fixed form field validator to handle unix newlines.

          mdonohue added a comment -
              • Issue 3210 has been marked as a duplicate of this issue. ***

          mdonohue added a comment - Issue 3210 has been marked as a duplicate of this issue. ***

          draco2k8 added a comment -

          @mindless: thanks for fixing this! I completely forgot I was in that loop!

          Sorry for the inconvenience cased.

          draco2k8 added a comment - @mindless: thanks for fixing this! I completely forgot I was in that loop! Sorry for the inconvenience cased.

          draco2k8 added a comment -
              • Issue 3214 has been marked as a duplicate of this issue. ***

          draco2k8 added a comment - Issue 3214 has been marked as a duplicate of this issue. ***

          mdonohue added a comment -
              • Issue 3232 has been marked as a duplicate of this issue. ***

          mdonohue added a comment - Issue 3232 has been marked as a duplicate of this issue. ***

            mindless Alan Harder
            mwebber Matthew Webber
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: