John Rose wrote: > 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.
Ahh, that's the step I was missing. Thanks. It loads and constructs now, so I can continue :) > So, I'll count your build success as a review. (Unless you have more > review comments? :-) None at all! > We'll tweak the JDK build shortly if necessary. (Remi, did you have to > take special action to get java.dyn.* into rt.jar?) > > -- John > _______________________________________________ mlvm-dev mailing list [email protected] http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev
