Rev the version of apache ant in third_party to 1.8.4. This will fix an issue with

Rev the version of apache ant in third_party to 1.8.4. This will fix an issue with
windows build failures not being reported correctly (older versions of ant didn't 
properly return process eit codes). It will also let us use some newer features of
ant.
Committed: https://code.google.com/p/dart/source/detail?r=17228

[devoncarew, 2013-01-16 19:02:20]

The new files in third_party/apache_ant/1.8.4 are not in this CL but will be
part of the commit (I didn't think the gcl upload could handle all the files).

Similarly, I'll delete third_party/apache_ant/v1_7_1 as part of the commit.

[ricow1, 2013-01-16 19:05:33]

Rubberstamp LGTM, thanks for letting me know. CC'ing Peter since there are gyp
changes in here

https://codereview.chromium.org/11973012/diff/1/compiler/dartium.gyp
File compiler/dartium.gyp (right):

https://codereview.chromium.org/11973012/diff/1/compiler/dartium.gyp#newcode2
compiler/dartium.gyp:2: # for details. All rights reserved. Use of this source
code is governed by a
why is this file called dartium.gyp?

[devoncarew, 2013-01-16 19:09:16]

https://codereview.chromium.org/11973012/diff/1/compiler/dartium.gyp
File compiler/dartium.gyp (right):

https://codereview.chromium.org/11973012/diff/1/compiler/dartium.gyp#newcode2
compiler/dartium.gyp:2: # for details. All rights reserved. Use of this source
code is governed by a
On 2013/01/16 19:05:33, ricow1 wrote:
> why is this file called dartium.gyp?

I am wholly unfamiliar with this file; I'm guessing it is debris from past work
and is no longer used. It did have a reference to the old ant however, so I
updated that.

[danrubel, 2013-01-16 20:46:14]

lgtm