I take that back. Welcome rinaldo to the OpenJDK commit list!
I will put you down as a reviewer. I didn't think you had the OpenJDK
ID yet.
-kto
On Feb 8, 2011, at 4:17 PM, Kelly O'Hair wrote:
I did not know that an ant 1.7.3 even existed.
Thanks for the review. But I still need a second reviewer.
-kto
On Feb 8, 2011, at 4:02 PM, Rinaldo DiGiorgio wrote:
Looks good to me. It also worked with 1.7.3
On Feb 8, 2011, at 7:00 PM, Kelly O'Hair wrote:
Need reviewer: ant 1.7.1 or newer needed
This changes the minimum required ant version to 1.7.1 and also
makes it a fatal sanity error
to try and use anything older.
The ant 1.7.1 release was done in July 2008 and is a well known
stable ant release.
7016976: Documentation for required ant version on JDK7 builds on
Solaris 10 and Solaris 11
http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-ant171/webrev/
-kto
P.S. Ant binary bundles can be found at
http://archive.apache.org/dist/ant/binaries/
