At 03:42  29/1/01 -0000, [EMAIL PROTECTED] wrote:
>bodewig     01/01/29 07:42:24
>
>  Modified:    .        build.xml
>               src/main/org/apache/tools/ant Constants.java
>  Log:
>  (1) Fix <copy> task to point to the "correct" source directory.
>  
>  (2) I think there are a couple of superfluos @ characters in
Constants.java.
>  
>  Revision  Changes    Path
>  1.117     +11 -2     jakarta-ant/build.xml

>  +    <javac srcdir="${build.src}"
>  +           destdir="${build.classes}"
>  +           debug="${debug}"
>  +           deprecation="${deprecation}"
>  +           optimize="${optimize}" >
>  +      <classpath refid="classpath" />
>  +      <include name="${constants.file}" />
>  +    </javac>
>  +

I am not sure this is needed because the 
      <src path="${build.src}" />
in following javac will compile constants.

>  -    <copy file="${build.src}/${ant.package}/defaultManifest.mf"
>  +    <copy file="${java.dir}/${ant.package}/defaultManifest.mf"
>             todir="${build.classes}/${ant.package}" />

Also this will copy the defaultManifest.mf without the constants replaces.

Cheers,

Pete

*-----------------------------------------------------*
| "Faced with the choice between changing one's mind, |
| and proving that there is no need to do so - almost |
| everyone gets busy on the proof."                   |
|              - John Kenneth Galbraith               |
*-----------------------------------------------------*

Reply via email to