Perhaps the symbolic name on this one should follow the mx4j name - org.apache.harmony.management.mx4j. At a minimum, we should probably name it org.apache.harmony.bcel.
On Tue, Jun 3, 2008 at 5:52 AM, Sian January <[EMAIL PROTECTED]> wrote: > That's interesting. I've learnt two new things about OSGi today! > > It looks like I have added that header anyway as I copied one of the other > manifests to create the one for bcel, so should all be fine. > > Thanks, > > Sian > > On 03/06/2008, Tim Ellison <[EMAIL PROTECTED]> wrote: > > > > Sian January wrote: > > > >> Thanks Nathan - I hadn't realised that because all the other manifests > >> have > >> java.* imports. Do you think it would be worth changing them? > >> > > > > Normally you would not have to import java.* packages, because they are > > part of the Java platform assumed to be present to the OSGi runtime. But, > of > > course, we are implementing the java.* packages so we do want to import > and > > export them explicitly to get the modularity in the class libraries. > > > > The special header > > Eclipse-JREBundle: true > > tells Equinox not to import the java.* packages by default, so we see > > breakages when the import of platform packages is missing. > > > > Regards, > > Tim > > > > > > On 03/06/2008, Nathan Beyer <[EMAIL PROTECTED]> wrote: > >> > >>> BTW - java.* classes don't need to be imported; however, javax.* > package > >>> do > >>> > >>> On Mon, Jun 2, 2008 at 9:26 AM, <[EMAIL PROTECTED]> wrote: > >>> > >>> Author: sjanuary > >>>> Date: Mon Jun 2 07:26:26 2008 > >>>> New Revision: 662447 > >>>> > >>>> URL: http://svn.apache.org/viewvc?rev=662447&view=rev > >>>> Log: > >>>> New manifest file for bcel (to be used by pack200) > >>>> > >>>> Added: > >>>> harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/ > >>>> harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/ > >>>> > >>>> > >>>> > > harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/MANIFEST.MF > >>>> (with props) > >>>> > >>>> Added: > >>>> > >>>> > >>> > harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/MANIFEST.MF > >>> > >>>> URL: > >>>> > >>>> > >>> > http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/MANIFEST.MF?rev=662447&view=auto > >>> > >>>> > >>>> > >>> > ============================================================================== > >>> > >>>> --- > >>>> > >>>> > >>> > harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/MANIFEST.MF > >>> > >>>> (added) > >>>> +++ > >>>> > >>>> > >>> > harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/MANIFEST.MF > >>> > >>>> Mon Jun 2 07:26:26 2008 > >>>> @@ -0,0 +1,17 @@ > >>>> +Manifest-Version: 1.0 > >>>> +Bundle-ManifestVersion: 2 > >>>> +Bundle-Name: BCEL > >>>> +Bundle-SymbolicName: org.apache.bcel;singleton:=true > >>>> +Bundle-Version: 5.2 > >>>> +Bundle-ClassPath: bcel-5.2.jar > >>>> +Eclipse-JREBundle: true > >>>> +Import-Package: java.io, > >>>> + java.lang > >>>> +Export-Package: org.apache.bcel, > >>>> + org.apache.bcel.classfile, > >>>> + org.apache.bcel.generic, > >>>> + org.apache.bcel.util, > >>>> + org.apache.bcel.verifier, > >>>> + org.apache.bcel.verifier.exc, > >>>> + org.apache.bcel.verifier.statics, > >>>> + org.apache.bcel.verifier.structurals > >>>> > >>>> Propchange: > >>>> > >>>> > >>> > harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/MANIFEST.MF > >>> > >>>> > >>>> > >>> > ------------------------------------------------------------------------------ > >>> > >>>> svn:eol-style = native > >>>> > >>>> Propchange: > >>>> > >>>> > >>> > harmony/enhanced/classlib/trunk/depends/manifests/bcel-5.2/META-INF/MANIFEST.MF > >>> > >>>> > >>>> > >>> > ------------------------------------------------------------------------------ > >>> > >>>> svn:mime-type = text/plain > >>>> > >>>> > >>>> > >>>> > >> > >> > >> > > > -- > Unless stated otherwise above: > IBM United Kingdom Limited - Registered in England and Wales with number > 741598. > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU >
