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 2016/02/10 13:55:07 UTC

tomee git commit: TOMEE-1708 use dir instead of arbitrary using target to download tomee

Repository: tomee
Updated Branches:
  refs/heads/master c4cccb5a6 -> c4bd8633d


TOMEE-1708 use dir instead of arbitrary using target to download tomee


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/c4bd8633
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/c4bd8633
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/c4bd8633

Branch: refs/heads/master
Commit: c4bd8633d96868be061d08390dd03b65cdc90cac
Parents: c4cccb5
Author: Romain manni-Bucau <rm...@gmail.com>
Authored: Wed Feb 10 13:54:50 2016 +0100
Committer: Romain manni-Bucau <rm...@gmail.com>
Committed: Wed Feb 10 13:54:50 2016 +0100

----------------------------------------------------------------------
 .../org/apache/openejb/arquillian/common/Setup.java   | 14 ++++++++------
 .../tomee/arquillian/remote/RemoteTomEEContainer.java |  2 +-
 .../tomee/arquillian/webapp/TomEEWebappContainer.java | 12 ++++++------
 3 files changed, 15 insertions(+), 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/c4bd8633/arquillian/arquillian-tomee-common/src/main/java/org/apache/openejb/arquillian/common/Setup.java
----------------------------------------------------------------------
diff --git a/arquillian/arquillian-tomee-common/src/main/java/org/apache/openejb/arquillian/common/Setup.java b/arquillian/arquillian-tomee-common/src/main/java/org/apache/openejb/arquillian/common/Setup.java
index 1d67531..eab496c 100644
--- a/arquillian/arquillian-tomee-common/src/main/java/org/apache/openejb/arquillian/common/Setup.java
+++ b/arquillian/arquillian-tomee-common/src/main/java/org/apache/openejb/arquillian/common/Setup.java
@@ -145,19 +145,21 @@ public class Setup {
         return null;
     }
 
-    public static File downloadAndUnpack(final File dir, final String artifactID) throws LifecycleException {
+    public static File downloadAndUnpack(final File dir, final String artifactID, final String defaultTempDir) throws LifecycleException {
 
-        final File zipFile = downloadFile(artifactID, null);
+        final File zipFile = downloadFile(artifactID, null, defaultTempDir);
 
         Zips.unzip(zipFile, dir);
 
         return findHome(dir);
     }
 
-    public static File downloadFile(final String artifactName, final String altUrl) {
-        final String cache = SystemInstance.get().getOptions().get(ProvisioningResolver.OPENEJB_DEPLOYER_CACHE_FOLDER, (String) null);
-        if (cache == null) { // let the user override it
-            System.setProperty(ProvisioningResolver.OPENEJB_DEPLOYER_CACHE_FOLDER, "target");
+    public static File downloadFile(final String artifactName, final String altUrl, final String defaultTempDir) {
+        final String cache = SystemInstance.isInitialized() ?
+            SystemInstance.get().getOptions().get(ProvisioningResolver.OPENEJB_DEPLOYER_CACHE_FOLDER, (String) null) :
+            System.getProperty(ProvisioningResolver.OPENEJB_DEPLOYER_CACHE_FOLDER);
+        if (cache == null && defaultTempDir != null) { // let the user override it
+            System.setProperty(ProvisioningResolver.OPENEJB_DEPLOYER_CACHE_FOLDER, defaultTempDir);
         }
 
         try {

http://git-wip-us.apache.org/repos/asf/tomee/blob/c4bd8633/arquillian/arquillian-tomee-remote/src/main/java/org/apache/tomee/arquillian/remote/RemoteTomEEContainer.java
----------------------------------------------------------------------
diff --git a/arquillian/arquillian-tomee-remote/src/main/java/org/apache/tomee/arquillian/remote/RemoteTomEEContainer.java b/arquillian/arquillian-tomee-remote/src/main/java/org/apache/tomee/arquillian/remote/RemoteTomEEContainer.java
index 1593db2..efdc76e 100644
--- a/arquillian/arquillian-tomee-remote/src/main/java/org/apache/tomee/arquillian/remote/RemoteTomEEContainer.java
+++ b/arquillian/arquillian-tomee-remote/src/main/java/org/apache/tomee/arquillian/remote/RemoteTomEEContainer.java
@@ -215,7 +215,7 @@ public class RemoteTomEEContainer extends TomEEContainer<RemoteTomEEConfiguratio
         tomeeHome = Setup.findHome(workingDirectory);
 
         if (tomeeHome == null) {
-            tomeeHome = Setup.downloadAndUnpack(workingDirectory, configuration.getArtifactName());
+            tomeeHome = Setup.downloadAndUnpack(workingDirectory, configuration.getArtifactName(), configuration.getDir());
 
             logger.log(Level.INFO, "Downloaded container to: " + tomeeHome);
         }

http://git-wip-us.apache.org/repos/asf/tomee/blob/c4bd8633/arquillian/arquillian-tomee-webapp-remote/src/main/java/org/apache/tomee/arquillian/webapp/TomEEWebappContainer.java
----------------------------------------------------------------------
diff --git a/arquillian/arquillian-tomee-webapp-remote/src/main/java/org/apache/tomee/arquillian/webapp/TomEEWebappContainer.java b/arquillian/arquillian-tomee-webapp-remote/src/main/java/org/apache/tomee/arquillian/webapp/TomEEWebappContainer.java
index 152ce0d..53d139a 100644
--- a/arquillian/arquillian-tomee-webapp-remote/src/main/java/org/apache/tomee/arquillian/webapp/TomEEWebappContainer.java
+++ b/arquillian/arquillian-tomee-webapp-remote/src/main/java/org/apache/tomee/arquillian/webapp/TomEEWebappContainer.java
@@ -83,7 +83,7 @@ public class TomEEWebappContainer extends TomEEContainer<TomEEWebappConfiguratio
 
             if (openejbHome == null) {
 
-                downloadTomcat(workingDirectory, configuration.getTomcatVersion());
+                downloadTomcat(workingDirectory, configuration.getTomcatVersion(), configuration.getDir());
 
                 openejbHome = Setup.findHome(workingDirectory);
 
@@ -92,7 +92,7 @@ public class TomEEWebappContainer extends TomEEContainer<TomEEWebappConfiguratio
                 final File webapp = new File(openejbHome, "webapps" + s + "tomee");
 
                 Files.mkdir(webapp);
-                downloadOpenEJBWebapp(webapp);
+                downloadOpenEJBWebapp(webapp, configuration.getDir());
 
                 System.setProperty("catalina.home", openejbHome.getAbsolutePath());
                 System.setProperty("catalina.base", openejbHome.getAbsolutePath());
@@ -202,13 +202,13 @@ public class TomEEWebappContainer extends TomEEContainer<TomEEWebappConfiguratio
         }
     }
 
-    protected void downloadOpenEJBWebapp(final File targetDirectory) throws LifecycleException {
+    protected void downloadOpenEJBWebapp(final File targetDirectory, final String defaultTempDir) throws LifecycleException {
         final String artifactName = configuration.getArtifactName();
-        final File zipFile = Setup.downloadFile(artifactName, null);
+        final File zipFile = Setup.downloadFile(artifactName, null, defaultTempDir);
         Zips.unzip(zipFile, targetDirectory);
     }
 
-    protected void downloadTomcat(final File catalinaDirectory, final String tomcatVersion) throws LifecycleException {
+    protected void downloadTomcat(final File catalinaDirectory, final String tomcatVersion, final String defaultTempDir) throws LifecycleException {
         String source = null;
 
         try {
@@ -222,7 +222,7 @@ public class TomEEWebappContainer extends TomEEContainer<TomEEWebappConfiguratio
             throw new LifecycleException("Unable to find URL for Tomcat " + tomcatVersion);
         }
 
-        final File zipFile = Setup.downloadFile("org.apache.tomcat:tomcat:" + tomcatVersion + ":zip", source);
+        final File zipFile = Setup.downloadFile("org.apache.tomcat:tomcat:" + tomcatVersion + ":zip", source, defaultTempDir);
         Zips.unzip(zipFile, catalinaDirectory);
     }