You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2013/01/27 20:38:02 UTC

svn commit: r1439154 - in /tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader: TomEEJarScanner.java TomcatHelper.java

Author: rmannibucau
Date: Sun Jan 27 19:38:02 2013
New Revision: 1439154

URL: http://svn.apache.org/viewvc?rev=1439154&view=rev
Log:
myfaces-impl tlds should be parsed too

Modified:
    tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomEEJarScanner.java
    tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomcatHelper.java

Modified: tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomEEJarScanner.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomEEJarScanner.java?rev=1439154&r1=1439153&r2=1439154&view=diff
==============================================================================
--- tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomEEJarScanner.java (original)
+++ tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomEEJarScanner.java Sun Jan 27 19:38:02 2013
@@ -31,7 +31,6 @@ import org.apache.openejb.OpenEJBExcepti
 import org.apache.openejb.OpenEJBRuntimeException;
 import org.apache.openejb.config.TldScanner;
 import org.apache.openejb.loader.IO;
-import org.apache.openejb.util.DaemonThreadFactory;
 import org.apache.openejb.util.URLs;
 import org.apache.openejb.util.reflection.Reflections;
 import org.apache.tomcat.JarScannerCallback;
@@ -54,17 +53,11 @@ import java.net.URISyntaxException;
 import java.net.URL;
 import java.net.URLConnection;
 import java.util.ArrayList;
-import java.util.Collection;
 import java.util.HashSet;
 import java.util.Hashtable;
-import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.StringTokenizer;
-import java.util.concurrent.Callable;
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
-import java.util.concurrent.Future;
 
 public class TomEEJarScanner extends StandardJarScanner {
 
@@ -158,11 +151,6 @@ public class TomEEJarScanner extends Sta
                     @Override
                     public void run() {
                         for (URL current : SERVER_URLS) {
-                            if (current.toExternalForm().contains("myfaces-impl-")) {
-                                // done elsewhere
-                                continue;
-                            }
-
                             tldConfig(config, current);
                             if (tldLocationsCache != null) {
                                 tldLocationCache(locationsCacheInstance, current);
@@ -171,6 +159,7 @@ public class TomEEJarScanner extends Sta
                     }
                 };
                 SERVER_SCANNING_THREAD.setName("TomEE-server-tld-reading");
+                SERVER_SCANNING_THREAD.setDaemon(true);
                 SERVER_SCANNING_THREAD.start();
             } else {
                 SERVER_SCANNING_THREAD = null;
@@ -214,7 +203,9 @@ public class TomEEJarScanner extends Sta
                         config.addTaglibUri(uri);
                     }
                     for (String listener : LISTENERS) {
-                        config.addApplicationListener(listener);
+                        if (!"org.apache.myfaces.webapp.StartupServletContextListener".equals(listener)) { // done elsewhere
+                            config.addApplicationListener(listener);
+                        }
                     }
 
                     return; // done, next code is a fallback if scan() throw an exception

Modified: tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomcatHelper.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomcatHelper.java?rev=1439154&r1=1439153&r2=1439154&view=diff
==============================================================================
--- tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomcatHelper.java (original)
+++ tomee/tomee/trunk/tomee/tomee-loader/src/main/java/org/apache/tomee/loader/TomcatHelper.java Sun Jan 27 19:38:02 2013
@@ -161,7 +161,9 @@ public class TomcatHelper {
 
 	public static void configureJarScanner(Context standardContext) {
 		try {
-            standardContext.setJarScanner(new TomEEJarScanner());
+            if (!(standardContext.getJarScanner() instanceof TomEEJarScanner)) {
+                standardContext.setJarScanner(new TomEEJarScanner());
+            }
 		} catch (Exception e) {
 			// ignore
 		}