You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jv...@apache.org on 2007/02/10 10:20:12 UTC

svn commit: r505664 - in /maven/components/trunk/maven-embedder: ./ src/main/java/org/apache/maven/embedder/ src/test/java/org/apache/maven/embedder/

Author: jvanzyl
Date: Sat Feb 10 01:20:11 2007
New Revision: 505664

URL: http://svn.apache.org/viewvc?view=rev&rev=505664
Log:
o renaming the embedder request to embedder configuration which is more appropropriate
o move toward using one constructor which takes a configuration for session wide configuration, the execution still 
  takes an execution request as per usual


Added:
    maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedderConfiguration.java   (contents, props changed)
      - copied, changed from r505612, maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedRequest.java
    maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderConfiguration.java   (contents, props changed)
      - copied, changed from r505612, maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedRequest.java
Removed:
    maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedRequest.java
    maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedRequest.java
Modified:
    maven/components/trunk/maven-embedder/pom.xml
    maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java
    maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java
    maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/TestComponentOverride.java

Modified: maven/components/trunk/maven-embedder/pom.xml
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/pom.xml?view=diff&rev=505664&r1=505663&r2=505664
==============================================================================
--- maven/components/trunk/maven-embedder/pom.xml (original)
+++ maven/components/trunk/maven-embedder/pom.xml Sat Feb 10 01:20:11 2007
@@ -27,7 +27,7 @@
   <artifactId>maven-embedder</artifactId>
   <name>Maven Embedder</name>
   <properties>
-    <bundleVersion>2.1.0.v20070209-2050</bundleVersion>
+    <bundleVersion>2.1.0.v20070209-2105</bundleVersion>
   </properties>
   <build>
     <resources>

Copied: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedderConfiguration.java (from r505612, maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedRequest.java)
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedderConfiguration.java?view=diff&rev=505664&p1=maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedRequest.java&r1=505612&p2=maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedderConfiguration.java&r2=505664
==============================================================================
--- maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedRequest.java (original)
+++ maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedderConfiguration.java Sat Feb 10 01:20:11 2007
@@ -25,12 +25,12 @@
 import org.codehaus.plexus.classworlds.ClassWorld;
 
 /**
- * Default implementation of MavenEmbedRequest intefrace.
+ * Default implementation of MavenEmbedderConfiguration intefrace.
  *
  * @author mkleint
  */
-public class DefaultMavenEmbedRequest
-    implements MavenEmbedRequest
+public class DefaultMavenEmbedderConfiguration
+    implements MavenEmbedderConfiguration
 {
     private List inactives;
 
@@ -53,30 +53,30 @@
 
     private ClassWorld classWorld;
 
-    /** Creates a new instance of DefaultMavenEmbedRequest */
-    public DefaultMavenEmbedRequest()
+    /** Creates a new instance of DefaultMavenEmbedderConfiguration */
+    public DefaultMavenEmbedderConfiguration()
     {
     }
 
-    public MavenEmbedRequest addActiveProfile( String profile )
+    public MavenEmbedderConfiguration addActiveProfile( String profile )
     {
         getActiveProfiles().add( profile );
         return this;
     }
 
-    public MavenEmbedRequest addInactiveProfile( String profile )
+    public MavenEmbedderConfiguration addInactiveProfile( String profile )
     {
         getInactiveProfiles().add( profile );
         return this;
     }
 
-    public MavenEmbedRequest addActiveProfiles( List profiles )
+    public MavenEmbedderConfiguration addActiveProfiles( List profiles )
     {
         getActiveProfiles().addAll( profiles );
         return this;
     }
 
-    public MavenEmbedRequest addInactiveProfiles( List profiles )
+    public MavenEmbedderConfiguration addInactiveProfiles( List profiles )
     {
         getInactiveProfiles().addAll( profiles );
         return this;
@@ -100,13 +100,13 @@
         return inactives;
     }
 
-    public MavenEmbedRequest setUserSettingsFile( File user )
+    public MavenEmbedderConfiguration setUserSettingsFile( File user )
     {
         userSettings = user;
         return this;
     }
 
-    public MavenEmbedRequest setGlobalSettingsFile( File global )
+    public MavenEmbedderConfiguration setGlobalSettingsFile( File global )
     {
         globalSettings = global;
         return this;
@@ -122,7 +122,7 @@
         return globalSettings;
     }
 
-    public MavenEmbedRequest setConfigurationCustomizer( ContainerCustomizer customizer )
+    public MavenEmbedderConfiguration setConfigurationCustomizer( ContainerCustomizer customizer )
     {
         this.customizer = customizer;
         return this;
@@ -133,7 +133,7 @@
         return customizer;
     }
 
-    public MavenEmbedRequest setSystemProperties( Properties properties )
+    public MavenEmbedderConfiguration setSystemProperties( Properties properties )
     {
         systemProperties = properties;
         return this;
@@ -154,7 +154,7 @@
         return extensions;
     }
 
-    public MavenEmbedRequest setMavenEmbedderLogger( MavenEmbedderLogger logger )
+    public MavenEmbedderConfiguration setMavenEmbedderLogger( MavenEmbedderLogger logger )
     {
         this.logger = logger;
         return this;
@@ -170,13 +170,13 @@
         return classWorld;
     }
 
-    public MavenEmbedRequest setClassWorld( ClassWorld classWorld )
+    public MavenEmbedderConfiguration setClassWorld( ClassWorld classWorld )
     {
         this.classWorld = classWorld;
         return this;
     }
 
-    public MavenEmbedRequest setClassLoader( ClassLoader loader )
+    public MavenEmbedderConfiguration setClassLoader( ClassLoader loader )
     {
         this.classWorld = new ClassWorld( "plexus.core", loader );
         return this;

Propchange: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedderConfiguration.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/DefaultMavenEmbedderConfiguration.java
------------------------------------------------------------------------------
    svn:keywords = "Author Date Id Revision"

Modified: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java?view=diff&rev=505664&r1=505663&r2=505664
==============================================================================
--- maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java (original)
+++ maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java Sat Feb 10 01:20:11 2007
@@ -142,7 +142,7 @@
     // User options
     // ----------------------------------------------------------------------
 
-    private MavenEmbedRequest embedderRequest;
+    private MavenEmbedderConfiguration embedderRequest;
 
     // ----------------------------------------------------------------------------
     // Constructors
@@ -158,7 +158,7 @@
                           MavenEmbedderLogger logger )
         throws MavenEmbedderException
     {
-        this( new DefaultMavenEmbedRequest().setClassWorld( classWorld ).setMavenEmbedderLogger( logger ) );
+        this( new DefaultMavenEmbedderConfiguration().setClassWorld( classWorld ).setMavenEmbedderLogger( logger ) );
     }
 
     public MavenEmbedder( ClassLoader classLoader )
@@ -174,10 +174,10 @@
         this( new ClassWorld( "plexus.core", classLoader ), logger );
     }
 
-    public MavenEmbedder( MavenEmbedRequest req )
+    public MavenEmbedder( MavenEmbedderConfiguration embedderConfiguration )
         throws MavenEmbedderException
     {
-        start( req );
+        start( embedderConfiguration );
     }
 
     // ----------------------------------------------------------------------
@@ -488,7 +488,7 @@
     //  Lifecycle
     // ----------------------------------------------------------------------
 
-    private void start( MavenEmbedRequest req )
+    private void start( MavenEmbedderConfiguration req )
         throws MavenEmbedderException
     {
         this.classWorld = req.getClassWorld();

Copied: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderConfiguration.java (from r505612, maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedRequest.java)
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderConfiguration.java?view=diff&rev=505664&p1=maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedRequest.java&r1=505612&p2=maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderConfiguration.java&r2=505664
==============================================================================
--- maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedRequest.java (original)
+++ maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderConfiguration.java Sat Feb 10 01:20:11 2007
@@ -27,46 +27,46 @@
  *
  * @author mkleint
  */
-public interface MavenEmbedRequest
+public interface MavenEmbedderConfiguration
 {
     /*
     * Add profile to activate.
     */
-    MavenEmbedRequest addActiveProfile( String profile );
+    MavenEmbedderConfiguration addActiveProfile( String profile );
 
     /*
      * Add profile to inactivate.
      */
-    MavenEmbedRequest addInactiveProfile( String profile );
+    MavenEmbedderConfiguration addInactiveProfile( String profile );
 
     /*
     * Add a list of String instances with names of profiles to activate.
     */
-    MavenEmbedRequest addActiveProfiles( List profiles );
+    MavenEmbedderConfiguration addActiveProfiles( List profiles );
 
     /*
     * Add a list of String instances with names of profiles to inactivate.
     */
-    MavenEmbedRequest addInactiveProfiles( List profiles );
+    MavenEmbedderConfiguration addInactiveProfiles( List profiles );
 
     /*
     * Set location of the user settings file to use for the embedder.
     */
-    MavenEmbedRequest setUserSettingsFile( File user );
+    MavenEmbedderConfiguration setUserSettingsFile( File user );
 
     /*
      * Set location of the global settings file to use for the embedder.
      */
-    MavenEmbedRequest setGlobalSettingsFile( File global );
+    MavenEmbedderConfiguration setGlobalSettingsFile( File global );
 
     /**
      * Set a customizer callback implemetation that will be given a chance to modify the plexus container
      * on startup.
      */
-    MavenEmbedRequest setConfigurationCustomizer( ContainerCustomizer customizer );
+    MavenEmbedderConfiguration setConfigurationCustomizer( ContainerCustomizer customizer );
 
     /** set the system properties to be used during the lifecycle of the embedder. Excluding the time when executing the project, then the properties from MavenExecutionRequestare used. */
-    MavenEmbedRequest setSystemProperties( Properties properties );
+    MavenEmbedderConfiguration setSystemProperties( Properties properties );
 
     List getActiveProfiles();
 
@@ -84,13 +84,13 @@
 
     List getExtensions();
 
-    public MavenEmbedRequest setMavenEmbedderLogger( MavenEmbedderLogger logger );
+    public MavenEmbedderConfiguration setMavenEmbedderLogger( MavenEmbedderLogger logger );
 
     public MavenEmbedderLogger getMavenEmbedderLogger();
 
     public ClassWorld getClassWorld();
 
-    public MavenEmbedRequest setClassWorld( ClassWorld classWorld );
+    public MavenEmbedderConfiguration setClassWorld( ClassWorld classWorld );
 
-    public MavenEmbedRequest setClassLoader( ClassLoader loader );
+    public MavenEmbedderConfiguration setClassLoader( ClassLoader loader );
 }

Propchange: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderConfiguration.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderConfiguration.java
------------------------------------------------------------------------------
    svn:keywords = "Author Date Id Revision"

Modified: maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java?view=diff&rev=505664&r1=505663&r2=505664
==============================================================================
--- maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java (original)
+++ maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java Sat Feb 10 01:20:11 2007
@@ -230,8 +230,6 @@
         assertEquals( 1, artifacts.size() );
 
         Artifact artifact = (Artifact) artifacts.iterator().next();
-
-        System.out.println( "artifact = " + artifact );
     }
 
     public void testProjectWithExtensionsReading()

Modified: maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/TestComponentOverride.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/TestComponentOverride.java?view=diff&rev=505664&r1=505663&r2=505664
==============================================================================
--- maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/TestComponentOverride.java (original)
+++ maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/TestComponentOverride.java Sat Feb 10 01:20:11 2007
@@ -25,7 +25,7 @@
 
         ClassLoader loader = Thread.currentThread().getContextClassLoader();
 
-        MavenEmbedRequest request = new DefaultMavenEmbedRequest();
+        MavenEmbedderConfiguration request = new DefaultMavenEmbedderConfiguration();
 
         request.setClassLoader( loader );