John Rose a écrit : > On Apr 24, 2008, at 1:23 PM, Charles Oliver Nutter wrote: >> John Rose wrote: >>> This is to fix the build bug observed by Selvan and Charlie. >>> It is a second-order patch (patch on a patch repository). Apply it >>> in davinci/patches/hotspot/. >>> May I have a reviewer, please? >> >> Checking it out now. Should this be built with or without testable >> (i.e. are you looking to get a testable patchset working as a result >> of this)? > > The default, buildable + testable. > >> I was able to build, but I can't seem to load >> java.dyn.AnonymousClassLoader in any code I'm running. That could be >> unrelated, perhaps some step I missed? > > Yes, it's unrelated. Check anonk.txt for an incremental javac command > plus bootclasspath stuff. That's what I've been using to start with, > just as a matter of taste. > > So, I'll count your build success as a review. (Unless you have more > review comments? :-) > > We'll tweak the JDK build shortly if necessary. (Remi, did you have > to take special action to get java.dyn.* into rt.jar?) no, i have just prepend the classes to the bootstrap classpath using: -Xbootclasspath/p:
> > -- John Rémi _______________________________________________ mlvm-dev mailing list [email protected] http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev
