Author: maartenc
Date: Wed Apr  4 14:13:09 2007
New Revision: 525605

URL: http://svn.apache.org/viewvc?view=rev&rev=525605
Log:
FIX: ivy-retrieve failure when explicit absolute 'ivy.dep.file' path specified 
(IVY-396)

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

Modified: incubator/ivy/core/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/incubator/ivy/core/trunk/CHANGES.txt?view=diff&rev=525605&r1=525604&r2=525605
==============================================================================
--- incubator/ivy/core/trunk/CHANGES.txt (original)
+++ incubator/ivy/core/trunk/CHANGES.txt Wed Apr  4 14:13:09 2007
@@ -60,6 +60,7 @@
 - IMPROVE: New "modules in use" section in console report at the end of 
resolve (IVY-373) (thanks to John Wiliams)
 - IMPROVE: Generated XML reports now contains more information about the 
resolved module (IVY-446)
 
+- FIX: ivy-retrieve failure when explicit absolute 'ivy.dep.file' path 
specified (IVY-396)
 - FIX: IvyPostResolve Task doesn't reuse Ivy file of previous resolve (IVY-458)
 - FIX: Ivy standalone is passing null args to main method when invoking with 
no args (IVY-457)
 - FIX: Invalid error report with m2compatible resolver (IVY-456)

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=525605&r1=525604&r2=525605
==============================================================================
--- 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 Wed 
Apr  4 14:13:09 2007
@@ -171,7 +171,7 @@
                        throw new BuildException("'module' not allowed when not 
using 'org' attribute");
                }
                    if (_file == null) {
-                       _file = new File(getProject().getBaseDir(), 
getProperty(settings, "ivy.dep.file"));
+                       _file = getProject().resolveFile(getProperty(settings, 
"ivy.dep.file"));
                    }
                    report = ivy.resolve(
                            _file.toURL(), 

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=525605&r1=525604&r2=525605
==============================================================================
--- 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 
Wed Apr  4 14:13:09 2007
@@ -24,6 +24,7 @@
 import org.apache.ivy.Ivy;
 import org.apache.ivy.TestHelper;
 import org.apache.ivy.core.module.id.ModuleRevisionId;
+import org.apache.ivy.core.report.ResolveReport;
 import org.apache.tools.ant.BuildException;
 import org.apache.tools.ant.Project;
 import org.apache.tools.ant.taskdefs.Delete;
@@ -285,6 +286,23 @@
         
assertNotNull(project.getReference("ivy.resolved.descriptor.testWithResolveId"));
         assertNotNull(project.getReference("ivy.resolved.configurations.ref"));
         
assertNotNull(project.getReference("ivy.resolved.configurations.ref.testWithResolveId"));
+    }
+    
+    public void testResolveWithAbsoluteFile() {
+       // IVY-396
+       File ivyFile = new File("test/java/org/apache/ivy/ant/ivy-simple.xml");
+       _resolve.getProject().setProperty("ivy.dep.file", 
ivyFile.getAbsolutePath());
+       _resolve.execute();
+        
+        
assertTrue(getResolvedIvyFileInCache(ModuleRevisionId.newInstance("apache", 
"resolve-simple", "1.0")).exists());
+    }
+    
+    public void testResolveWithRelativeFile() {
+       // IVY-396
+       _resolve.getProject().setProperty("ivy.dep.file", 
"test/java/org/apache/ivy/ant/ivy-simple.xml");
+       _resolve.execute();
+        
+        
assertTrue(getResolvedIvyFileInCache(ModuleRevisionId.newInstance("apache", 
"resolve-simple", "1.0")).exists());
     }
     
     private Ivy getIvy() {


Reply via email to