Hi,

As I have said many times in the past it is not valid to put attributes into
an Import-Service header. The OSGi Alliance specification that defines the
syntax says it is a comma separated list of service interface/class names. I
support any effort that reduces the use of invalid OSGi header syntax.

Alasdair

On 20 September 2011 08:12, Guillaume Nodet <[email protected]> wrote:

> Unless I hear something, I plan to revert this change.
>
> On Thu, Sep 8, 2011 at 15:42, Guillaume Nodet <[email protected]> wrote:
>
> > What is the reason for removing those informations ?
> > Those are used when computing the obr constraints ... and actually
> > used when resolving using OBR.
> >
> > On Thu, Jul 28, 2011 at 10:48,  <[email protected]> wrote:
> > > Author: cziegeler
> > > Date: Thu Jul 28 08:48:27 2011
> > > New Revision: 1151765
> > >
> > > URL: http://svn.apache.org/viewvc?rev=1151765&view=rev
> > > Log:
> > > FELIX-2156 - Remove Import-Service header in MANIFEST
> > >
> > > Modified:
> > >    felix/trunk/eventadmin/impl/changelog.txt
> > >    felix/trunk/eventadmin/impl/pom.xml
> > >
> > > Modified: felix/trunk/eventadmin/impl/changelog.txt
> > > URL:
> >
> http://svn.apache.org/viewvc/felix/trunk/eventadmin/impl/changelog.txt?rev=1151765&r1=1151764&r2=1151765&view=diff
> > >
> >
> ==============================================================================
> > > --- felix/trunk/eventadmin/impl/changelog.txt (original)
> > > +++ felix/trunk/eventadmin/impl/changelog.txt Thu Jul 28 08:48:27 2011
> > > @@ -7,6 +7,9 @@ Changes from 1.2.12 to 1.2.14
> > >     * [FELIX-3053] - Potential deadlock if event handler throws
> Throwable
> > and is bypassing timeout handling
> > >     * [FELIX-3055] - Event Admin deadlocks when sendEvent is called
> from
> > within a handleEvent method
> > >
> > > +** Improvement
> > > +    * [FELIX-2156] - Remove Import-Service header in MANIFEST
> > > +
> > >
> > >  Changes from 1.2.10 to 1.2.12
> > >  -----------------------------
> > >
> > > Modified: felix/trunk/eventadmin/impl/pom.xml
> > > URL:
> >
> http://svn.apache.org/viewvc/felix/trunk/eventadmin/impl/pom.xml?rev=1151765&r1=1151764&r2=1151765&view=diff
> > >
> >
> ==============================================================================
> > > --- felix/trunk/eventadmin/impl/pom.xml (original)
> > > +++ felix/trunk/eventadmin/impl/pom.xml Thu Jul 28 08:48:27 2011
> > > @@ -98,14 +98,6 @@
> > >                         </Import-Package>
> > >
> > <Export-Package>org.osgi.service.event</Export-Package>
> > >
> > <Private-Package>org.apache.felix.eventadmin.impl.*</Private-Package>
> > > -                        <Import-Service>
> > > -
> >
>  org.osgi.service.event.EventHandler;availability:=optional;multiple:=true,
> > > -
> >  org.osgi.service.log.LogService;availability:=optional;multiple:=false,
> > > -
> >
>  org.osgi.service.log.LogReaderService;availability:=optional;multiple:=false
> > > -                        </Import-Service>
> > > -                        <Export-Service>
> > > -                           org.osgi.service.event.EventAdmin
> > > -                        </Export-Service>
> > >                         <!-- Include concurrent lib but not sub
> packages
> > -->
> > >                         <Embed-Dependency>
> > >
> > concurrent;inline="EDU/oswego/cs/dl/util/concurrent/[A-Z]*"
> > >
> > >
> > >
> >
> >
> >
> > --
> > ------------------------
> > Guillaume Nodet
> > ------------------------
> > Blog: http://gnodet.blogspot.com/
> > ------------------------
> > Open Source SOA
> > http://fusesource.com
> >
>
>
>
> --
> ------------------------
> Guillaume Nodet
> ------------------------
> Blog: http://gnodet.blogspot.com/
> ------------------------
> Open Source SOA
> http://fusesource.com
>



-- 
Alasdair Nottingham
[email protected]

Reply via email to