Return-Path: X-Original-To: apmail-felix-dev-archive@www.apache.org Delivered-To: apmail-felix-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 18CD511FDB for ; Tue, 20 May 2014 05:39:06 +0000 (UTC) Received: (qmail 52847 invoked by uid 500); 20 May 2014 05:39:06 -0000 Delivered-To: apmail-felix-dev-archive@felix.apache.org Received: (qmail 52762 invoked by uid 500); 20 May 2014 05:39:05 -0000 Mailing-List: contact dev-help@felix.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@felix.apache.org Delivered-To: mailing list dev@felix.apache.org Received: (qmail 52753 invoked by uid 99); 20 May 2014 05:39:05 -0000 Received: from minotaur.apache.org (HELO minotaur.apache.org) (140.211.11.9) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 20 May 2014 05:39:05 +0000 Received: from localhost (HELO mail-ob0-f178.google.com) (127.0.0.1) (smtp-auth username cziegeler, mechanism plain) by minotaur.apache.org (qpsmtpd/0.29) with ESMTP; Tue, 20 May 2014 05:39:05 +0000 Received: by mail-ob0-f178.google.com with SMTP id va2so7284206obc.37 for ; Mon, 19 May 2014 22:39:04 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.60.44.243 with SMTP id h19mr40839166oem.46.1400564344847; Mon, 19 May 2014 22:39:04 -0700 (PDT) Received: by 10.60.73.9 with HTTP; Mon, 19 May 2014 22:39:04 -0700 (PDT) In-Reply-To: References: Date: Tue, 20 May 2014 07:39:04 +0200 Message-ID: Subject: Re: [DS] Various factory component issues, and progress on RFC 190 From: Carsten Ziegeler To: dev@felix.apache.org Content-Type: multipart/alternative; boundary=001a11331cb04283ca04f9ce4c44 --001a11331cb04283ca04f9ce4c44 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi, 2014-05-19 22:57 GMT+02:00 David Jencks : > There are some difficulties with factory components and DTO/mutliple pid > support. In terms of the spec, DTO and factory components don't work wel= l > together, see bug 2683 (I'm not sure how visible this is to > non-osgi-members). In Felix, > > 1. I think we should not support the obsolete factory configuration >> > newInstance-like behavior on namespace 1.3+ components. We shouldn't hav= e > supported it after namespace 1.0, but we should stop now for sure. Trying > to support this with multiple pids is just too weird. > > +1, yes we should have dropped that support with earlier namespaces :( > 2. Because of how I implemented multiple pid support for normal > components, I think it would be easy to Implement factory config > multip= le > instances of the ComponentFactory service. I believe this behavior was > requested by Pierre de Rop. There's an additional problem if you do this > that you can't distinguish the ComponentFactory services registered from > the multiple factory configurations. To solve this I propose that > configuration properties prefixed with "org.apache.felix.scr.factory." > (exact string up for discussion) be added to the ComponentFactory service > properties. Of course all this would have to be enabled by an flag in th= e > xml such as felix:factoryComponentFactoryPID=3D"true" (no good ideas on t= his > one yet). > Sounds fine to me. Would it make sense to get this into RFC190? > > As a reminder, a few days ago I said I was about to remove the legacy > configuration system entirely in favor of the spec one based on DTOs sinc= e > they are conceptually incompatible and the existing one is fundamentally > confused. So far no response so it's going soon=E2=80=A6. > I agree, let's just go forward with the DTOs Carsten > > Comments? > > thanks > david jencks > > --=20 Carsten Ziegeler cziegeler@apache.org --001a11331cb04283ca04f9ce4c44--