On 23 June 2006 at 23:43, "Alexey Petrenko" <[EMAIL PROTECTED]> wrote: > Mark, > > thanks for the fast fix. > > But probably Tim was right and we should not fix properties file but > should fix drlvm to work inside classlib deploy directory?
Indeed. See my comment in the JIRA HARMONY-651. I would have used the version from classlib, but there is an outstanding issue with serializer.jar - which drlvm current adds to jre/lib/boot but classlib does not. As I suggested in an earlier message (subject Re: [drlvm] Doing the minimum to support Java 5 classfiles) I think drlvm's deploy tree shouldn't contain any classlib files but should produce a deploy tree that a top-level build can combine with a classlib tree to produce a fully working deploy tree. Much like we do manually when we combine the IBM VME and classlib. That is, we should do more to reduce the coupling between drlvm and classlib. At the moment, drlvm is trying to understand classlib - i.e. knowing what to copy across but it is not going to be maintainable. (And arguably it is already failing because it is not copying the jre/ lib/ext/bcprov.jar that classlib uses - instead drlvm puts an older version in jre/lib/boot. This is just confusing we should stop trying to do it in drlvm and delegate this task to the top-level build.) > Gregory? I'd like to know what Gregory thinks about this too. Regards, Mark. > 2006/6/23, Mark Hindess <[EMAIL PROTECTED]>: > > > > On 23 June 2006 at 22:37, Gregory Shimansky <[EMAIL PROTECTED]> wrote: > > > Hello Mark > > > > > > While resolving conflict between your and mine changes to this file > > > I've found that you didn't include awt.jar. Should it be added > > > perhaps? > > > > Yes. Fixed in r416804 (and I've fixed the reason why it was missing > > from my build). > > > > -Mark. > > > > > On Friday 23 June 2006 19:22 [EMAIL PROTECTED] wrote: > > > > Author: hindessm > > > > Date: Fri Jun 23 08:22:20 2006 > > > > New Revision: 416738 > > > > > > > > URL: http://svn.apache.org/viewvc?rev=416738&view=rev > > > > Log: > > > > Fixed to point to all jars (or at least a version of all jars). > > > > > > > > Modified: > > > > > > > > incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/init/bootclasspath > .pro > > > >perties > > > > > > > > Modified: > > > > incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/init/bootclasspath > .pro > > > >perties URL: > > > > http://svn.apache.org/viewvc/incubator/harmony/enhanced/drlvm/trunk/vm/ > vmco > > > >re/src/init/bootclasspath.properties?rev=416738&r1=416737&r2=416738&view > =dif > > > >f > > > > ======================================================================= > ==== > > > >=== --- > > > > incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/init/bootclasspath > .pro > > > >perties (original) +++ > > > > incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/init/bootclasspath > .pro > > > >perties Fri Jun 23 08:22:20 2006 @@ -38,46 +38,68 @@ > > > > # unspecified then the source is deemed to be available rooted at '/'. > > > > # Packageroots given with no source location are ignored. > > > > > > > > -bootclasspath.1=archive.jar > > > > +bootclasspath.1=accessibility.jar > > > > > > > > -bootclasspath.2=bcprov-jdk14-129.jar > > > > +bootclasspath.2=annotation.jar > > > > > > > > -bootclasspath.3=beans.jar > > > > +bootclasspath.3=applet.jar > > > > > > > > -bootclasspath.4=crypto.jar > > > > +bootclasspath.4=archive.jar > > > > > > > > -bootclasspath.5=icu4j_3_4.jar > > > > +bootclasspath.5=auth.jar > > > > > > > > -bootclasspath.6=icu4jni-3.4.jar > > > > +bootclasspath.6=bcprov-jdk14-129.jar > > > > > > > > -bootclasspath.7=kernel.jar > > > > +bootclasspath.7=beans.jar > > > > > > > > -bootclasspath.8=logging.jar > > > > +bootclasspath.8=crypto.jar > > > > > > > > -bootclasspath.9=luni.jar > > > > +bootclasspath.9=icu4j_3.4.4/icu4j_3_4_4.jar > > > > > > > > -bootclasspath.10=math.jar > > > > +bootclasspath.10=icu4jni-3.4.jar > > > > > > > > -bootclasspath.11=nio.jar > > > > +bootclasspath.11=jndi.jar > > > > > > > > -bootclasspath.12=nio_char.jar > > > > +bootclasspath.12=kernel.jar > > > > > > > > -bootclasspath.13=regex.jar > > > > +bootclasspath.13=logging.jar > > > > > > > > -bootclasspath.14=resolver.jar > > > > +bootclasspath.14=luni.jar > > > > > > > > -bootclasspath.15=security.jar > > > > +bootclasspath.15=math.jar > > > > > > > > -bootclasspath.16=serializer.jar > > > > +bootclasspath.16=misc.jar > > > > > > > > -bootclasspath.17=sql.jar > > > > +bootclasspath.17=mx4j_3.0.1/mx4j-remote.jar > > > > > > > > -bootclasspath.18=text.jar > > > > +bootclasspath.18=mx4j_3.0.1/mx4j.jar > > > > > > > > -bootclasspath.19=x_net.jar > > > > +bootclasspath.19=nio_char.jar > > > > > > > > -bootclasspath.20=xalan.jar > > > > +bootclasspath.20=nio.jar > > > > > > > > -bootclasspath.21=xercesImpl.jar > > > > +bootclasspath.21=prefs.jar > > > > > > > > -bootclasspath.22=xml-apis.jar > > > > +bootclasspath.22=regex.jar > > > > + > > > > +bootclasspath.23=resolver.jar > > > > + > > > > +bootclasspath.24=rmi.jar > > > > + > > > > +bootclasspath.25=security.jar > > > > + > > > > +bootclasspath.26=serializer.jar > > > > + > > > > +bootclasspath.27=sql.jar > > > > + > > > > +bootclasspath.28=swing.jar > > > > + > > > > +bootclasspath.29=text.jar > > > > + > > > > +bootclasspath.30=xalan.jar > > > > + > > > > +bootclasspath.31=xercesImpl.jar > > > > + > > > > +bootclasspath.32=xml-apis.jar > > > > + > > > > +bootclasspath.33=x-net.jar > > > > > > -- > > > Gregory Shimansky, 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] > > > > > > > > --------------------------------------------------------------------- > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > -- > Alexey A. Petrenko > 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] --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
