Deprecating the bundleall goal

2011-06-16 Thread Stuart McCulloch
Hi folks,

I'm planning to deprecate the bundleall goal since it has several faults, 
produces low quality bundles, and was never meant to be used by everyday 
projects.

Any objections?  The goal will still be available in the next minor release, 
but marked as deprecated with a warning - the next major release would remove 
it.

--
Cheers, Stuart

Begin forwarded message:

 From: mccu...@apache.org
 Date: 16 June 2011 18:41:19 GMT+01:00
 To: comm...@felix.apache.org
 Subject: svn commit: r1136563 - 
 /felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
 Reply-To: dev@felix.apache.org
 
 Author: mcculls
 Date: Thu Jun 16 17:41:18 2011
 New Revision: 1136563
 
 URL: http://svn.apache.org/viewvc?rev=1136563view=rev
 Log:
 Deprecate the bundleall goal
 
 Modified:

 felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
 
 Modified: 
 felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
 URL: 
 http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java?rev=1136563r1=1136562r2=1136563view=diff
 ==
 --- 
 felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
  (original)
 +++ 
 felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
  Thu Jun 16 17:41:18 2011
 @@ -64,7 +64,9 @@ import aQute.lib.osgi.Jar;
  * @phase package
  * @requiresDependencyResolution test
  * @description build an OSGi bundle jar for all transitive dependencies
 + * @deprecated The bundleall goal is no longer supported and may be removed 
 in a future release
  */
 +@Deprecated
 public class BundleAllPlugin extends ManifestPlugin
 {
 private static final String LS = System.getProperty( line.separator );
 @@ -174,6 +176,9 @@ public class BundleAllPlugin extends Man
  */
 protected BundleInfo bundleAll( MavenProject project, int maxDepth ) 
 throws MojoExecutionException
 {
 +getLog().warn( 
 
  );
 +getLog().warn( ! The bundleall goal is no longer supported and may 
 be removed in a future release ! );
 +getLog().warn( 
 
  );
 
 if ( alreadyBundled( project.getArtifact() ) )
 {
 
 



Re: Deprecating the bundleall goal

2011-06-16 Thread Jean-Baptiste Onofré

+1

Regards
JB

On 06/16/2011 07:49 PM, Stuart McCulloch wrote:

Hi folks,

I'm planning to deprecate the bundleall goal since it has several faults, 
produces low quality bundles, and was never meant to be used by everyday projects.

Any objections?  The goal will still be available in the next minor release, 
but marked as deprecated with a warning - the next major release would remove 
it.

--
Cheers, Stuart

Begin forwarded message:


From: mccu...@apache.org
Date: 16 June 2011 18:41:19 GMT+01:00
To: comm...@felix.apache.org
Subject: svn commit: r1136563 - 
/felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
Reply-To: dev@felix.apache.org

Author: mcculls
Date: Thu Jun 16 17:41:18 2011
New Revision: 1136563

URL: http://svn.apache.org/viewvc?rev=1136563view=rev
Log:
Deprecate the bundleall goal

Modified:

felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java

Modified: 
felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java?rev=1136563r1=1136562r2=1136563view=diff
==
--- 
felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
 (original)
+++ 
felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/BundleAllPlugin.java
 Thu Jun 16 17:41:18 2011
@@ -64,7 +64,9 @@ import aQute.lib.osgi.Jar;
  * @phase package
  * @requiresDependencyResolution test
  * @description build an OSGi bundle jar for all transitive dependencies
+ * @deprecated The bundleall goal is no longer supported and may be removed in 
a future release
  */
+@Deprecated
public class BundleAllPlugin extends ManifestPlugin
{
 private static final String LS = System.getProperty( line.separator );
@@ -174,6 +176,9 @@ public class BundleAllPlugin extends Man
  */
 protected BundleInfo bundleAll( MavenProject project, int maxDepth ) 
throws MojoExecutionException
 {
+getLog().warn( 

 );
+getLog().warn( ! The bundleall goal is no longer supported and may be 
removed in a future release ! );
+getLog().warn( 

 );

 if ( alreadyBundled( project.getArtifact() ) )
 {