Re: svn commit: r923057 - /gump/metadata/project/commons-proper.xml

2010-03-15 Thread sebb
On 15/03/2010, bode...@apache.org bode...@apache.org wrote:
 Author: bodewig
  Date: Mon Mar 15 05:23:18 2010
  New Revision: 923057

  URL: http://svn.apache.org/viewvc?rev=923057view=rev
  Log:
  canonical property to skip tests in mvn

  Modified:
 gump/metadata/project/commons-proper.xml

  Modified: gump/metadata/project/commons-proper.xml
  URL: 
 http://svn.apache.org/viewvc/gump/metadata/project/commons-proper.xml?rev=923057r1=923056r2=923057view=diff
  
 ==
  --- gump/metadata/project/commons-proper.xml (original)
  +++ gump/metadata/project/commons-proper.xml Mon Mar 15 05:23:18 2010
  @@ -449,7 +449,7 @@
  descriptionCommons I/O Utility Package/description

  mvn basedir=io goal=package
  -  property name=skipTests value=true/
  +  property name=maven.test.skip.exec value=true/

maven.test.skip.exec is deprecated:

http://maven.apache.org/plugins/maven-surefire-plugin/test-mojo.html#skipExec

which is why I used skipTests.

  /mvn

  option project=commons-lang-2.x /




-
To unsubscribe, e-mail: general-unsubscr...@gump.apache.org
For additional commands, e-mail: general-h...@gump.apache.org



Re: svn commit: r923057 - /gump/metadata/project/commons-proper.xml

2010-03-15 Thread Stefan Bodewig
On 2010-03-15, sebb seb...@gmail.com wrote:

 On 15/03/2010, bode...@apache.org bode...@apache.org wrote:

  URL: http://svn.apache.org/viewvc?rev=923057view=rev
  Log:
  canonical property to skip tests in mvn

  -  property name=skipTests value=true/
  + property name=maven.test.skip.exec value=true/

 maven.test.skip.exec is deprecated:

 http://maven.apache.org/plugins/maven-surefire-plugin/test-mojo.html#skipExec

 which is why I used skipTests.

I didn't know that and wanted things to be consistent - we use the now
deprecated version all over the place.

What is plugin version determined by?  The installed mvn version (2.2 by
now) or the project's POM?  skipTests would require Surefire 2.4 to work
and I don't know whether this is actually used by all projects.

Stefan

-
To unsubscribe, e-mail: general-unsubscr...@gump.apache.org
For additional commands, e-mail: general-h...@gump.apache.org



Re: svn commit: r923057 - /gump/metadata/project/commons-proper.xml

2010-03-15 Thread sebb
On 15/03/2010, Stefan Bodewig bode...@apache.org wrote:
 On 2010-03-15, sebb seb...@gmail.com wrote:

   On 15/03/2010, bode...@apache.org bode...@apache.org wrote:


   URL: http://svn.apache.org/viewvc?rev=923057view=rev
Log:
canonical property to skip tests in mvn


   -  property name=skipTests value=true/
+ property name=maven.test.skip.exec value=true/

   maven.test.skip.exec is deprecated:

   
 http://maven.apache.org/plugins/maven-surefire-plugin/test-mojo.html#skipExec

   which is why I used skipTests.


 I didn't know that and wanted things to be consistent - we use the now
  deprecated version all over the place.

  What is plugin version determined by?  The installed mvn version (2.2 by
  now) or the project's POM?

The project POM determines the version, assuming that the POM defines
the version.

  skipTests would require Surefire 2.4 to work
  and I don't know whether this is actually used by all projects.

Good point, but Commons-parent 13 uses 2.5

Note that skipExec itself requires 2.3, so could cause problems if a
project uses an earlier version of surefire.

Only skip is valid for all versions of Surefire, but that is not ideal.

  Stefan

  -
  To unsubscribe, e-mail: general-unsubscr...@gump.apache.org
  For additional commands, e-mail: general-h...@gump.apache.org



-
To unsubscribe, e-mail: general-unsubscr...@gump.apache.org
For additional commands, e-mail: general-h...@gump.apache.org



Re: svn commit: r923057 - /gump/metadata/project/commons-proper.xml

2010-03-15 Thread Stefan Bodewig
On 2010-03-15, sebb seb...@gmail.com wrote:

 On 15/03/2010, Stefan Bodewig bode...@apache.org wrote:
 On 2010-03-15, sebb seb...@gmail.com wrote:

 On 15/03/2010, bode...@apache.org bode...@apache.org wrote:


  URL: http://svn.apache.org/viewvc?rev=923057view=rev
  Log:
  canonical property to skip tests in mvn


  -  property name=skipTests value=true/
 property name=maven.test.skip.exec value=true/

 maven.test.skip.exec is deprecated:

 http://maven.apache.org/plugins/maven-surefire-plugin/test-mojo.html#skipExec

 which is why I used skipTests.


 I didn't know that and wanted things to be consistent - we use the now
 deprecated version all over the place.

  What is plugin version determined by?  The installed mvn version (2.2 by
  now) or the project's POM?

 The project POM determines the version, assuming that the POM defines
 the version.

  skipTests would require Surefire 2.4 to work
  and I don't know whether this is actually used by all projects.

 Good point, but Commons-parent 13 uses 2.5

I've reverted the commons-io project, if it works we can start moving
over other occurances of maven.test.skip.exec.  It wouldn't be the first
mvn property that failed to work as advertised in the Gump context
(project.build.finalName is one such example) - it may be that we just
don't understand Maven well enough, though.

Stefan

-
To unsubscribe, e-mail: general-unsubscr...@gump.apache.org
For additional commands, e-mail: general-h...@gump.apache.org