You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Benjamin Bentmann <be...@udo.edu> on 2008/08/07 20:53:40 UTC

svn commit: r683667 - in /maven/plugins/trunk: maven-changelog-plugin/ maven-checkstyle-plugin/ maven-clean-plugin/ maven-compiler-plugin/ maven-doap-plugin/ maven-docck-plugin/ maven-ejb-plugin/ maven-gpg-plugin/ maven-help-plugin/ maven-idea-plugin/ ... - bentmann Aug 07, 2008; 08:08pm

> Author: bentmann
> Date: Thu Aug  7 11:08:15 2008
> New Revision: 683667
> 
> URL: http://svn.apache.org/viewvc?rev=683667&view=rev
> Log:
> o Added stub IT profile to highlight plugins during CI

I fear I need to revert this, a reactor invocation in combinatin with 
the run-its profile is now failing due to a cyclic reference.


Benjamin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org

Re: svn commit: r683667 - in /maven/plugins/trunk: maven-changelog-plugin/ maven-checkstyle-plugin/ maven-clean-plugin/ maven-compiler-plugin/ maven-doap-plugin/ maven-docck-plugin/ maven-ejb-plugin/ maven-gpg-plugin/ maven-help-plugin/ maven-idea-plugin/ ... - bentmann Aug 07, 2008; 08:08pm

Posted by Benjamin Bentmann <be...@udo.edu>.
Benjamin Bentmann wrote:

> I fear I need to revert this, a reactor invocation in combinatin with 
> the run-its profile is now failing due to a cyclic reference.

This problem has now shown up on Hudson (due to r684249 I believe):
   https://ci.sonatype.org/job/plugins-CI-with-maven-2.0.x/105/console

Looking at the sources of ProjectSorter [0], I wondered why versions are 
not considered at all. The source code comments on trunk indicate that 
the problem is known.

Anyway, just wanted to point out that we now have a reproducible test 
case ;-)


Benjamin


[0] 
http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/ProjectSorter.java?view=markup

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org

Re: svn commit: r683667 - in /maven/plugins/trunk: maven-changelog-plugin/ maven-checkstyle-plugin/ maven-clean-plugin/ maven-compiler-plugin/ maven-doap-plugin/ maven-docck-plugin/ maven-ejb-plugin/ maven-gpg-plugin/ maven-help-plugin/ maven-idea-plugin/ ... - bentmann Aug 07, 2008; 08:08pm

Posted by Benjamin Bentmann <be...@udo.edu>.
Brett Porter wrote:

> Does that just mean some plugins need to be locked down to their 
> non-latest equivalent in the reactor build, so you aren't using what you 
> are building?

Actually, that should already be the case unless I am something missing. 
To reproduce the issue, simply add the snippet

   <profiles>
     <profile>
       <id>run-its</id>
       <build>
         <plugins>
           <plugin>
             <groupId>org.apache.maven.plugins</groupId>
             <artifactId>maven-antrun-plugin</artifactId>
           </plugin>
         </plugins>
       </build>
     </profile>
   </profiles>

to the POM of maven-invoker-plugin and run "mvn validate -P run-its" on 
maven-plugins. This gives the following build failure:

   The projects in the reactor contain a cyclic reference:
   Edge between
     'Vertex{label='org.apache.maven.plugins:maven-invoker-plugin'}'
   and
     'Vertex{label='org.apache.maven.plugins:maven-antrun-plugin'}'
   introduces to cycle in the graph
     org.apache.maven.plugins:maven-antrun-plugin -->
     org.apache.maven.plugins:maven-invoker-plugin -->
     org.apache.maven.plugins:maven-antrun-plugin

My personal impression is that the plugin versions are not properly 
considered. Starting at the POM for AntRun, we get the edge

   maven-antrun-plugin:1.3-SNAPSHOT -->
   maven-invoker-plugin:1.2.1-SNAPSHOT

Moving on to the Invoker POM, the next edge is

   maven-invoker-plugin:1.3-SNAPSHOT -->
   maven-antrun-plugin:1.2

so the artifacts involved are clearly different.


Benjamin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org

Re: svn commit: r683667 - in /maven/plugins/trunk: maven-changelog-plugin/ maven-checkstyle-plugin/ maven-clean-plugin/ maven-compiler-plugin/ maven-doap-plugin/ maven-docck-plugin/ maven-ejb-plugin/ maven-gpg-plugin/ maven-help-plugin/ maven-idea-plugin/ ... - bentmann Aug 07, 2008; 08:08pm

Posted by Brett Porter <br...@apache.org>.
Does that just mean some plugins need to be locked down to their non- 
latest equivalent in the reactor build, so you aren't using what you  
are building?

- Brett

On 08/08/2008, at 4:53 AM, Benjamin Bentmann wrote:

>> Author: bentmann
>> Date: Thu Aug  7 11:08:15 2008
>> New Revision: 683667
>> URL: http://svn.apache.org/viewvc?rev=683667&view=rev
>> Log:
>> o Added stub IT profile to highlight plugins during CI
>
> I fear I need to revert this, a reactor invocation in combinatin  
> with the run-its profile is now failing due to a cyclic reference.
>
>
> Benjamin
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>

--
Brett Porter
brett@apache.org
http://blogs.exist.com/bporter/


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org