I would suggest to commit the change at the moment and make new patch for other improvement. (btw, I actually don't think the directory naming is confusing, since version number of GC is not necessary to be in the directory name. For example, Java1.5 doesn't have version name in java.exe. Well if we really want to have version number in directory name, we can do that later with a new patch. )
Thanks, xiaofeng On 10/8/06, Weldon Washburn <[EMAIL PROTECTED]> wrote:
Good work. This helps. Could we not change the naming for the collectors right now? Its likely to confuse folks. How about using gcv4_1 and gcv5. Also, at least for a while we need to be able to build and run gcv4. On 10/8/06, Xiao-Feng Li <[EMAIL PROTECTED]> wrote: > > I've submitted a patch to the JIRA to make DRLVM compile two GCs, one > under gc_cc/ directory (originally gc/ of gcv4.1), the other under > gc_gen/ for gcv5 subtree. The default collector is still gcv4.1 in > DRLVM. People can specify gcv5 with command line option > -Dvm.dlls=gc_gen.dll . > > In future we may put all GCs under subtrees of gc/ directory. > > Thanks, > xiaofeng > > On 10/6/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: > > Can you make it selectable for build? IOW > > > > sh build.sh -Dbuild.gc=5 > > > > or something... > > > > Weldon Washburn wrote: > > > Good progress. I will plug GCV5 in today or tomorrow and report what > runs. > > > Provided this code sits well w/ drlvm tree, I will go ahead an commit > to > > > vm/gcv5 > > > > > > On 10/6/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: > > >> > > >> > > >> > > >> Xiao-Feng Li wrote: > > >> > The submitted revision is downloadable in JIRA-1428 at: > > >> > > http://issues.apache.org/jira/secure/attachment/12342430/gcv5-r0.10.zip > > >> > > > >> > > >> Nice! w00t! > > >> > > >> > Attached in this email is the gc.xml file I am using that replaces > > >> > existing one for building gc. > > >> > > >> Please attach that to the JIRA as well. I can't wait to try this... > > >> > > >> Products Division > > > > > > > --------------------------------------------------------------------- > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > --------------------------------------------------------------------- > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Weldon Washburn Intel Middleware Products Division
--------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]