Looks fine. -kto
On Apr 22, 2012, at 8:51 AM, Alan Bateman wrote: > > I need a reviewer for a trivial change to get jdk8/tl building on Mac again. > Jim's changes for 7130404 [1] change the os.arch variable to "x86_64" to be > compatible with Apple's JDK but the change-set doesn't add a jvm.cfg to the > x86_64 directory. The issue is trivially fixed by moving the jvm,cfg to the > right location: > > $ hg diff -g > diff --git a/src/macosx/bin/amd64/jvm.cfg b/src/macosx/bin/x86_64/jvm.cfg > rename from src/macosx/bin/amd64/jvm.cfg > rename to src/macosx/bin/x86_64/jvm.cfg > > Note that jdk7u doesn't have this issue as the corresponding change-set added > a new jvm,cfg. > > Thanks, > Alan. > > [1] http://hg.openjdk.java.net/jdk8/tl/jdk/rev/77b35c5c4b95