Deprecating the bundleall goal
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
+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() ) ) {