You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2012/09/29 17:01:00 UTC

svn commit: r1391818 - in /maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal: MavenRepositorySystemSession.java MavenServiceLocator.java

Author: hboutemy
Date: Sat Sep 29 15:00:59 2012
New Revision: 1391818

URL: http://svn.apache.org/viewvc?rev=1391818&view=rev
Log:
fixed wrong explanations: repository system is a Plexus component to use instead of MavenServiceLocator, repository system session can be injected as plugin parameter

Modified:
    maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenRepositorySystemSession.java
    maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenServiceLocator.java

Modified: maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenRepositorySystemSession.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenRepositorySystemSession.java?rev=1391818&r1=1391817&r2=1391818&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenRepositorySystemSession.java (original)
+++ maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenRepositorySystemSession.java Sat Sep 29 15:00:59 2012
@@ -43,8 +43,13 @@ import org.sonatype.aether.util.reposito
 
 /**
  * A simplistic repository system session that mimics Maven's behavior to help third-party developers that want to embed
- * Maven's dependency resolution into their own applications. <strong>Warning:</strong> This class is not intended for
- * usage by Maven plugins, those should always acquire the current repository system session via parameter injection.
+ * Maven's dependency resolution into their own applications.
+ * 
+ * <p><strong>Warning:</strong> This class is not intended for
+ * usage by Maven plugins, those should always acquire the current repository system session via
+ * <a href="/ref/current/maven-core/apidocs/org/apache/maven/plugin/PluginParameterExpressionEvaluator.html">plugin
+ * parameter injection</a>, since the current repository system session is created by Maven in
+ * <code><a href="/ref/current/maven-core/apidocs/org/apache/maven/DefaultMaven.html">DefaultMaven.newRepositorySession(MavenExecutionRequest request)</a></code>.</p>
  * 
  * @author Benjamin Bentmann
  */

Modified: maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenServiceLocator.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenServiceLocator.java?rev=1391818&r1=1391817&r2=1391818&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenServiceLocator.java (original)
+++ maven/maven-3/trunk/maven-aether-provider/src/main/java/org/apache/maven/repository/internal/MavenServiceLocator.java Sat Sep 29 15:00:59 2012
@@ -30,10 +30,8 @@ import org.sonatype.aether.impl.VersionR
  * repository system, clients need to add some repository connectors for remote transfers.
  * 
  * <em>Note:</em> This component is meant to assist those clients that employ the repository systems outside of an IoC
- * container, Maven plugins should instead always use
- * <a href="/ref/current/maven-core/apidocs/org/apache/maven/plugin/PluginParameterExpressionEvaluator.html">regular
- * plugin parameter injection</a> to acquire the repository system (created by Maven in
- * <code><a href="/ref/current/maven-core/apidocs/org/apache/maven/DefaultMaven.html">DefaultMaven.newRepositorySession(MavenExecutionRequest request)</a></code>).
+ * container, Maven plugins should instead always use regular dependency injection to acquire the repository system:
+ * it is defined as Plexus component in aether-impl with its dependencies.
  * 
  * @author Benjamin Bentmann
  */