Looks good to me. -kto
On Oct 29, 2012, at 8:37 AM, Anthony Petrov wrote: > Hello, > > Please review a fix for > http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8001764 at: > > http://cr.openjdk.java.net/~anthony/8-46-full-vs2012.0/ > > We're not yet switching JDK builds to VS2012. However, the vsvars.sh script > is useful for other applications in order to set up the build environment, so > I think the proposed changes should be useful. Once we start using VS2012 (or > VS2014, or whatever it will be) for JDK, the vsvars.sh will already support > that. > > -- > best regards, > Anthony