Fix review comments

apache · Apr 13, 2024 · 5332031 · 5332031
1 parent 658773e
commit 5332031
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 23 deletions.
diff --git a/container/openejb-loader/src/main/java/org/apache/openejb/loader/Embedder.java b/container/openejb-loader/src/main/java/org/apache/openejb/loader/Embedder.java
@@ -105,7 +105,6 @@ private Class forcefulLoad(final ClassPath classPath, final ClassLoader classLoa
             }
             classPath.addJarsToPath(libsDir);
         } catch (final Exception e2) {
-            e2.printStackTrace();
             throw new Exception("Could not load OpenEJB libraries. Exception: " + e2.getClass().getName() + " " + e2.getMessage());
         }
 

diff --git a/tomee/tomee-webaccess/pom.xml b/tomee/tomee-webaccess/pom.xml
@@ -25,36 +25,15 @@
   <properties>
     <tomee.build.name>${project.groupId}.tomee.webaccess</tomee.build.name>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+    <version.groovy>2.5.23</version.groovy>
   </properties>
   <profiles>
-    <profile>
-      <id>java11plus</id>
-      <activation>
-        <jdk>[11,)</jdk>
-      </activation>
-      <properties>
-        <version.groovy>2.5.23</version.groovy>
-      </properties>
-    </profile>
     <profile>
       <id>tomee</id>
       <activation>
         <activeByDefault>true</activeByDefault>
       </activation>
     </profile>
-    <profile>
-      <id>jdk15+</id>
-      <activation>
-        <jdk>[15,)</jdk>
-      </activation>
-      <dependencies>
-        <dependency>
-          <groupId>org.openjdk.nashorn</groupId>
-          <artifactId>nashorn-core</artifactId>
-          <scope>test</scope>
-        </dependency>
-      </dependencies>
-    </profile>
   </profiles>
   <dependencyManagement>
     <dependencies>
@@ -236,6 +215,11 @@
       <version>${version.groovy}</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.openjdk.nashorn</groupId>
+      <artifactId>nashorn-core</artifactId>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.apache.ivy</groupId>
       <artifactId>ivy</artifactId>