harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Geir Magnusson Jr." <g...@pobox.com>
Subject Re: [drlvm/classlib/jdktools] system property o.a.h.boot.class.path
Date Thu, 30 Nov 2006 03:33:41 GMT


Alexey Varlamov wrote:
> 2006/11/30, Geir Magnusson Jr. <geir@pobox.com>:
>>
>>
>> Alexey Varlamov wrote:
>> > 2006/11/30, Geir Magnusson Jr. <geir@pobox.com>:
>> >>
>> >>
>> >> Alexey Varlamov wrote:
>> >> > 2006/11/30, Geir Magnusson Jr. <geir@pobox.com>:
>> >> >> (How's that for a category in the subject line?)
>> >> >>
>> >> >> I'm working on jdktools, and was getting javac going.  We have
a 
>> small
>> >> >> issue.  Currently, the wrapper code grabs the boot class path 
>> via the
>> >> >> system property
>> >> >>
>> >> >>     org.apache.harmony.boot.class.path
>> >> >>
>> >> >> This is initially set by luni, which collects all the entries in
>> >> >> bootclasspath.properties and adds them to the path.
>> >> >>
>> >> >> Now, the one thing that it doesn't do is include the kernel.jar,
as
>> >> >> that's a degree of freedom for the vm which provides that jar.
>> >> >>
>> >> >> Now, in DRLVM, we take the o.a.h.b.c.p and prefix the kernel.jar,
>> >> prefix
>> >> >> and postfix -Xbootclasspath/? for a complete runtime bootclasspath,
>> >> and
>> >> >> call it
>> >> >>
>> >> >>     vm.boot.class.path
>> >> >
>> >> > Things are changing :) Since recent H-2008 commit, magics support 
>> jars
>> >> > are going to bypass this machinery and slip into boot loader 
>> directly
>> >> > via SetBCPElement().
>> >>
>> >> Bypass what machinery?
>> >
>> > Composing BCP from prefixes and postfixes, and keeping whole runtime
>> > BCP string as a property. They just add jars directly to a vector of
>> > searching elements of BootstrapClassLoader, and this is no good from
>> > my POV.
>>
>> I'm confused.  We do need to have the string, at least able to be
>> created on demand.
> Indeed - and I see no reason why treat magics differently.
> 
>> Do you believe that adding jars directly is no good?
> I prefer to have the consistent way.
> IMHO BootstrapClassLoader::SetBCPElement() should not be public, it is
> just a part of BootstrapClassLoader initialization and not intended
> for regular usage.

OH!  So you're saying that you don't like the direct injection into the 
vector, but rather would normalize it into the way it's happening now in 
BootstrapClassloader?

if so, I agree 100% - sorry about the confusion

geir

Mime
View raw message