You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2005/05/06 20:56:35 UTC

svn commit: r168635 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/cli/MavenCli.java

Author: brett
Date: Fri May  6 11:56:34 2005
New Revision: 168635

URL: http://svn.apache.org/viewcvs?rev=168635&view=rev
Log:
make sure debug really is debug by initialising before DefaultMaven gets looked up.

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/cli/MavenCli.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/cli/MavenCli.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/cli/MavenCli.java?rev=168635&r1=168634&r2=168635&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/cli/MavenCli.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/cli/MavenCli.java Fri May  6 11:56:34 2005
@@ -40,7 +40,6 @@
 import org.apache.maven.plugin.Mojo;
 import org.apache.maven.reactor.ReactorException;
 import org.apache.maven.settings.MavenSettingsBuilder;
-import org.apache.maven.settings.Profile;
 import org.apache.maven.settings.Settings;
 import org.codehaus.classworlds.ClassWorld;
 import org.codehaus.plexus.PlexusContainerException;
@@ -192,9 +191,16 @@
         MavenExecutionRequest request = null;
         try
         {
-            maven = createMavenInstance( embedder );
+            // logger must be created first
+            LoggerManager manager = (LoggerManager) embedder.lookup( LoggerManager.ROLE );
+            if ( debug )
+            {
+                manager.setThreshold( Logger.LEVEL_DEBUG );
+            }
+
+            request = createRequest( projectFiles, embedder, commandLine, settings, eventDispatcher, manager );
 
-            request = createRequest( projectFiles, embedder, commandLine, settings, eventDispatcher, debug );
+            maven = createMavenInstance( embedder );
         }
         catch ( ComponentLookupException e )
         {
@@ -247,7 +253,7 @@
 
     private static MavenExecutionRequest createRequest( List files, ArtifactEnabledEmbedder embedder,
                                                         CommandLine commandLine, Settings settings,
-                                                        EventDispatcher eventDispatcher, boolean debug )
+                                                        EventDispatcher eventDispatcher, LoggerManager manager )
         throws ComponentLookupException
     {
         MavenExecutionRequest request = null;
@@ -256,12 +262,6 @@
 
         request = new DefaultMavenExecutionRequest( localRepository, settings, eventDispatcher,
                                                     commandLine.getArgList(), files, userDir.getPath() );
-
-        LoggerManager manager = (LoggerManager) embedder.lookup( LoggerManager.ROLE );
-        if ( debug )
-        {
-            manager.setThreshold( Logger.LEVEL_DEBUG );
-        }
 
         // TODO [BP]: do we set one per mojo? where to do it?
         Logger logger = manager.getLoggerForComponent( Mojo.ROLE );



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org