You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ct...@apache.org on 2008/09/26 05:36:34 UTC

svn commit: r699165 - in /continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum: buildcontroller/DefaultBuildController.java execution/maven/m2/MavenTwoBuildExecutor.java

Author: ctan
Date: Thu Sep 25 20:36:34 2008
New Revision: 699165

URL: http://svn.apache.org/viewvc?rev=699165&view=rev
Log:
[CONTINUUM-1815] fixed scheduled builds of recently released projects

Modified:
    continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
    continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java

Modified: continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
URL: http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=699165&r1=699164&r2=699165&view=diff
==============================================================================
--- continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java (original)
+++ continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java Thu Sep 25 20:36:34 2008
@@ -24,6 +24,7 @@
 import org.apache.continuum.dao.ProjectDao;
 import org.apache.maven.continuum.core.action.AbstractContinuumAction;
 import org.apache.maven.continuum.execution.ContinuumBuildExecutor;
+import org.apache.maven.continuum.execution.ContinuumBuildExecutorConstants;
 import org.apache.maven.continuum.execution.manager.BuildExecutorManager;
 import org.apache.maven.continuum.model.project.BuildDefinition;
 import org.apache.maven.continuum.model.project.BuildResult;
@@ -539,7 +540,8 @@
         }
 
         // Check changes
-        if ( !shouldBuild && !allChangesUnknown && !context.getScmResult().getChanges().isEmpty() )
+        if ( !shouldBuild && ( ( !allChangesUnknown && !context.getScmResult().getChanges().isEmpty() )
+             || project.getExecutorId().equals( ContinuumBuildExecutorConstants.MAVEN_TWO_BUILD_EXECUTOR ) ) )
         {
             try
             {

Modified: continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
URL: http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java?rev=699165&r1=699164&r2=699165&view=diff
==============================================================================
--- continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java (original)
+++ continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java Thu Sep 25 20:36:34 2008
@@ -407,8 +407,25 @@
             return true;
         }
 
-        //check if changes are only in sub-modules or not
         MavenProject project = getMavenProject( continuumProject, workingDirectory, buildDefinition );
+
+        //CONTINUUM-1815: additional check for projects recently released
+        if ( !continuumProject.getVersion().equals( project.getVersion() ) )
+        {
+            getLogger().info( "Found changes in project's version ( maybe project was recently released ), building" );
+            return true;
+        }
+        
+        if ( changes.isEmpty() )
+        {
+            if ( getLogger().isDebugEnabled() )
+            {
+                getLogger().info( "Found no changes, not building" );
+            }
+            return false;
+        }
+        
+        //check if changes are only in sub-modules or not
         List<String> modules = project.getModules();
 
         List<ChangeFile> files = new ArrayList<ChangeFile>();



Re: svn commit: r699165 - in /continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum: buildcontroller/DefaultBuildController.java execution/maven/m2/MavenTwoBuildExecutor.java

Posted by Marica Tan <ct...@exist.com>.
Hmm.. I already added myself as a developer before in the parent pom.
--
Marica

On Fri, Sep 26, 2008 at 4:13 PM, Emmanuel Venisse <
emmanuel.venisse@gmail.com> wrote:

> Thanks.
> Can you add you as developer in the parent pom so your name will appear on
> scm changes on vmbuild.
>
> Emmanuel
>
> On Fri, Sep 26, 2008 at 9:29 AM, Marica Tan <ct...@exist.com> wrote:
>
> > Sorry. It's now fixed.
> >
> > Thanks,
> > --
> > Marica
> >
> > On Fri, Sep 26, 2008 at 12:50 PM, Emmanuel Venisse <
> > emmanuel.venisse@gmail.com> wrote:
> >
> > > This patch break the build.
> > >
> > > Emmanuel
> > >
> > > On Fri, Sep 26, 2008 at 5:36 AM, <ct...@apache.org> wrote:
> > >
> > > > Author: ctan
> > > > Date: Thu Sep 25 20:36:34 2008
> > > > New Revision: 699165
> > > >
> > > > URL: http://svn.apache.org/viewvc?rev=699165&view=rev
> > > > Log:
> > > > [CONTINUUM-1815] fixed scheduled builds of recently released projects
> > > >
> > > > Modified:
> > > >
> > > >
> > >
> >
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > > >
> > > >
> > >
> >
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > > >
> > > > Modified:
> > > >
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > > > URL:
> > > >
> > >
> >
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=699165&r1=699164&r2=699165&view=diff
> > > >
> > > >
> > >
> >
> ==============================================================================
> > > > ---
> > > >
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > > > (original)
> > > > +++
> > > >
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > > > Thu Sep 25 20:36:34 2008
> > > > @@ -24,6 +24,7 @@
> > > >  import org.apache.continuum.dao.ProjectDao;
> > > >  import
> org.apache.maven.continuum.core.action.AbstractContinuumAction;
> > > >  import org.apache.maven.continuum.execution.ContinuumBuildExecutor;
> > > > +import
> > > > org.apache.maven.continuum.execution.ContinuumBuildExecutorConstants;
> > > >  import
> > > org.apache.maven.continuum.execution.manager.BuildExecutorManager;
> > > >  import org.apache.maven.continuum.model.project.BuildDefinition;
> > > >  import org.apache.maven.continuum.model.project.BuildResult;
> > > > @@ -539,7 +540,8 @@
> > > >         }
> > > >
> > > >         // Check changes
> > > > -        if ( !shouldBuild && !allChangesUnknown &&
> > > > !context.getScmResult().getChanges().isEmpty() )
> > > > +        if ( !shouldBuild && ( ( !allChangesUnknown &&
> > > > !context.getScmResult().getChanges().isEmpty() )
> > > > +             || project.getExecutorId().equals(
> > > > ContinuumBuildExecutorConstants.MAVEN_TWO_BUILD_EXECUTOR ) ) )
> > > >         {
> > > >             try
> > > >             {
> > > >
> > > > Modified:
> > > >
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > > > URL:
> > > >
> > >
> >
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java?rev=699165&r1=699164&r2=699165&view=diff
> > > >
> > > >
> > >
> >
> ==============================================================================
> > > > ---
> > > >
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > > > (original)
> > > > +++
> > > >
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > > > Thu Sep 25 20:36:34 2008
> > > > @@ -407,8 +407,25 @@
> > > >             return true;
> > > >         }
> > > >
> > > > -        //check if changes are only in sub-modules or not
> > > >         MavenProject project = getMavenProject( continuumProject,
> > > > workingDirectory, buildDefinition );
> > > > +
> > > > +        //CONTINUUM-1815: additional check for projects recently
> > > released
> > > > +        if ( !continuumProject.getVersion().equals(
> > project.getVersion()
> > > )
> > > > )
> > > > +        {
> > > > +            getLogger().info( "Found changes in project's version (
> > > maybe
> > > > project was recently released ), building" );
> > > > +            return true;
> > > > +        }
> > > > +
> > > > +        if ( changes.isEmpty() )
> > > > +        {
> > > > +            if ( getLogger().isDebugEnabled() )
> > > > +            {
> > > > +                getLogger().info( "Found no changes, not building"
> );
> > > > +            }
> > > > +            return false;
> > > > +        }
> > > > +
> > > > +        //check if changes are only in sub-modules or not
> > > >         List<String> modules = project.getModules();
> > > >
> > > >         List<ChangeFile> files = new ArrayList<ChangeFile>();
> > > >
> > > >
> > > >
> > >
> >
>

Re: svn commit: r699165 - in /continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum: buildcontroller/DefaultBuildController.java execution/maven/m2/MavenTwoBuildExecutor.java

Posted by Emmanuel Venisse <em...@gmail.com>.
Thanks.
Can you add you as developer in the parent pom so your name will appear on
scm changes on vmbuild.

Emmanuel

On Fri, Sep 26, 2008 at 9:29 AM, Marica Tan <ct...@exist.com> wrote:

> Sorry. It's now fixed.
>
> Thanks,
> --
> Marica
>
> On Fri, Sep 26, 2008 at 12:50 PM, Emmanuel Venisse <
> emmanuel.venisse@gmail.com> wrote:
>
> > This patch break the build.
> >
> > Emmanuel
> >
> > On Fri, Sep 26, 2008 at 5:36 AM, <ct...@apache.org> wrote:
> >
> > > Author: ctan
> > > Date: Thu Sep 25 20:36:34 2008
> > > New Revision: 699165
> > >
> > > URL: http://svn.apache.org/viewvc?rev=699165&view=rev
> > > Log:
> > > [CONTINUUM-1815] fixed scheduled builds of recently released projects
> > >
> > > Modified:
> > >
> > >
> >
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > >
> > >
> >
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > >
> > > Modified:
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > > URL:
> > >
> >
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=699165&r1=699164&r2=699165&view=diff
> > >
> > >
> >
> ==============================================================================
> > > ---
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > > (original)
> > > +++
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > > Thu Sep 25 20:36:34 2008
> > > @@ -24,6 +24,7 @@
> > >  import org.apache.continuum.dao.ProjectDao;
> > >  import org.apache.maven.continuum.core.action.AbstractContinuumAction;
> > >  import org.apache.maven.continuum.execution.ContinuumBuildExecutor;
> > > +import
> > > org.apache.maven.continuum.execution.ContinuumBuildExecutorConstants;
> > >  import
> > org.apache.maven.continuum.execution.manager.BuildExecutorManager;
> > >  import org.apache.maven.continuum.model.project.BuildDefinition;
> > >  import org.apache.maven.continuum.model.project.BuildResult;
> > > @@ -539,7 +540,8 @@
> > >         }
> > >
> > >         // Check changes
> > > -        if ( !shouldBuild && !allChangesUnknown &&
> > > !context.getScmResult().getChanges().isEmpty() )
> > > +        if ( !shouldBuild && ( ( !allChangesUnknown &&
> > > !context.getScmResult().getChanges().isEmpty() )
> > > +             || project.getExecutorId().equals(
> > > ContinuumBuildExecutorConstants.MAVEN_TWO_BUILD_EXECUTOR ) ) )
> > >         {
> > >             try
> > >             {
> > >
> > > Modified:
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > > URL:
> > >
> >
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java?rev=699165&r1=699164&r2=699165&view=diff
> > >
> > >
> >
> ==============================================================================
> > > ---
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > > (original)
> > > +++
> > >
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > > Thu Sep 25 20:36:34 2008
> > > @@ -407,8 +407,25 @@
> > >             return true;
> > >         }
> > >
> > > -        //check if changes are only in sub-modules or not
> > >         MavenProject project = getMavenProject( continuumProject,
> > > workingDirectory, buildDefinition );
> > > +
> > > +        //CONTINUUM-1815: additional check for projects recently
> > released
> > > +        if ( !continuumProject.getVersion().equals(
> project.getVersion()
> > )
> > > )
> > > +        {
> > > +            getLogger().info( "Found changes in project's version (
> > maybe
> > > project was recently released ), building" );
> > > +            return true;
> > > +        }
> > > +
> > > +        if ( changes.isEmpty() )
> > > +        {
> > > +            if ( getLogger().isDebugEnabled() )
> > > +            {
> > > +                getLogger().info( "Found no changes, not building" );
> > > +            }
> > > +            return false;
> > > +        }
> > > +
> > > +        //check if changes are only in sub-modules or not
> > >         List<String> modules = project.getModules();
> > >
> > >         List<ChangeFile> files = new ArrayList<ChangeFile>();
> > >
> > >
> > >
> >
>

Re: svn commit: r699165 - in /continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum: buildcontroller/DefaultBuildController.java execution/maven/m2/MavenTwoBuildExecutor.java

Posted by Marica Tan <ct...@exist.com>.
Sorry. It's now fixed.

Thanks,
--
Marica

On Fri, Sep 26, 2008 at 12:50 PM, Emmanuel Venisse <
emmanuel.venisse@gmail.com> wrote:

> This patch break the build.
>
> Emmanuel
>
> On Fri, Sep 26, 2008 at 5:36 AM, <ct...@apache.org> wrote:
>
> > Author: ctan
> > Date: Thu Sep 25 20:36:34 2008
> > New Revision: 699165
> >
> > URL: http://svn.apache.org/viewvc?rev=699165&view=rev
> > Log:
> > [CONTINUUM-1815] fixed scheduled builds of recently released projects
> >
> > Modified:
> >
> >
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> >
> >
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> >
> > Modified:
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > URL:
> >
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=699165&r1=699164&r2=699165&view=diff
> >
> >
> ==============================================================================
> > ---
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > (original)
> > +++
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> > Thu Sep 25 20:36:34 2008
> > @@ -24,6 +24,7 @@
> >  import org.apache.continuum.dao.ProjectDao;
> >  import org.apache.maven.continuum.core.action.AbstractContinuumAction;
> >  import org.apache.maven.continuum.execution.ContinuumBuildExecutor;
> > +import
> > org.apache.maven.continuum.execution.ContinuumBuildExecutorConstants;
> >  import
> org.apache.maven.continuum.execution.manager.BuildExecutorManager;
> >  import org.apache.maven.continuum.model.project.BuildDefinition;
> >  import org.apache.maven.continuum.model.project.BuildResult;
> > @@ -539,7 +540,8 @@
> >         }
> >
> >         // Check changes
> > -        if ( !shouldBuild && !allChangesUnknown &&
> > !context.getScmResult().getChanges().isEmpty() )
> > +        if ( !shouldBuild && ( ( !allChangesUnknown &&
> > !context.getScmResult().getChanges().isEmpty() )
> > +             || project.getExecutorId().equals(
> > ContinuumBuildExecutorConstants.MAVEN_TWO_BUILD_EXECUTOR ) ) )
> >         {
> >             try
> >             {
> >
> > Modified:
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > URL:
> >
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java?rev=699165&r1=699164&r2=699165&view=diff
> >
> >
> ==============================================================================
> > ---
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > (original)
> > +++
> >
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> > Thu Sep 25 20:36:34 2008
> > @@ -407,8 +407,25 @@
> >             return true;
> >         }
> >
> > -        //check if changes are only in sub-modules or not
> >         MavenProject project = getMavenProject( continuumProject,
> > workingDirectory, buildDefinition );
> > +
> > +        //CONTINUUM-1815: additional check for projects recently
> released
> > +        if ( !continuumProject.getVersion().equals( project.getVersion()
> )
> > )
> > +        {
> > +            getLogger().info( "Found changes in project's version (
> maybe
> > project was recently released ), building" );
> > +            return true;
> > +        }
> > +
> > +        if ( changes.isEmpty() )
> > +        {
> > +            if ( getLogger().isDebugEnabled() )
> > +            {
> > +                getLogger().info( "Found no changes, not building" );
> > +            }
> > +            return false;
> > +        }
> > +
> > +        //check if changes are only in sub-modules or not
> >         List<String> modules = project.getModules();
> >
> >         List<ChangeFile> files = new ArrayList<ChangeFile>();
> >
> >
> >
>

Re: svn commit: r699165 - in /continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum: buildcontroller/DefaultBuildController.java execution/maven/m2/MavenTwoBuildExecutor.java

Posted by Emmanuel Venisse <em...@gmail.com>.
This patch break the build.

Emmanuel

On Fri, Sep 26, 2008 at 5:36 AM, <ct...@apache.org> wrote:

> Author: ctan
> Date: Thu Sep 25 20:36:34 2008
> New Revision: 699165
>
> URL: http://svn.apache.org/viewvc?rev=699165&view=rev
> Log:
> [CONTINUUM-1815] fixed scheduled builds of recently released projects
>
> Modified:
>
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
>
>  continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
>
> Modified:
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> URL:
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=699165&r1=699164&r2=699165&view=diff
>
> ==============================================================================
> ---
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> (original)
> +++
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
> Thu Sep 25 20:36:34 2008
> @@ -24,6 +24,7 @@
>  import org.apache.continuum.dao.ProjectDao;
>  import org.apache.maven.continuum.core.action.AbstractContinuumAction;
>  import org.apache.maven.continuum.execution.ContinuumBuildExecutor;
> +import
> org.apache.maven.continuum.execution.ContinuumBuildExecutorConstants;
>  import org.apache.maven.continuum.execution.manager.BuildExecutorManager;
>  import org.apache.maven.continuum.model.project.BuildDefinition;
>  import org.apache.maven.continuum.model.project.BuildResult;
> @@ -539,7 +540,8 @@
>         }
>
>         // Check changes
> -        if ( !shouldBuild && !allChangesUnknown &&
> !context.getScmResult().getChanges().isEmpty() )
> +        if ( !shouldBuild && ( ( !allChangesUnknown &&
> !context.getScmResult().getChanges().isEmpty() )
> +             || project.getExecutorId().equals(
> ContinuumBuildExecutorConstants.MAVEN_TWO_BUILD_EXECUTOR ) ) )
>         {
>             try
>             {
>
> Modified:
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> URL:
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java?rev=699165&r1=699164&r2=699165&view=diff
>
> ==============================================================================
> ---
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> (original)
> +++
> continuum/branches/continuum-1.2.x/continuum-core/src/main/java/org/apache/maven/continuum/execution/maven/m2/MavenTwoBuildExecutor.java
> Thu Sep 25 20:36:34 2008
> @@ -407,8 +407,25 @@
>             return true;
>         }
>
> -        //check if changes are only in sub-modules or not
>         MavenProject project = getMavenProject( continuumProject,
> workingDirectory, buildDefinition );
> +
> +        //CONTINUUM-1815: additional check for projects recently released
> +        if ( !continuumProject.getVersion().equals( project.getVersion() )
> )
> +        {
> +            getLogger().info( "Found changes in project's version ( maybe
> project was recently released ), building" );
> +            return true;
> +        }
> +
> +        if ( changes.isEmpty() )
> +        {
> +            if ( getLogger().isDebugEnabled() )
> +            {
> +                getLogger().info( "Found no changes, not building" );
> +            }
> +            return false;
> +        }
> +
> +        //check if changes are only in sub-modules or not
>         List<String> modules = project.getModules();
>
>         List<ChangeFile> files = new ArrayList<ChangeFile>();
>
>
>