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

Subversion plugin 1.14 does not have same behavior for "Excluded Regions" as 1.13

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      There are several "excluded regions" configured in our Hudson system when using Subversion plugin 1.13, and it works fine. After upgrading to SVN plugin 1.14, it does not work at all.

      Instead we have to configure "Included Regions" and then whole system can work as before.

      It is easier to configure only included regions when we have much more paths to be ignored, but I think the plugin should have same behavior no matter "included" or "excluded" is used.

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in hudson
            User: : abayer
            Path:
            trunk/hudson/plugins/subversion/src/test/java/hudson/scm/SubversionSCMTest.java
            trunk/hudson/plugins/subversion/src/test/resources/hudson/scm/JENKINS-6030.zip
            http://jenkins-ci.org/commit/29163
            Log:
            JENKINS-6030 Better test.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : abayer Path: trunk/hudson/plugins/subversion/src/test/java/hudson/scm/SubversionSCMTest.java trunk/hudson/plugins/subversion/src/test/resources/hudson/scm/ JENKINS-6030 .zip http://jenkins-ci.org/commit/29163 Log: JENKINS-6030 Better test.
            Hide
            abayer Andrew Bayer added a comment -

            I'm pretty sure the latest is a good test - when I comment out the assertFalse after the "bar" commit, it all works. i.e., the "foo" commit gets seen just fine. But the "bar" commit is also being seen, which is the problem.

            Show
            abayer Andrew Bayer added a comment - I'm pretty sure the latest is a good test - when I comment out the assertFalse after the "bar" commit, it all works. i.e., the "foo" commit gets seen just fine. But the "bar" commit is also being seen, which is the problem.
            Hide
            pgweiss pgweiss added a comment -

            No it isn't. I have a later checkin that fixes it.

            The problem is that the regex has to match the whole line. So "bar" is wrong. It needs to be ".*/bar".

            Show
            pgweiss pgweiss added a comment - No it isn't. I have a later checkin that fixes it. The problem is that the regex has to match the whole line. So "bar" is wrong. It needs to be ".*/bar".
            Hide
            pgweiss pgweiss added a comment -

            Shouldn't have been reopened. The test was buggy.

            Show
            pgweiss pgweiss added a comment - Shouldn't have been reopened. The test was buggy.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in hudson
            User: : pgweiss
            Path:
            trunk/hudson/plugins/subversion/src/test/java/hudson/scm/SubversionSCMTest.java
            http://jenkins-ci.org/commit/29165
            Log:
            JENKINS-6030 Fix buggy testExcludeByRegion and add testIncludeByRegion. Also fix testConfigRoundTrip.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : pgweiss Path: trunk/hudson/plugins/subversion/src/test/java/hudson/scm/SubversionSCMTest.java http://jenkins-ci.org/commit/29165 Log: JENKINS-6030 Fix buggy testExcludeByRegion and add testIncludeByRegion. Also fix testConfigRoundTrip.

              People

              • Assignee:
                pgweiss pgweiss
                Reporter:
                howjay howjay
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: