Now that the files are versioned, should ant clean do
 * 1. a "brute force" equivalent of rm clojure*.jar in %CLOJURE_INSTALL%
 * 2. or just try to delete clojure jars with names equal to the
current values in version.properties

I prefer 1., since with 2. if version numbers have changed in
version.properties, ant clean will not delete jars with old names and
it could be very confusing.

Rich, please find attached the patch (one line) for 1., if you agree
with Michael and with 1. option.

HTH,

-- 
Laurent

2009/4/28 Michael Wood <esiot...@gmail.com>:
>
> 2009/4/28 Laurent PETIT <laurent.pe...@gmail.com>:
>>
>> 2009/4/28 Marko Kocić <marko.ko...@gmail.com>:
>>>
>>> Shouldn't "ant clean" remove all generated files, including jars from
>>> the source tree?
>>
>> I also asked this to myself, but it was the previous behaviour and I
>> didn't want to do more than one thing in the patch.
>
> "ant clean" in the clojure-contrib build tree removes the jars.  I
> think it makes sense to do the same for clojure.
>
> --
> Michael Wood <esiot...@gmail.com>
>
> >
>

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To post to this group, send email to clojure@googlegroups.com
To unsubscribe from this group, send email to 
clojure+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/clojure?hl=en
-~----------~----~----~----~------~----~------~--~---

Index: build.xml
===================================================================
--- build.xml	(révision 1359)
+++ build.xml	(copie de travail)
@@ -144,6 +144,11 @@
   <target name="clean"
           description="Remove autogenerated files and directories.">
     <delete dir="${build}"/>
+    <delete>
+      <fileset dir=".">
+        <include name="clojure*.jar"/>
+      </fileset>
+    </delete>
     <delete file="pom.xml"/>
   </target>
 

Reply via email to