Reintroduce changes from r8440, with a fix to ZipFileResource and update related tests.
Review at http://gwt-code-reviews.appspot.com/728801
Review by: scottb@google.com
git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@8478 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/dev/core/src/com/google/gwt/dev/cfg/ModuleDef.java b/dev/core/src/com/google/gwt/dev/cfg/ModuleDef.java
index 4add18c..0f323d3 100644
--- a/dev/core/src/com/google/gwt/dev/cfg/ModuleDef.java
+++ b/dev/core/src/com/google/gwt/dev/cfg/ModuleDef.java
@@ -28,6 +28,7 @@
import com.google.gwt.dev.resource.impl.DefaultFilters;
import com.google.gwt.dev.resource.impl.PathPrefix;
import com.google.gwt.dev.resource.impl.PathPrefixSet;
+import com.google.gwt.dev.resource.impl.ResourceFilter;
import com.google.gwt.dev.resource.impl.ResourceOracleImpl;
import com.google.gwt.dev.util.Empty;
import com.google.gwt.dev.util.Util;
@@ -51,8 +52,13 @@
* Represents a module specification. In principle, this could be built without
* XML for unit tests.
*/
-@SuppressWarnings("deprecation")
-public class ModuleDef implements PublicOracle {
+public class ModuleDef {
+
+ private static final ResourceFilter NON_JAVA_RESOURCES = new ResourceFilter() {
+ public boolean allows(String path) {
+ return !path.endsWith(".java") && !path.endsWith(".class");
+ }
+ };
private static final Comparator<Map.Entry<String, ?>> REV_NAME_CMP = new Comparator<Map.Entry<String, ?>>() {
public int compare(Map.Entry<String, ?> entry1, Map.Entry<String, ?> entry2) {
@@ -115,6 +121,7 @@
private final Map<String, String> servletClassNamesByPath = new HashMap<String, String>();
private PathPrefixSet sourcePrefixSet = new PathPrefixSet();
+
private final Styles styles = new Styles();
public ModuleDef(String name) {
@@ -327,8 +334,8 @@
PathPrefixSet pathPrefixes = lazySourceOracle.getPathPrefixes();
PathPrefixSet newPathPrefixes = new PathPrefixSet();
for (PathPrefix pathPrefix : pathPrefixes.values()) {
- newPathPrefixes.add(new PathPrefix(pathPrefix.getPrefix(), null,
- pathPrefix.shouldReroot()));
+ newPathPrefixes.add(new PathPrefix(pathPrefix.getPrefix(),
+ NON_JAVA_RESOURCES, pathPrefix.shouldReroot()));
}
lazyResourcesOracle.setPathPrefixes(newPathPrefixes);
lazyResourcesOracle.refresh(TreeLogger.NULL);
diff --git a/dev/core/src/com/google/gwt/dev/cfg/PublicOracle.java b/dev/core/src/com/google/gwt/dev/cfg/PublicOracle.java
deleted file mode 100644
index fe4093a..0000000
--- a/dev/core/src/com/google/gwt/dev/cfg/PublicOracle.java
+++ /dev/null
@@ -1,42 +0,0 @@
-/*
- * Copyright 2008 Google Inc.
- *
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not
- * use this file except in compliance with the License. You may obtain a copy of
- * the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations under
- * the License.
- */
-package com.google.gwt.dev.cfg;
-
-import com.google.gwt.dev.resource.Resource;
-
-/**
- * Abstracts the process of querying for public files.
- *
- * @deprecated with no replacement, just use {@link ModuleDef} directly
- */
-@Deprecated
-public interface PublicOracle {
-
- /**
- * Finds a file on the public path.
- *
- * @param partialPath a file path relative to the root of any public package
- * @return the url of the file, or <code>null</code> if no such file exists
- */
- Resource findPublicFile(String partialPath);
-
- /**
- * Returns all available public files.
- *
- * @return an array containing the partial path to each available public file
- */
- String[] getAllPublicFiles();
-}
diff --git a/dev/core/src/com/google/gwt/dev/resource/ResourceOracle.java b/dev/core/src/com/google/gwt/dev/resource/ResourceOracle.java
index 0fe3eca..f23ea9c 100644
--- a/dev/core/src/com/google/gwt/dev/resource/ResourceOracle.java
+++ b/dev/core/src/com/google/gwt/dev/resource/ResourceOracle.java
@@ -27,37 +27,15 @@
* <code>com/google/gwt/blah.txt</code>.
*
* <p>
- * The identity of the returned sets and maps will change exactly when the
- * underlying module is refreshed.
- * </p>
- *
- * <p>
- * Even when the identity of a returned set changes, the identity of any
- * contained {@link Resource} values is guaranteed to differ from a previous
- * result exactly when that particular resource becomes invalid.
- * </p>
- *
- * <p>
- * A resource could become invalid for various reasons, including:
- * <ul>
- * <li>the underlying file was deleted or modified</li>
- * <li>another file with the same logical name superceded it on the classpath</li>
- * <li>the underlying module changed to exclude this file or supercede it with
- * another file</li>
- * </ul>
- * </p>
- *
- * <p>
- * After a refresh, a client can reliably detect changes by checking which of
- * its cached resource is still contained in the new result of
- * {@link #getResources()}.
+ * The identity of the returned sets and maps will change when the underlying
+ * module is refreshed.
* </p>
*/
public interface ResourceOracle {
/**
* Frees up all existing resources and transient internal state. All returned
- * collections will be empty after this call until this ResoruceOracle is
+ * collections will be empty after this call until this ResourceOracle is
* refreshed.
*/
void clear();
diff --git a/dev/core/src/com/google/gwt/dev/resource/impl/AbstractResource.java b/dev/core/src/com/google/gwt/dev/resource/impl/AbstractResource.java
index 99c9fd9..84b7155 100644
--- a/dev/core/src/com/google/gwt/dev/resource/impl/AbstractResource.java
+++ b/dev/core/src/com/google/gwt/dev/resource/impl/AbstractResource.java
@@ -27,7 +27,5 @@
* within this package.
*/
public abstract ClassPathEntry getClassPathEntry();
-
- public abstract boolean isStale();
}
diff --git a/dev/core/src/com/google/gwt/dev/resource/impl/DirectoryClassPathEntry.java b/dev/core/src/com/google/gwt/dev/resource/impl/DirectoryClassPathEntry.java
index 0eac9e1..0359a1b 100644
--- a/dev/core/src/com/google/gwt/dev/resource/impl/DirectoryClassPathEntry.java
+++ b/dev/core/src/com/google/gwt/dev/resource/impl/DirectoryClassPathEntry.java
@@ -23,7 +23,7 @@
import java.util.Map;
/**
- * TODO(bruce): write me.
+ * A {@link ClassPathEntry} for a directory on the file system.
*/
public class DirectoryClassPathEntry extends ClassPathEntry {
@@ -41,10 +41,20 @@
TreeLogger.DEBUG, "Including file: $0");
}
+ /**
+ * Absolute directory.
+ */
private final File dir;
+ private final String location;
+
+ /**
+ * @param dir an absolute directory
+ */
public DirectoryClassPathEntry(File dir) {
+ assert (dir.isAbsolute());
this.dir = dir;
+ this.location = dir.toURI().toString();
}
@Override
@@ -57,7 +67,7 @@
@Override
public String getLocation() {
- return dir.getAbsoluteFile().toURI().toString();
+ return location;
}
/**
diff --git a/dev/core/src/com/google/gwt/dev/resource/impl/FileResource.java b/dev/core/src/com/google/gwt/dev/resource/impl/FileResource.java
index ea1a669..317e89e 100644
--- a/dev/core/src/com/google/gwt/dev/resource/impl/FileResource.java
+++ b/dev/core/src/com/google/gwt/dev/resource/impl/FileResource.java
@@ -28,7 +28,6 @@
private final String abstractPathName;
private final DirectoryClassPathEntry classPathEntry;
private final File file;
- private final long modificationSeconds;
public FileResource(DirectoryClassPathEntry classPathEntry,
String abstractPathName, File file) {
@@ -36,7 +35,6 @@
this.classPathEntry = classPathEntry;
this.abstractPathName = abstractPathName;
this.file = file;
- this.modificationSeconds = lastModifiedSeconds(file);
}
@Override
@@ -51,7 +49,7 @@
@Override
public String getLocation() {
- return file.getAbsoluteFile().toURI().toString();
+ return file.toURI().toString();
}
@Override
@@ -60,21 +58,6 @@
}
@Override
- public boolean isStale() {
- if (!file.exists()) {
- // File was deleted. Always stale.
- return true;
- }
-
- long currentModificationSeconds = lastModifiedSeconds(file);
- /*
- * We use != instead of > because the point is to reflect what's actually on
- * the file system, not to worry about freshness per se.
- */
- return (currentModificationSeconds != modificationSeconds);
- }
-
- @Override
public InputStream openContents() {
try {
return new FileInputStream(file);
@@ -87,9 +70,4 @@
public boolean wasRerooted() {
return false;
}
-
- private long lastModifiedSeconds(File file) {
- return file.lastModified() / 1000;
- }
-
}
diff --git a/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java b/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java
index a38691c..18bdf48 100644
--- a/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java
+++ b/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java
@@ -41,8 +41,6 @@
import java.util.Map;
import java.util.Set;
import java.util.Map.Entry;
-import java.util.jar.JarFile;
-import java.util.zip.ZipFile;
/**
* The normal implementation of {@link ResourceOracle}.
@@ -105,11 +103,6 @@
}
@Override
- public boolean isStale() {
- return resource.isStale();
- }
-
- @Override
public InputStream openContents() {
return resource.openContents();
}
@@ -170,18 +163,14 @@
if (f.isDirectory()) {
return new DirectoryClassPathEntry(f);
} else if (f.isFile() && lowerCaseFileName.endsWith(".jar")) {
- return new ZipFileClassPathEntry(new JarFile(f));
+ return new ZipFileClassPathEntry(f);
} else if (f.isFile() && lowerCaseFileName.endsWith(".zip")) {
- return new ZipFileClassPathEntry(new ZipFile(f));
+ return new ZipFileClassPathEntry(f);
} else {
// It's a file ending in neither jar nor zip, speculatively try to
// open as jar/zip anyway.
try {
- return new ZipFileClassPathEntry(new JarFile(f));
- } catch (Exception ignored) {
- }
- try {
- return new ZipFileClassPathEntry(new ZipFile(f));
+ return new ZipFileClassPathEntry(f);
} catch (Exception ignored) {
}
logger.log(TreeLogger.TRACE, "Unexpected entry in classpath; " + f
@@ -258,8 +247,6 @@
private Set<Resource> exposedResources = Collections.emptySet();
- private Map<String, ResourceData> internalMap = Collections.emptyMap();
-
private PathPrefixSet pathPrefixSet = new PathPrefixSet();
/**
@@ -295,7 +282,6 @@
exposedPathNames = Collections.emptySet();
exposedResourceMap = Collections.emptyMap();
exposedResources = Collections.emptySet();
- internalMap = Collections.emptyMap();
}
public Set<String> getPathNames() {
@@ -363,39 +349,9 @@
}
}
- /*
- * Update the newInternalMap to preserve identity for any resources that
- * have not changed; also record whether or not there are ANY changes.
- *
- * There's definitely a change if the sizes don't match; even if the sizes
- * do match, every new resource must match an old resource for there to be
- * no changes.
- */
- boolean didChange = internalMap.size() != newInternalMap.size();
- for (Map.Entry<String, ResourceData> entry : newInternalMap.entrySet()) {
- String resourcePath = entry.getKey();
- ResourceData newData = entry.getValue();
- ResourceData oldData = internalMap.get(resourcePath);
- if (shouldUseNewResource(logger, oldData, newData)) {
- didChange = true;
- } else {
- if (oldData.resource != newData.resource) {
- newInternalMap.put(resourcePath, oldData);
- }
- }
- }
-
- if (!didChange) {
- // Nothing to do, keep the same identities.
- SpeedTracerLogger.end(CompilerEventType.RESOURCE_ORACLE);
- return;
- }
-
- internalMap = newInternalMap;
-
Map<String, Resource> externalMap = new HashMap<String, Resource>();
Set<Resource> externalSet = new HashSet<Resource>();
- for (Entry<String, ResourceData> entry : internalMap.entrySet()) {
+ for (Entry<String, ResourceData> entry : newInternalMap.entrySet()) {
String path = entry.getKey();
ResourceData data = entry.getValue();
externalMap.put(path, data.resource);
@@ -417,31 +373,4 @@
List<ClassPathEntry> getClassPath() {
return classPath;
}
-
- private boolean shouldUseNewResource(TreeLogger logger, ResourceData oldData,
- ResourceData newData) {
- AbstractResource newResource = newData.resource;
- String resourcePath = newResource.getPath();
- if (oldData != null) {
- // Test 1: Is the resource found in a different location than before?
- AbstractResource oldResource = oldData.resource;
- if (oldResource.getClassPathEntry() == newResource.getClassPathEntry()) {
- // Test 2: Has the resource changed since we last found it?
- if (!oldResource.isStale()) {
- // The resource has not changed.
- return false;
- } else {
- Messages.RESOURCE_BECAME_INVALID_BECAUSE_IT_IS_STALE.log(logger,
- resourcePath, null);
- }
- } else {
- Messages.RESOURCE_BECAME_INVALID_BECAUSE_IT_MOVED.log(logger,
- resourcePath, null);
- }
- } else {
- Messages.NEW_RESOURCE_FOUND.log(logger, resourcePath, null);
- }
-
- return true;
- }
}
diff --git a/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileClassPathEntry.java b/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileClassPathEntry.java
index c05b979..5194594 100644
--- a/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileClassPathEntry.java
+++ b/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileClassPathEntry.java
@@ -23,10 +23,12 @@
import com.google.gwt.dev.util.msg.Message1String;
import java.io.File;
+import java.io.IOException;
import java.util.Enumeration;
import java.util.Map;
import java.util.Set;
import java.util.zip.ZipEntry;
+import java.util.zip.ZipException;
import java.util.zip.ZipFile;
/**
@@ -55,8 +57,8 @@
}
private static class ZipFileSnapshot {
- private final int prefixSetSize;
private final Map<AbstractResource, PathPrefix> cachedAnswers;
+ private final int prefixSetSize;
ZipFileSnapshot(int prefixSetSize,
Map<AbstractResource, PathPrefix> cachedAnswers) {
@@ -72,11 +74,14 @@
*/
private final Map<PathPrefixSet, ZipFileSnapshot> cachedSnapshots = new IdentityHashMap<PathPrefixSet, ZipFileSnapshot>();
- private String cachedLocation;
+ private final String location;
+
private final ZipFile zipFile;
- public ZipFileClassPathEntry(ZipFile zipFile) {
- this.zipFile = zipFile;
+ public ZipFileClassPathEntry(File zipFile) throws ZipException, IOException {
+ assert zipFile.isAbsolute();
+ this.zipFile = new ZipFile(zipFile);
+ this.location = zipFile.toURI().toString();
}
/**
@@ -101,10 +106,7 @@
@Override
public String getLocation() {
- if (cachedLocation == null) {
- cachedLocation = new File(zipFile.getName()).toURI().toString();
- }
- return cachedLocation;
+ return location;
}
public ZipFile getZipFile() {
@@ -127,7 +129,7 @@
continue;
}
ZipFileResource zipResource = new ZipFileResource(this,
- zipEntry.getName());
+ zipEntry.getName(), zipEntry.getTime());
results.add(zipResource);
Messages.READ_ZIP_ENTRY.log(logger, zipEntry.getName(), null);
}
diff --git a/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileResource.java b/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileResource.java
index 602bd3a..e715bf6 100644
--- a/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileResource.java
+++ b/dev/core/src/com/google/gwt/dev/resource/impl/ZipFileResource.java
@@ -17,7 +17,6 @@
import java.io.IOException;
import java.io.InputStream;
-import java.util.jar.JarFile;
import java.util.zip.ZipEntry;
/**
@@ -27,10 +26,13 @@
private final ZipFileClassPathEntry classPathEntry;
private final String path;
+ private long lastModified;
- public ZipFileResource(ZipFileClassPathEntry classPathEntry, String path) {
+ public ZipFileResource(ZipFileClassPathEntry classPathEntry, String path,
+ long lastModified) {
this.classPathEntry = classPathEntry;
this.path = path;
+ this.lastModified = lastModified;
}
@Override
@@ -40,16 +42,12 @@
@Override
public long getLastModified() {
- return getEntry().getTime();
+ return lastModified;
}
@Override
public String getLocation() {
- // CHECKSTYLE_OFF
- String proto = classPathEntry.getZipFile() instanceof JarFile ? "jar:"
- : "zip:";
- // CHECKSTYLE_ON
- return proto + classPathEntry.getLocation() + "!/" + path;
+ return "jar:" + classPathEntry.getLocation() + "!/" + path;
}
@Override
@@ -57,19 +55,10 @@
return path;
}
- /**
- * Since we don't dynamically reload zips during a run, zip-based resources
- * cannot become stale.
- */
- @Override
- public boolean isStale() {
- return false;
- }
-
@Override
public InputStream openContents() {
try {
- return classPathEntry.getZipFile().getInputStream(getEntry());
+ return classPathEntry.getZipFile().getInputStream(new ZipEntry(path));
} catch (IOException e) {
// The spec for this method says it can return null.
return null;
@@ -80,8 +69,4 @@
public boolean wasRerooted() {
return false;
}
-
- private ZipEntry getEntry() {
- return classPathEntry.getZipFile().getEntry(path);
- }
}
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/AbstractResourceOrientedTestBase.java b/dev/core/test/com/google/gwt/dev/resource/impl/AbstractResourceOrientedTestBase.java
index 2b18497..3d20c7f 100644
--- a/dev/core/test/com/google/gwt/dev/resource/impl/AbstractResourceOrientedTestBase.java
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/AbstractResourceOrientedTestBase.java
@@ -28,7 +28,6 @@
import java.util.IdentityHashMap;
import java.util.Map;
import java.util.Set;
-import java.util.jar.JarFile;
/**
* Shared abstract class for tests that rely on well-known test data.
@@ -154,8 +153,8 @@
return file;
}
- protected File findJarFile(String name) throws URISyntaxException {
- URL url = findJarUrl(name);
+ protected File findFile(String name) throws URISyntaxException {
+ URL url = findUrl(name);
assertNotNull(
"Expecting on the classpath: "
+ name
@@ -170,7 +169,7 @@
* @param name
* @return
*/
- protected URL findJarUrl(String name) {
+ protected URL findUrl(String name) {
ClassLoader classLoader = getClass().getClassLoader();
return classLoader.getResource(name);
}
@@ -193,9 +192,13 @@
protected ClassPathEntry getClassPathEntry1AsJar() throws IOException,
URISyntaxException {
- File file = findJarFile("com/google/gwt/dev/resource/impl/testdata/cpe1.jar");
- return new ExcludeSvnClassPathEntry(new ZipFileClassPathEntry(new JarFile(
- file)));
+ File file = findFile("com/google/gwt/dev/resource/impl/testdata/cpe1.jar");
+ return new ExcludeSvnClassPathEntry(new ZipFileClassPathEntry(file));
+ }
+
+ protected ClassPathEntry getClassPathEntry1AsZip() throws IOException, URISyntaxException {
+ File file = findFile("com/google/gwt/dev/resource/impl/testdata/cpe1.zip");
+ return new ExcludeSvnClassPathEntry(new ZipFileClassPathEntry(file));
}
protected ClassPathEntry getClassPathEntry1AsMock() {
@@ -210,9 +213,13 @@
protected ClassPathEntry getClassPathEntry2AsJar() throws URISyntaxException,
IOException {
- File file = findJarFile("com/google/gwt/dev/resource/impl/testdata/cpe2.jar");
- return new ExcludeSvnClassPathEntry(new ZipFileClassPathEntry(new JarFile(
- file)));
+ File file = findFile("com/google/gwt/dev/resource/impl/testdata/cpe2.jar");
+ return new ExcludeSvnClassPathEntry(new ZipFileClassPathEntry(file));
+ }
+
+ protected ClassPathEntry getClassPathEntry2AsZip() throws URISyntaxException, IOException {
+ File file = findFile("com/google/gwt/dev/resource/impl/testdata/cpe2.zip");
+ return new ExcludeSvnClassPathEntry(new ZipFileClassPathEntry(file));
}
protected ClassPathEntry getClassPathEntry2AsMock() {
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/ClassPathEntryTest.java b/dev/core/test/com/google/gwt/dev/resource/impl/ClassPathEntryTest.java
index 5bfddad..2beae8b 100644
--- a/dev/core/test/com/google/gwt/dev/resource/impl/ClassPathEntryTest.java
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/ClassPathEntryTest.java
@@ -26,49 +26,80 @@
public void testAllCpe1FilesFound() throws URISyntaxException, IOException {
testAllCpe1FilesFound(getClassPathEntry1AsJar());
testAllCpe1FilesFound(getClassPathEntry1AsDirectory());
+ testAllCpe1FilesFound(getClassPathEntry1AsZip());
}
public void testAllCpe2FilesFound() throws URISyntaxException, IOException {
testAllCpe2FilesFound(getClassPathEntry2AsJar());
testAllCpe2FilesFound(getClassPathEntry2AsDirectory());
+ testAllCpe2FilesFound(getClassPathEntry2AsZip());
}
public void testPathPrefixSetChanges() throws IOException, URISyntaxException {
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
testPathPrefixSetChanges(cpe1jar, cpe2jar);
+ testPathPrefixSetChanges(cpe1dir, cpe2jar);
+ testPathPrefixSetChanges(cpe1zip, cpe2jar);
+
testPathPrefixSetChanges(cpe1dir, cpe2dir);
testPathPrefixSetChanges(cpe1jar, cpe2dir);
- testPathPrefixSetChanges(cpe1dir, cpe2jar);
+ testPathPrefixSetChanges(cpe1zip, cpe2dir);
+
+ testPathPrefixSetChanges(cpe1dir, cpe2zip);
+ testPathPrefixSetChanges(cpe1jar, cpe2zip);
+ testPathPrefixSetChanges(cpe1zip, cpe2zip);
}
public void testUseOfPrefixesWithFiltering() throws IOException,
URISyntaxException {
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
+ testUseOfPrefixesWithFiltering(cpe1dir, cpe2jar);
testUseOfPrefixesWithFiltering(cpe1jar, cpe2jar);
+ testUseOfPrefixesWithFiltering(cpe1dir, cpe2jar);
+ testUseOfPrefixesWithFiltering(cpe1zip, cpe2jar);
+
testUseOfPrefixesWithFiltering(cpe1dir, cpe2dir);
testUseOfPrefixesWithFiltering(cpe1jar, cpe2dir);
- testUseOfPrefixesWithFiltering(cpe1dir, cpe2jar);
+ testUseOfPrefixesWithFiltering(cpe1zip, cpe2dir);
+
+ testUseOfPrefixesWithFiltering(cpe1dir, cpe2zip);
+ testUseOfPrefixesWithFiltering(cpe1jar, cpe2zip);
+ testUseOfPrefixesWithFiltering(cpe1zip, cpe2zip);
}
public void testUseOfPrefixesWithoutFiltering() throws URISyntaxException,
IOException {
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
+ testUseOfPrefixesWithoutFiltering(cpe1dir, cpe2jar);
testUseOfPrefixesWithoutFiltering(cpe1jar, cpe2jar);
+ testUseOfPrefixesWithoutFiltering(cpe1dir, cpe2jar);
+ testUseOfPrefixesWithoutFiltering(cpe1zip, cpe2jar);
+
testUseOfPrefixesWithoutFiltering(cpe1dir, cpe2dir);
testUseOfPrefixesWithoutFiltering(cpe1jar, cpe2dir);
- testUseOfPrefixesWithoutFiltering(cpe1dir, cpe2jar);
+ testUseOfPrefixesWithoutFiltering(cpe1zip, cpe2dir);
+
+ testUseOfPrefixesWithoutFiltering(cpe1dir, cpe2zip);
+ testUseOfPrefixesWithoutFiltering(cpe1jar, cpe2zip);
+ testUseOfPrefixesWithoutFiltering(cpe1zip, cpe2zip);
}
public void testUseOfPrefixesWithoutFiltering(ClassPathEntry cpe1,
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/FileResourceTest.java b/dev/core/test/com/google/gwt/dev/resource/impl/FileResourceTest.java
index 43dfe33..673c65d 100644
--- a/dev/core/test/com/google/gwt/dev/resource/impl/FileResourceTest.java
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/FileResourceTest.java
@@ -24,6 +24,29 @@
public class FileResourceTest extends TestCase {
+ public void testBasic() {
+ File f = null;
+ try {
+ f = File.createTempFile("com.google.gwt.dev.javac.impl.FileResourceTest",
+ ".tmp");
+ f.deleteOnExit();
+ Util.writeStringAsFile(f, "contents 1");
+ } catch (IOException e) {
+ fail("Failed to create test file");
+ }
+
+ File dir = f.getParentFile();
+ DirectoryClassPathEntry cpe = new DirectoryClassPathEntry(dir);
+ FileResource r = new FileResource(cpe, f.getName(), f);
+ assertEquals(f.getAbsoluteFile().toURI().toString(), r.getLocation());
+
+ /*
+ * In this case, there's no subdirectory, so the path should match the
+ * simple filename.
+ */
+ assertEquals(f.getName(), r.getPath());
+ }
+
public void testDeletion() {
File f = null;
try {
@@ -46,51 +69,9 @@
*/
assertEquals(f.getName(), r.getPath());
- // Shouldn't be stale yet.
- assertFalse(r.isStale());
-
- /*
- * Touch the file at more than one second to ensure there's a noticeable
- * difference on every platform.
- */
- // Ignore failure of setLastModified
- f.setLastModified(f.lastModified() + 1500);
-
- // Should be stale now.
- assertTrue(r.isStale());
- }
-
- public void testModification() {
- File f = null;
- try {
- f = File.createTempFile("com.google.gwt.dev.javac.impl.FileResourceTest",
- ".tmp");
- f.deleteOnExit();
- Util.writeStringAsFile(f, "contents 1");
- } catch (IOException e) {
- fail("Failed to create test file");
- }
-
- File dir = f.getParentFile();
- DirectoryClassPathEntry cpe = new DirectoryClassPathEntry(dir);
- FileResource r = new FileResource(cpe, f.getName(), f);
- assertEquals(f.getAbsoluteFile().toURI().toString(), r.getLocation());
-
- /*
- * In this case, there's no subdirectory, so the path should match the
- * simple filename.
- */
- assertEquals(f.getName(), r.getPath());
-
- // Shouldn't be stale yet.
- assertFalse(r.isStale());
-
// Delete the file.
f.delete();
- // Should be stale now.
- assertTrue(r.isStale());
-
// Get can't contents anymore, either.
assertNull(r.openContents());
}
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/MockAbstractResource.java b/dev/core/test/com/google/gwt/dev/resource/impl/MockAbstractResource.java
index 775d555..ba2b8b2 100644
--- a/dev/core/test/com/google/gwt/dev/resource/impl/MockAbstractResource.java
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/MockAbstractResource.java
@@ -50,11 +50,6 @@
}
@Override
- public boolean isStale() {
- return isStale;
- }
-
- @Override
public InputStream openContents() {
Assert.fail("Not implemented");
return null;
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplRealClasspathTest.java b/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplRealClasspathTest.java
index 2d228bc..e322887 100644
--- a/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplRealClasspathTest.java
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplRealClasspathTest.java
@@ -20,6 +20,8 @@
import java.util.Map;
+import static com.google.gwt.dev.resource.impl.ResourceOracleImplTest.assertResourcesEqual;
+
/**
* Tests {@link ResourceOracleImpl} using the real class path.
*
@@ -56,6 +58,7 @@
});
private final TreeLogger logger = createTestTreeLogger();
+
private final ResourceOracleImpl resourceOracle = new ResourceOracleImpl(
logger);
@@ -80,12 +83,12 @@
// Plain refresh should have no effect.
resourceOracle.refresh(logger);
- assertSame(resourceMap, resourceOracle.getResourceMap());
+ assertResourcesEqual(resourceMap, resourceOracle.getResourceMap());
// Setting same path entries should have no effect.
resourceOracle.setPathPrefixes(pathPrefixSet);
resourceOracle.refresh(logger);
- assertSame(resourceMap, resourceOracle.getResourceMap());
+ assertResourcesEqual(resourceMap, resourceOracle.getResourceMap());
// Setting identical path entries should have no effect.
pathPrefixSet = new PathPrefixSet();
@@ -93,18 +96,16 @@
pathPrefixSet.add(THIS_CLASS_PREFIX);
resourceOracle.setPathPrefixes(pathPrefixSet);
resourceOracle.refresh(logger);
- assertSame(resourceMap, resourceOracle.getResourceMap());
+ assertResourcesEqual(resourceMap, resourceOracle.getResourceMap());
// Setting identical result should have no effect.
pathPrefixSet.add(JUNIT_PREFIX_DUP);
resourceOracle.refresh(logger);
- assertSame(resourceMap, resourceOracle.getResourceMap());
+ assertResourcesEqual(resourceMap, resourceOracle.getResourceMap());
// Actually change the working set.
pathPrefixSet.add(THIS_CLASS_PREFIX_PLUS);
resourceOracle.refresh(logger);
- Map<String, Resource> newResourceMap = resourceOracle.getResourceMap();
- assertNotSame(resourceMap, newResourceMap);
- assertEquals(3, newResourceMap.size());
+ assertEquals(3, resourceOracle.getResourceMap().size());
}
}
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplTest.java b/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplTest.java
index 788449a..3360424 100644
--- a/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplTest.java
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/ResourceOracleImplTest.java
@@ -17,9 +17,11 @@
import com.google.gwt.core.ext.TreeLogger;
import com.google.gwt.dev.resource.Resource;
+import com.google.gwt.dev.util.Util;
import com.google.gwt.util.tools.Utility;
import java.io.BufferedReader;
+import java.io.File;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
@@ -29,6 +31,8 @@
import java.net.URLClassLoader;
import java.util.ArrayList;
import java.util.Arrays;
+import java.util.Collection;
+import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;
@@ -117,9 +121,9 @@
}
public void assertSameCollections(ResourceOracleSnapshot other) {
- assertSame(resourceMap, other.resourceMap);
- assertSame(resources, other.resources);
- assertSame(pathNames, other.pathNames);
+ assertResourcesEqual(resourceMap, other.resourceMap);
+ assertResourcesEqual(resources, other.resources);
+ assertEquals(pathNames, other.pathNames);
}
public AbstractResource findResourceWithPath(String path) {
@@ -130,13 +134,47 @@
}
return null;
}
+
+ public void assertResourcesGetURL() {
+ for (Resource resource : resources) {
+ URL url = resource.getURL();
+ assertNotNull("Resource " + resource + " had a null getURL()", url);
+
+ InputStream is = resource.openContents();
+ assertNotNull(is);
+
+ String urlString = Util.readURLAsString(url);
+ String inputString = Util.readStreamAsString(is);
+ assertEquals(urlString, inputString);
+ assertNotNull(urlString);
+ assertNotNull(inputString);
+ }
+ }
+ }
+
+ public static void assertResourcesEqual(Collection<Resource> expected,
+ Collection<Resource> actual) {
+ Set<String> expectedLocs = new HashSet<String>();
+ for (Resource resource : expected) {
+ expectedLocs.add(resource.getLocation());
+ }
+ Set<String> actualLocs = new HashSet<String>();
+ for (Resource resource : actual) {
+ actualLocs.add(resource.getLocation());
+ }
+ assertEquals(expectedLocs, actualLocs);
+ }
+
+ public static void assertResourcesEqual(Map<String, Resource> expected,
+ Map<String, Resource> actual) {
+ assertResourcesEqual(expected.values(), actual.values());
}
public void testCachingOfJarResources() throws IOException,
URISyntaxException {
TreeLogger logger = createTestTreeLogger();
- ClassPathEntry cpe1jar = new ZipFileClassPathEntry(new JarFile(
- findJarFile("com/google/gwt/dev/resource/impl/testdata/cpe1.jar")));
+ File jarFile = findFile("com/google/gwt/dev/resource/impl/testdata/cpe1.jar");
+ ClassPathEntry cpe1jar = new ZipFileClassPathEntry(jarFile);
// test basic caching
PathPrefixSet pps1 = new PathPrefixSet();
@@ -162,6 +200,31 @@
assertSame(resourceMap2, cpe1jar.findApplicableResources(logger, pps1));
assertSame(resourceMap3, cpe1jar.findApplicableResources(logger, pps2));
}
+
+ public void testGetUrlOnResources() throws URISyntaxException, IOException {
+ ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
+ ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
+
+ ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
+ ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
+
+ testGetURLOnResourcesInCPE(cpe1jar);
+ testGetURLOnResourcesInCPE(cpe1dir);
+ testGetURLOnResourcesInCPE(cpe1zip);
+ testGetURLOnResourcesInCPE(cpe2jar);
+ testGetURLOnResourcesInCPE(cpe2dir);
+ testGetURLOnResourcesInCPE(cpe2zip);
+ }
+
+ private void testGetURLOnResourcesInCPE(ClassPathEntry cpe) {
+ TreeLogger logger = createTestTreeLogger();
+
+ ResourceOracleImpl oracle = createResourceOracle(cpe);
+ ResourceOracleSnapshot s = refreshAndSnapshot(logger, oracle);
+ s.assertResourcesGetURL();
+ }
/**
* Test that ResourceOracleImpl preserves the order in which the same logical
@@ -176,8 +239,8 @@
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
- ClassPathEntry[] cp12 = new ClassPathEntry[] {cpe1jar, cpe2jar};
- ClassPathEntry[] cp21 = new ClassPathEntry[] {cpe2jar, cpe1jar};
+ ClassPathEntry[] cp12 = new ClassPathEntry[]{cpe1jar, cpe2jar};
+ ClassPathEntry[] cp21 = new ClassPathEntry[]{cpe2jar, cpe1jar};
String resKeyNormal = "org/example/bar/client/BarClient2.txt";
String resKeyReroot = "/BarClient2.txt";
PathPrefix pathPrefixNormal = new PathPrefix("org/example/bar/client",
@@ -214,8 +277,8 @@
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
- ClassPathEntry[] cp12 = new ClassPathEntry[] {cpe1jar, cpe2jar};
- ClassPathEntry[] cp21 = new ClassPathEntry[] {cpe2jar, cpe1jar};
+ ClassPathEntry[] cp12 = new ClassPathEntry[]{cpe1jar, cpe2jar};
+ ClassPathEntry[] cp21 = new ClassPathEntry[]{cpe2jar, cpe1jar};
String keyReroot = "/BarClient1.txt";
@@ -242,15 +305,23 @@
public void testReadingResource() throws IOException, URISyntaxException {
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
testReadingResource(cpe1jar, cpe2jar);
testReadingResource(cpe1dir, cpe2jar);
-
+ testReadingResource(cpe1zip, cpe2jar);
+
testReadingResource(cpe1jar, cpe2dir);
testReadingResource(cpe1dir, cpe2dir);
+ testReadingResource(cpe1zip, cpe2dir);
+
+ testReadingResource(cpe1jar, cpe2zip);
+ testReadingResource(cpe1dir, cpe2zip);
+ testReadingResource(cpe1zip, cpe2zip);
}
/**
@@ -262,15 +333,15 @@
*/
public void testRemoveDuplicates() throws MalformedURLException {
TreeLogger logger = createTestTreeLogger();
- URL cpe1 = findJarUrl("com/google/gwt/dev/resource/impl/testdata/cpe1.jar");
- URL cpe2 = findJarUrl("com/google/gwt/dev/resource/impl/testdata/cpe2.jar");
- URLClassLoader classLoader = new URLClassLoader(new URL[] {
+ URL cpe1 = findUrl("com/google/gwt/dev/resource/impl/testdata/cpe1.jar");
+ URL cpe2 = findUrl("com/google/gwt/dev/resource/impl/testdata/cpe2.zip");
+ URLClassLoader classLoader = new URLClassLoader(new URL[]{
cpe1, cpe2, cpe2, cpe1, cpe2,}, null);
ResourceOracleImpl oracle = new ResourceOracleImpl(logger, classLoader);
List<ClassPathEntry> classPath = oracle.getClassPath();
assertEquals(2, classPath.size());
assertJarEntry(classPath.get(0), "cpe1.jar");
- assertJarEntry(classPath.get(1), "cpe2.jar");
+ assertJarEntry(classPath.get(1), "cpe2.zip");
oracle = new ResourceOracleImpl(logger, classLoader);
List<ClassPathEntry> classPath2 = oracle.getClassPath();
assertEquals(2, classPath2.size());
@@ -283,66 +354,96 @@
ClassPathEntry cpe1mock = getClassPathEntry1AsMock();
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
ClassPathEntry cpe2mock = getClassPathEntry2AsMock();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
testResourceAddition(cpe1jar, cpe2jar);
testResourceAddition(cpe1dir, cpe2jar);
testResourceAddition(cpe1mock, cpe2jar);
+ testResourceAddition(cpe1zip, cpe2jar);
testResourceAddition(cpe1jar, cpe2dir);
testResourceAddition(cpe1dir, cpe2dir);
testResourceAddition(cpe1mock, cpe2dir);
+ testResourceAddition(cpe1zip, cpe2dir);
testResourceAddition(cpe1jar, cpe2mock);
testResourceAddition(cpe1dir, cpe2mock);
testResourceAddition(cpe1mock, cpe2mock);
+ testResourceAddition(cpe1zip, cpe2mock);
+
+ testResourceAddition(cpe1jar, cpe2zip);
+ testResourceAddition(cpe1dir, cpe2zip);
+ testResourceAddition(cpe1mock, cpe2zip);
+ testResourceAddition(cpe1zip, cpe2zip);
}
public void testResourceDeletion() throws IOException, URISyntaxException {
ClassPathEntry cpe1mock = getClassPathEntry1AsMock();
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
ClassPathEntry cpe2mock = getClassPathEntry2AsMock();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
testResourceDeletion(cpe1jar, cpe2jar);
testResourceDeletion(cpe1dir, cpe2jar);
testResourceDeletion(cpe1mock, cpe2jar);
+ testResourceDeletion(cpe1zip, cpe2jar);
testResourceDeletion(cpe1jar, cpe2dir);
testResourceDeletion(cpe1dir, cpe2dir);
testResourceDeletion(cpe1mock, cpe2dir);
+ testResourceDeletion(cpe1zip, cpe2dir);
testResourceDeletion(cpe1jar, cpe2mock);
testResourceDeletion(cpe1dir, cpe2mock);
testResourceDeletion(cpe1mock, cpe2mock);
+ testResourceDeletion(cpe1zip, cpe2mock);
+
+ testResourceDeletion(cpe1jar, cpe2zip);
+ testResourceDeletion(cpe1dir, cpe2zip);
+ testResourceDeletion(cpe1mock, cpe2zip);
+ testResourceDeletion(cpe1zip, cpe2zip);
}
public void testResourceModification() throws IOException, URISyntaxException {
ClassPathEntry cpe1mock = getClassPathEntry1AsMock();
ClassPathEntry cpe1jar = getClassPathEntry1AsJar();
ClassPathEntry cpe1dir = getClassPathEntry1AsDirectory();
+ ClassPathEntry cpe1zip = getClassPathEntry1AsZip();
ClassPathEntry cpe2mock = getClassPathEntry2AsMock();
ClassPathEntry cpe2jar = getClassPathEntry2AsJar();
ClassPathEntry cpe2dir = getClassPathEntry2AsDirectory();
+ ClassPathEntry cpe2zip = getClassPathEntry2AsZip();
testResourceModification(cpe1jar, cpe2jar);
testResourceModification(cpe1dir, cpe2jar);
testResourceModification(cpe1mock, cpe2jar);
+ testResourceModification(cpe1zip, cpe2jar);
testResourceModification(cpe1jar, cpe2dir);
testResourceModification(cpe1dir, cpe2dir);
testResourceModification(cpe1mock, cpe2dir);
+ testResourceModification(cpe1zip, cpe2dir);
testResourceModification(cpe1jar, cpe2mock);
testResourceModification(cpe1dir, cpe2mock);
testResourceModification(cpe1mock, cpe2mock);
+ testResourceModification(cpe1zip, cpe2mock);
+
+ testResourceModification(cpe1jar, cpe2zip);
+ testResourceModification(cpe1dir, cpe2zip);
+ testResourceModification(cpe1mock, cpe2zip);
+ testResourceModification(cpe1zip, cpe2zip);
/*
* TODO(bruce): figure out a good way to test real resource modifications of
@@ -361,7 +462,7 @@
PathPrefix pp1 = new PathPrefix("org/example/bar/client", null, false);
PathPrefix pp2 = new PathPrefix("org/example/bar", null, false);
testResourceInCPE(logger, "org/example/bar/client/BarClient2.txt", cpe1,
- new ClassPathEntry[] {cpe1, cpe2}, pp1, pp2);
+ new ClassPathEntry[]{cpe1, cpe2}, pp1, pp2);
}
/**
@@ -382,15 +483,15 @@
// Ensure the translatable overrides the basic despite swapping CPE order.
testResourceInCPE(logger, "java/lang/Object.java", cpe2,
- new ClassPathEntry[] {cpe1, cpe2}, pp1, pp2);
+ new ClassPathEntry[]{cpe1, cpe2}, pp1, pp2);
testResourceInCPE(logger, "java/lang/Object.java", cpe2,
- new ClassPathEntry[] {cpe2, cpe1}, pp1, pp2);
+ new ClassPathEntry[]{cpe2, cpe1}, pp1, pp2);
// Ensure the translatable overrides the basic despite swapping PPS order.
testResourceInCPE(logger, "java/lang/Object.java", cpe2,
- new ClassPathEntry[] {cpe1, cpe2}, pp2, pp1);
+ new ClassPathEntry[]{cpe1, cpe2}, pp2, pp1);
testResourceInCPE(logger, "java/lang/Object.java", cpe2,
- new ClassPathEntry[] {cpe2, cpe1}, pp2, pp1);
+ new ClassPathEntry[]{cpe2, cpe1}, pp2, pp1);
}
/**
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/testdata/cpe1.zip b/dev/core/test/com/google/gwt/dev/resource/impl/testdata/cpe1.zip
new file mode 100644
index 0000000..5650918
--- /dev/null
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/testdata/cpe1.zip
Binary files differ
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/testdata/cpe2.zip b/dev/core/test/com/google/gwt/dev/resource/impl/testdata/cpe2.zip
new file mode 100644
index 0000000..5cd2d5f
--- /dev/null
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/testdata/cpe2.zip
Binary files differ
diff --git a/dev/core/test/com/google/gwt/dev/resource/impl/testdata/rebuild_jars b/dev/core/test/com/google/gwt/dev/resource/impl/testdata/rebuild_jars
index 630dc1a..3c3ccca 100644
--- a/dev/core/test/com/google/gwt/dev/resource/impl/testdata/rebuild_jars
+++ b/dev/core/test/com/google/gwt/dev/resource/impl/testdata/rebuild_jars
@@ -1,7 +1,9 @@
cd cpe1
find . -type f | grep -v "\.svn" | xargs jar cvf ../cpe1.jar
+find . -type f | grep -v "\.svn" | xargs zip ../cpe1.zip
cd ..
cd cpe2
find . -type f | grep -v "\.svn" | xargs jar cvf ../cpe2.jar
+find . -type f | grep -v "\.svn" | xargs zip ../cpe2.zip
cd ..
