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

Subversion Plugin should be able to ignore property only commits (eps. Mergeinfos)

    Details

    • Similar Issues:

      Description

      (I already created a pull request for this: https://github.com/jenkinsci/subversion-plugin/pull/14, this issue is just to keep track of it)

      This is useful for Incremental Builds in Maven in a branch that is target of merges (i.e.: trunk). Without that this modification consider a project of 100 modules. Every merge (even a "block-revision" merge, that doesn't change any files) changes the top directory of the project (svn:mergeinfo), i.e. from the perspective of Jenkins, the aggregator project has changed (which is usually inherited by all modules). Jenkins thus includes the aggregator in the list of changed modules and ALL modules are build.

      When doing frequent merges this results in continual rebuild of the whole project, totally negating the advantages of incremental builds.

      Subversion plugin should be able to exclude property-only changes on directories from:

      • the SCM-Polling (i.e. a change that only consists of property changes on directories does not result in a new build)
      • the changelog used by MavenModuleSet (or Ivy) when calculating the changed modules of a new revision.

      A changed path is considered Property Only when it consists of a "M"odified Action on a directory.

        Attachments

          Issue Links

            Activity

            paux Stephan Pauxberger created issue -
            kutzi kutzi made changes -
            Field Original Value New Value
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            paux Stephan Pauxberger made changes -
            Link This issue is related to JENKINS-16296 [ JENKINS-16296 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 145381 ] JNJira + In-Review [ 191462 ]

              People

              • Assignee:
                Unassigned
                Reporter:
                paux Stephan Pauxberger
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: