ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Donald <dona...@apache.org>
Subject Re: cvs commit: jakarta-ant/src/main/org/apache/tools/ant Constants.java
Date Mon, 29 Jan 2001 16:03:41 GMT
At 03:42  29/1/01 -0000, bodewig@apache.org 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               |
*-----------------------------------------------------*


Mime
View raw message