Github user ddekany commented on a diff in the pull request:

    https://github.com/apache/incubator-freemarker/pull/20#discussion_r111104418
  
    --- Diff: build.xml ---
    @@ -133,6 +133,20 @@
         <mkdir dir="build"/>
       </target>
     
    +  <property name="ivy.install.version" value="2.3.0" />
    +  <condition property="ivy.home" value="${env.IVY_HOME}">
    --- End diff --
    
    If we utilize the `IVY_HOME` then I think we run into two problems. One is 
that `download-ivy` will try to overwrite the jar in it. The other is that the 
jar in `IVY_HOME` is certainly not part of Ant's classpath, so it won't be able 
to use the ivy tasks. So I guess we just shouldn't use `IVY_HOME`.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to