You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ivy-commits@incubator.apache.org by ma...@apache.org on 2007/03/08 22:39:05 UTC
svn commit: r516201 - in /incubator/ivy/core/trunk: CHANGES.txt
src/java/org/apache/ivy/ant/IvyResolve.java
test/java/org/apache/ivy/ant/IvyResolveTest.java
test/java/org/apache/ivy/ant/ivy-double.xml
Author: maartenc
Date: Thu Mar 8 14:39:03 2007
New Revision: 516201
URL: http://svn.apache.org/viewvc?view=rev&rev=516201
Log:
FIX: ivy.revision property not set correctly for second resolve (IVY-429)
Modified:
incubator/ivy/core/trunk/CHANGES.txt
incubator/ivy/core/trunk/src/java/org/apache/ivy/ant/IvyResolve.java
incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/IvyResolveTest.java
incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/ivy-double.xml
Modified: incubator/ivy/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/ivy/core/trunk/CHANGES.txt?view=diff&rev=516201&r1=516200&r2=516201
==============================================================================
--- incubator/ivy/core/trunk/CHANGES.txt (original)
+++ incubator/ivy/core/trunk/CHANGES.txt Thu Mar 8 14:39:03 2007
@@ -16,6 +16,7 @@
- IMPROVE: Add a unit test to verify that latest.integration accepts released modules (IVY-394) (thanks to Gilles Scokart)
- IMPROVE: New "modules in use" section in console report at the end of resolve (IVY-373) (thanks to John Wiliams)
+- FIX: ivy.revision property not set correctly for second resolve (IVY-429)
- FIX: NPE when no organisation or no name is provided in module element of ivyconf (IVY-422)
- FIX: FileUtil#copy(File src, File dest, CopyProgressListener l, boolean overwrite) (IVY-420)
- FIX: ${project.groupId} and ${project.version} not processed correctly in poms (IVY-425)
Modified: incubator/ivy/core/trunk/src/java/org/apache/ivy/ant/IvyResolve.java
URL: http://svn.apache.org/viewvc/incubator/ivy/core/trunk/src/java/org/apache/ivy/ant/IvyResolve.java?view=diff&rev=516201&r1=516200&r2=516201
==============================================================================
--- incubator/ivy/core/trunk/src/java/org/apache/ivy/ant/IvyResolve.java (original)
+++ incubator/ivy/core/trunk/src/java/org/apache/ivy/ant/IvyResolve.java Thu Mar 8 14:39:03 2007
@@ -173,7 +173,6 @@
if (_file == null) {
_file = new File(getProject().getBaseDir(), getProperty(settings, "ivy.dep.file"));
}
- _revision = getProperty(_revision, settings, "ivy.revision");
report = ivy.resolve(
_file.toURL(),
getResolveOptions(confs, settings));
Modified: incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/IvyResolveTest.java
URL: http://svn.apache.org/viewvc/incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/IvyResolveTest.java?view=diff&rev=516201&r1=516200&r2=516201
==============================================================================
--- incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/IvyResolveTest.java (original)
+++ incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/IvyResolveTest.java Thu Mar 8 14:39:03 2007
@@ -138,11 +138,13 @@
_resolve.execute();
assertEquals("resolve-simple", getIvy().getVariable("ivy.module"));
+ assertEquals("1.0", getIvy().getVariable("ivy.revision"));
_resolve.setFile(new File("test/java/org/apache/ivy/ant/ivy-double.xml"));
_resolve.execute();
assertEquals("resolve-double", getIvy().getVariable("ivy.module"));
+ assertEquals("1.1", getIvy().getVariable("ivy.revision"));
}
public void testFailure() throws Exception {
Modified: incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/ivy-double.xml
URL: http://svn.apache.org/viewvc/incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/ivy-double.xml?view=diff&rev=516201&r1=516200&r2=516201
==============================================================================
--- incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/ivy-double.xml (original)
+++ incubator/ivy/core/trunk/test/java/org/apache/ivy/ant/ivy-double.xml Thu Mar 8 14:39:03 2007
@@ -1,7 +1,7 @@
<ivy-module version="1.0">
<info organisation="apache"
module="resolve-double"
- revision="1.0"
+ revision="1.1"
status="release"
/>
<dependencies>