LGTM, with one whitespace nit.  Pity ant doesn't have a built-in
$ant.target property, though.


http://gwt-code-reviews.appspot.com/91810/diff/1/3
File user/build.xml (right):

http://gwt-code-reviews.appspot.com/91810/diff/1/3#newcode380
Line 380: test.cases="test.dev.htmlunit.tests" >
I don't think you want the 3rd line to indent deeper, do you?  Means the
4th and later lines run out of space. ;-)

http://gwt-code-reviews.appspot.com/91810

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to