Allon Mureinik has uploaded a new change for review.

Change subject: engine: Remove unused javassisst dependencies
......................................................................

engine: Remove unused javassisst dependencies

The engine and its test suite do not use Javassisst directly - its is
only used as part of the PowerMock mocking (and is required directly due
to a bug in PowerMock's pom).

Most of PowerMock's usages were removed from the engine a long time ago,
with the exception being the jaxrs module.
This patch removes the leftover javassisst dependencies from all the
other modules that no longer require it.

Change-Id: I360789865b8de899d1a04b0f8a974e95eecc65ca
Signed-off-by: Allon Mureinik <[email protected]>
---
M backend/manager/modules/bll/pom.xml
M backend/manager/modules/dal/pom.xml
M backend/manager/modules/restapi/types/pom.xml
3 files changed, 2 insertions(+), 19 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/65/30865/1

diff --git a/backend/manager/modules/bll/pom.xml 
b/backend/manager/modules/bll/pom.xml
index 58b523e..f0b929a 100644
--- a/backend/manager/modules/bll/pom.xml
+++ b/backend/manager/modules/bll/pom.xml
@@ -128,13 +128,7 @@
       <artifactId>mail</artifactId>
     </dependency>
 
-    <dependency>
-      <groupId>javassist</groupId>
-      <artifactId>javassist</artifactId>
-      <scope>test</scope>
-    </dependency>
-
-    <dependency>
+   <dependency>
       <groupId>org.jboss.spec.javax.interceptor</groupId>
       <artifactId>jboss-interceptors-api_1.1_spec</artifactId>
     </dependency>
diff --git a/backend/manager/modules/dal/pom.xml 
b/backend/manager/modules/dal/pom.xml
index cbbd13b..9b5cd5b 100644
--- a/backend/manager/modules/dal/pom.xml
+++ b/backend/manager/modules/dal/pom.xml
@@ -70,11 +70,6 @@
       <groupId>javax.transaction</groupId>
       <artifactId>jta</artifactId>
     </dependency>
-    <dependency>
-      <groupId>javassist</groupId>
-      <artifactId>javassist</artifactId>
-      <scope>test</scope>
-    </dependency>
   </dependencies>
   <build>
     <filters>
diff --git a/backend/manager/modules/restapi/types/pom.xml 
b/backend/manager/modules/restapi/types/pom.xml
index 8594e2b..1968a0b 100644
--- a/backend/manager/modules/restapi/types/pom.xml
+++ b/backend/manager/modules/restapi/types/pom.xml
@@ -31,13 +31,7 @@
       <scope>test</scope>
     </dependency>
 
-    <dependency>
-      <groupId>javassist</groupId>
-      <artifactId>javassist</artifactId>
-      <scope>test</scope>
-    </dependency>
-
-    <dependency>
+   <dependency>
       <groupId>org.ovirt.engine.core</groupId>
       <artifactId>common</artifactId>
       <version>${engine.version}</version>


-- 
To view, visit http://gerrit.ovirt.org/30865
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I360789865b8de899d1a04b0f8a974e95eecc65ca
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to