You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kh...@apache.org on 2016/04/10 15:09:18 UTC
svn commit: r1738433 - in /maven/shared/trunk/maven-archiver/src:
main/java/org/apache/maven/archiver/ManifestConfiguration.java
test/java/org/apache/maven/archiver/MavenArchiverTest.java
Author: khmarbaise
Date: Sun Apr 10 13:09:18 2016
New Revision: 1738433
URL: http://svn.apache.org/viewvc?rev=1738433&view=rev
Log:
[MSHARED-512] Remove @Deprecated marked code which has been missed.
o Removed classpathMavenRepositoryLayout can be replaced by
using calling setClasspathLayoutType and using the appropriate
expression. See setCustomClasspathLayout().
Modified:
maven/shared/trunk/maven-archiver/src/main/java/org/apache/maven/archiver/ManifestConfiguration.java
maven/shared/trunk/maven-archiver/src/test/java/org/apache/maven/archiver/MavenArchiverTest.java
Modified: maven/shared/trunk/maven-archiver/src/main/java/org/apache/maven/archiver/ManifestConfiguration.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-archiver/src/main/java/org/apache/maven/archiver/ManifestConfiguration.java?rev=1738433&r1=1738432&r2=1738433&view=diff
==============================================================================
--- maven/shared/trunk/maven-archiver/src/main/java/org/apache/maven/archiver/ManifestConfiguration.java (original)
+++ maven/shared/trunk/maven-archiver/src/main/java/org/apache/maven/archiver/ManifestConfiguration.java Sun Apr 10 13:09:18 2016
@@ -70,15 +70,6 @@ public class ManifestConfiguration
*/
private boolean addDefaultImplementationEntries;
- /**
- * The created Class-Path entry will contains paths that follow the Maven 2 repository layout:
- * $groupId[0]/../${groupId[n]/$artifactId/$version/{fileName}
- *
- * @since 2.3
- * @deprecated Use {@link ManifestConfiguration#classpathLayoutType} instead.
- */
- private boolean classpathMavenRepositoryLayout = false;
-
private String classpathLayoutType = CLASSPATH_LAYOUT_TYPE_SIMPLE;
private String customClasspathLayout;
@@ -134,17 +125,6 @@ public class ManifestConfiguration
}
/**
- * @deprecated Use {@link ManifestConfiguration#getClasspathLayoutType()}, and compare to
- * CLASSPATH_LAYOUT_TYPE_SIMPLE or CLASSPATH_LAYOUT_TYPE_REPOSITORY, also declared in
- * {@link ManifestConfiguration}.
- * @return The state of classpath Maven repository layout.
- */
- public boolean isClasspathMavenRepositoryLayout()
- {
- return classpathMavenRepositoryLayout;
- }
-
- /**
* @param addClasspath turn on addClasspath or off.
*/
public void setAddClasspath( boolean addClasspath )
@@ -177,17 +157,6 @@ public class ManifestConfiguration
}
/**
- * @deprecated Use {@link ManifestConfiguration#setClasspathLayoutType(String)}, and use
- * CLASSPATH_LAYOUT_TYPE_SIMPLE, CLASSPATH_LAYOUT_TYPE_CUSTOM, or CLASSPATH_LAYOUT_TYPE_REPOSITORY, also
- * declared in {@link ManifestConfiguration}.
- * @param classpathMavenRepositoryLayout true/false classpath maven repository
- */
- public void setClasspathMavenRepositoryLayout( boolean classpathMavenRepositoryLayout )
- {
- this.classpathMavenRepositoryLayout = classpathMavenRepositoryLayout;
- }
-
- /**
* @param classpathPrefix The prefix.
*/
public void setClasspathPrefix( String classpathPrefix )
@@ -238,7 +207,7 @@ public class ManifestConfiguration
public String getClasspathLayoutType()
{
// CHECKSTYLE_OFF: LineLength
- return CLASSPATH_LAYOUT_TYPE_SIMPLE.equals( classpathLayoutType ) && classpathMavenRepositoryLayout ? CLASSPATH_LAYOUT_TYPE_REPOSITORY
+ return CLASSPATH_LAYOUT_TYPE_SIMPLE.equals( classpathLayoutType ) ? CLASSPATH_LAYOUT_TYPE_REPOSITORY
: classpathLayoutType;
// CHECKSTYLE_ON: LineLength
}
@@ -284,6 +253,14 @@ public class ManifestConfiguration
* </ol>
* <br>
* <b>NOTE:</b> If you specify a layout type of 'custom' you MUST set this layout expression.
+ * You can take a look at
+ * <ol>
+ * <li>{@link MavenArchiver#SIMPLE_LAYOUT}</li>
+ * <li>{@link MavenArchiver#SIMPLE_LAYOUT_NONUNIQUE}</li>
+ * <li>{@link MavenArchiver#REPOSITORY_LAYOUT}</li>
+ * <li>{@link MavenArchiver#REPOSITORY_LAYOUT_NONUNIQUE}</li>
+ * </ol>
+ * how such an expression looks like.
* @param customClasspathLayout The custom classpath layout.
*/
public void setCustomClasspathLayout( String customClasspathLayout )
Modified: maven/shared/trunk/maven-archiver/src/test/java/org/apache/maven/archiver/MavenArchiverTest.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-archiver/src/test/java/org/apache/maven/archiver/MavenArchiverTest.java?rev=1738433&r1=1738432&r2=1738433&view=diff
==============================================================================
--- maven/shared/trunk/maven-archiver/src/test/java/org/apache/maven/archiver/MavenArchiverTest.java (original)
+++ maven/shared/trunk/maven-archiver/src/test/java/org/apache/maven/archiver/MavenArchiverTest.java Sun Apr 10 13:09:18 2016
@@ -598,6 +598,8 @@ public class MavenArchiverTest
config.getManifest().setAddDefaultSpecificationEntries( true );
config.getManifest().setMainClass( "org.apache.maven.Foo" );
config.getManifest().setAddClasspath( true );
+ config.getManifest().setClasspathLayoutType( ManifestConfiguration.CLASSPATH_LAYOUT_TYPE_CUSTOM );
+ config.getManifest().setCustomClasspathLayout( "${artifact.artifactId}-${artifact.version}${dashClassifier?}.${artifact.extension}" );
archiver.createArchive( session, project, config );
assertTrue( jarFile.exists() );
final Manifest manifest = getJarFileManifest( jarFile );
@@ -631,6 +633,8 @@ public class MavenArchiverTest
config.getManifest().setAddDefaultSpecificationEntries( true );
config.getManifest().setMainClass( "org.apache.maven.Foo" );
config.getManifest().setAddClasspath( true );
+ config.getManifest().setClasspathLayoutType( ManifestConfiguration.CLASSPATH_LAYOUT_TYPE_CUSTOM );
+ config.getManifest().setCustomClasspathLayout( "${artifact.artifactId}-${artifact.version}${dashClassifier?}.${artifact.extension}" );
archiver.createArchive( session, project, config );
assertTrue( jarFile.exists() );