Mailing-List: contact dev-help@karaf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@karaf.apache.org
Received-SPF: pass (athena.apache.org: local policy)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
  s=s1024; d=yahoo.com;
  h=DKIM-Signature:X-Yahoo-Newman-Property:X-YMail-OSG:X-Yahoo-SMTP:Received:Content-Type:Mime-Version:Subject:From:In-Reply-To:Date:Content-Transfer-Encoding:Message-Id:References:To:X-Mailer;
  b=uhkiH3W9g3ByCJpzjXsp+1dsbIiL46bDKBd+qYk9N+5g1J4cEWV+h/WXLIvPbiBkdgUkWPXon2Da9116n5bXPuop51dA54VsmezHAoppW/wDwqOBEKm2yGqLYAOxUGHtLoG7jBSL04+/URrd9tzwZHb7rrh3N/Whhf0KRqf2ln8=
  ;
Content-Type: text/plain; charset=us-ascii
Mime-Version: 1.0 (Apple Message framework v1084)
Subject: Re: SCR Tools for Karaf
From: David Jencks <david_jencks@yahoo.com>
In-Reply-To: <9257B584-EE29-447A-8BC4-D760D36F950E@yahoo.com>
Date: Sun, 8 Jan 2012 12:16:57 -0800
Content-Transfer-Encoding: quoted-printable
Message-Id: <A03DF41F-4D57-4FF1-A382-4BD6FDCAFA6C@yahoo.com>
References: <1325977850874-3641090.post@n3.nabble.com>
 <998CB80A-A177-4699-9936-70849567BBD8@yahoo.com>
 <9257B584-EE29-447A-8BC4-D760D36F950E@yahoo.com>
To: dev@karaf.apache.org

Hi... inline

On Jan 8, 2012, at 10:54 AM, Scott England-Sullivan wrote:

> Hi Dave,
>=20
> My comments are inlined below.=20
>=20
> Scott ES
>=20
> On Jan 8, 2012, at 3:17 AM, David Jencks <david_jencks@yahoo.com> =
wrote:
>=20
>> I've modified this a bit to work with trunk and committed the =
results.  This looks really really useful!  I've used scr a lot in =
geronimo and this looks like it will help diagnose the problems I keep =
running into.
>>=20
>> I have a few questions. =20
>>=20
>> - I don't understand some of the feature descriptor.  I've replaced =
it with one generated from the dependencies.  I don't understand why the =
repository, eventadmin, or second feature are needed at all.  Also I =
don't see why the felix scr and metatype are dependencies rather than =
just listed.  I didn't remove the original feature.xml until we =
straighten this out.
>=20
> Most of is for the web console. The web console has a nice interface =
for SCR  If you load the web console first it doesn't pick up the SCR =
dependency and show the tab for it. The SCR WC interface also has a =
dependency on MetaType and EventAdmin, at least it used to. I don't know =
about 3.x.  So it was more of a convenient work around if I was in an =
environment where I had to load it after the WC.=20
>=20
> MetaType also has some additional benefits SCR development when =
combined with bndlib. There is a convenience class in bndlib that takes =
your component config interface and will create a type safe proxy (see =
the example code).
>=20
> Because of all this I wrapped all the dependencies in a feature for =
convenience.=20
>=20
> I am not sure what the best approach is given the WCs optional =
dependency on SCR. It could be that loading of an optional WC capability =
works correctly in trunk. I will look at it this afternoon and get back =
with a response.

I know just about nothing about the webconsole.  I did sort of think =
that the idea generally was to put pieces of the web console in with the =
stuff it worked with, which would mean we might want to move the =
webconsole scr support into the scr module.  Maybe someone with more =
clue can comment :-)

thanks
david jencks

>=20
>>=20
>> - The colors of the output on the console are very faint (light gray =
and light yellow).  I'm not sure how this is configured but would it be =
possible to make it darker?
>=20
> I will take a look at this also and get a fix out for trunk and 2.2.x.=20=

>=20
>>=20
>> thanks!
>> david jencks
>>=20
>> On Jan 7, 2012, at 3:10 PM, sully6768 wrote:
>>=20
>>> Hello All,
>>>=20
>>> I have submitted a set of commands, mbeans, a feature config and =
some
>>> examples.  I attached them to the existing issue KARAF-52.
>>>=20
>>> https://issues.apache.org/jira/browse/KARAF-52
>>>=20
>>> Thanks,
>>> Scott ES
>>>=20
>>> --
>>> View this message in context: =
http://karaf.922171.n3.nabble.com/SCR-Tools-for-Karaf-tp3641090p3641090.ht=
ml
>>> Sent from the Karaf - Dev mailing list archive at Nabble.com.
>>=20