Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: neutral (asf.osuosl.org: 195.212.29.138 is neither permitted nor
 denied by domain of mark.hindess@googlemail.com)
Message-Id: <200606232004.k5NK4rYf031693@d06av02.portsmouth.uk.ibm.com>
From: Mark Hindess <mark.hindess@googlemail.com>
To: harmony-dev@incubator.apache.org
Subject: Re: svn commit: r416738 -
 /incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/init/bootclasspath.properties 
In-reply-to: Your message of "Fri, 23 Jun 2006 23:43:04 +0400."
             <c3755b3a0606231243g8e09537v3a6ae00ba0a693bd@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Date: Fri, 23 Jun 2006 21:04:53 +0100


On 23 June 2006 at 23:43, "Alexey Petrenko" <alexey.a.petrenko@gmail.com> 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 <mark.hindess@googlemail.com>:
> >
> > On 23 June 2006 at 22:37, Gregory Shimansky <gshimansky@gmail.com> 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 hindessm@apache.org 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: harmony-dev-unsubscribe@incubator.apache.org
> > > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
> >
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
> >
> 
> 
> -- 
> Alexey A. Petrenko
> Intel Middleware Products Division
> 
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org


---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org