File maven2-2.2.1-migrate-to-plexus-containers-conta... of Package maven2.28016

Overview Repositories Revisions Requests Users Attributes Meta

File maven2-2.2.1-migrate-to-plexus-containers-container-default.patch of Package maven2.28016

From 7b57fd654710a1a21af22642eda12f8027289986 Mon Sep 17 00:00:00 2001
From: Michal Srb <msrb@redhat.com>
Date: Wed, 3 Jul 2013 10:04:03 +0200
Subject: [PATCH] Migrate to plexus-containers-container-default

---
 .../maven/profiles/DefaultProfileManager.java      | 28 ++++++++++++++++++----
 1 file changed, 23 insertions(+), 5 deletions(-)

diff --git a/maven-project/src/main/java/org/apache/maven/profiles/DefaultProfileManager.java b/maven-project/src/main/java/org/apache/maven/profiles/DefaultProfileManager.java
index e62d5ed..b72d515 100644
--- a/maven-project/src/main/java/org/apache/maven/profiles/DefaultProfileManager.java
+++ b/maven-project/src/main/java/org/apache/maven/profiles/DefaultProfileManager.java
@@ -26,8 +26,10 @@ import org.apache.maven.profiles.activation.ProfileActivator;
 import org.apache.maven.settings.Settings;
 import org.apache.maven.settings.SettingsUtils;
 import org.codehaus.plexus.PlexusContainer;
+import org.codehaus.plexus.component.factory.ComponentInstantiationException;
 import org.codehaus.plexus.component.repository.exception.ComponentLifecycleException;
 import org.codehaus.plexus.component.repository.exception.ComponentLookupException;
+import org.codehaus.plexus.logging.Logger;
 
 import java.util.ArrayList;
 import java.util.Iterator;
@@ -119,8 +121,12 @@ public class DefaultProfileManager
         Profile existing = (Profile) profilesById.get( profileId );
         if ( existing != null )
         {
-            container.getLogger().warn( "Overriding profile: \'" + profileId + "\' (source: " + existing.getSource() +
-                ") with new instance from source: " + profile.getSource() );
+            try {
+                container.lookup( Logger.class ).warn( "Overriding profile: \'" + profileId + "\' (source: " + existing.getSource() +
+                    ") with new instance from source: " + profile.getSource() );
+            } catch ( ComponentLookupException e ) {
+                throw new RuntimeException( e );
+            }
         }
 
         profilesById.put( profile.getId(), profile );
@@ -140,7 +146,11 @@ public class DefaultProfileManager
     {
         if ( !activatedIds.contains( profileId ) )
         {
-            container.getLogger().debug( "Profile with id: \'" + profileId + "\' has been explicitly activated." );
+            try {
+                container.lookup( Logger.class ).debug( "Profile with id: \'" + profileId + "\' has been explicitly activated." );
+            } catch ( ComponentLookupException e ) {
+                throw new RuntimeException( e );
+            }
 
             activatedIds.add( profileId );
         }
@@ -166,7 +176,11 @@ public class DefaultProfileManager
     {
         if ( !deactivatedIds.contains( profileId ) )
         {
-            container.getLogger().debug( "Profile with id: \'" + profileId + "\' has been explicitly deactivated." );
+            try {
+                container.lookup( Logger.class ).debug( "Profile with id: \'" + profileId + "\' has been explicitly deactivated." );
+            } catch ( ComponentLookupException e ) {
+                throw new RuntimeException( e );
+            }
 
             deactivatedIds.add( profileId );
         }
@@ -294,7 +308,11 @@ public class DefaultProfileManager
                 }
                 catch ( ComponentLifecycleException e )
                 {
-                    container.getLogger().debug( "Error releasing profile activators - ignoring.", e );
+                    try {
+                         container.lookup( Logger.class ).debug( "Error releasing profile activators - ignoring.", e );
+                    } catch ( ComponentLookupException ex ) {
+                        throw new RuntimeException( ex );
+                    }
                 }
             }
         }
-- 
1.8.1.4

Places

File maven2-2.2.1-migrate-to-plexus-containers-container-default.patch of Package maven2.28016

Places