• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • other
    • None
    • Platform: All, OS: All

      The subcomponent for this enhancement should really be SCM, but I didn't see
      that option :/

      Please note:
      The enhancement below is best suited for "Latest" type builds. That is, builds
      that are run to make sure that when a projected is checked out from an SCM that
      it will always result in a SUCCESSFUL build.

      Description:
      Please see http://www.nabble.com/Excludes-List-tf4488311.html#a12855652 for full
      description.

      "Better SCM polling support that doesn't trigger a build for changes in certain
      regions"

      On a Job by Job level, have an "Excludes List/Region" under the "Build Triggers"
      that would specify files/folders that are irrelevant from the build's status
      (success or failure).

      For instance if you are running an apache webserver and have virtual-host
      definitions that redirect static content to a folder outside the application
      war/ear file, all static content becomes irrelevant to the status of a build,
      thus Hudson should not have to build the job if the only thing that has been
      updated in the SCM is static content.

      Another example is *.sql scripts. If your build process does not automatically
      run these scripts, then they become irrelevant to the build's status (success or
      failure). If one of these *.sql scripts is modified, it will not effect the
      build directly, thus Hudson should not schedule and run a build for it.

      This would save Hudson from having to schedule builds that will always succeed.

          [JENKINS-848] Excludes List Enhancement

          draco2k8 created issue -

          draco2k8 added a comment -

          I've found out that this is more common than I had previously thought.

          http://confluence.public.thoughtworks.org/display/CCNET/Filtered+Source+Control+Block

          It must be possible to do, as others have already attempted it and succeeded.
          To me this would be a great addon to Hudson.

          draco2k8 added a comment - I've found out that this is more common than I had previously thought. http://confluence.public.thoughtworks.org/display/CCNET/Filtered+Source+Control+Block It must be possible to do, as others have already attempted it and succeeded. To me this would be a great addon to Hudson.

          draco2k8 added a comment -

          I'd love to get this implemented, but haven't thought of a great way to do this.
          I know we'd have to support something like getModifications() for each call to
          pollChanges then compare those modifications to a list of regex exclusions.
          Does that sound about right?

          draco2k8 added a comment - I'd love to get this implemented, but haven't thought of a great way to do this. I know we'd have to support something like getModifications() for each call to pollChanges then compare those modifications to a list of regex exclusions. Does that sound about right?

          draco2k8 added a comment -

          Created an attachment (id=287)
          Patch for CVSSCM to cater for excluded regions

          draco2k8 added a comment - Created an attachment (id=287) Patch for CVSSCM to cater for excluded regions

          draco2k8 added a comment -

          Created an attachment (id=298)
          New patch that fixes a bug in the previous patch due to splitting the regex regions by newline.

          draco2k8 added a comment - Created an attachment (id=298) New patch that fixes a bug in the previous patch due to splitting the regex regions by newline.

          draco2k8 added a comment -

          Created an attachment (id=299)
          HTML help file for excluded regions

          draco2k8 added a comment - Created an attachment (id=299) HTML help file for excluded regions

          draco2k8 added a comment -

          I found a bug in the original patch, so please review CVS_Excluded_Regions_v2.
          I've also added the HTML help file.

          I'd really like to get this into the core but don't want to check it in without
          it being accepted as a temporary solution.

          draco2k8 added a comment - I found a bug in the original patch, so please review CVS_Excluded_Regions_v2. I've also added the HTML help file. I'd really like to get this into the core but don't want to check it in without it being accepted as a temporary solution.

          Code changed in hudson
          User: : draco2k8
          Path:
          trunk/hudson/main/core/src/main/java/hudson/scm/CVSSCM.java
          trunk/hudson/main/core/src/main/resources/hudson/scm/CVSSCM/config.jelly
          trunk/hudson/main/war/resources/help/_cvs/excludedRegions.html
          http://fisheye4.cenqua.com/changelog/hudson/?cs=15426
          Log:
          [FIXED JENKINS-848] Added ability for CVS polling to ignore commits of certain files as unimportant to the build process succeeding (e.g. HTML files) - thus a build would not be triggered.

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : draco2k8 Path: trunk/hudson/main/core/src/main/java/hudson/scm/CVSSCM.java trunk/hudson/main/core/src/main/resources/hudson/scm/CVSSCM/config.jelly trunk/hudson/main/war/resources/help/_cvs/excludedRegions.html http://fisheye4.cenqua.com/changelog/hudson/?cs=15426 Log: [FIXED JENKINS-848] Added ability for CVS polling to ignore commits of certain files as unimportant to the build process succeeding (e.g. HTML files) - thus a build would not be triggered.
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

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

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

            draco2k8 draco2k8
            draco2k8 draco2k8
            Votes:
            4 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: