geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ivan <xhh...@gmail.com>
Subject Re: ConfigurationUtil.getEditableConfigurationManager no longer needed?
Date Thu, 22 Oct 2009 01:57:05 GMT
The ActiveMQ plugin also depends on it.
Basically, since we allow the user to add new gbeans in the config.xml file
for an existed configuration, we may need the API to do it programically.


2009/10/22 Rick McGuire <rickmcg@gmail.com>

> David Jencks wrote:
>
>> I would rather make the server work without this functionality.  I'm
>> really not sure its appropriate to modify plugins in this way.  Can we
>> catalog the places this functionality is absolutely required before we put
>> it back in?
>>
> Well, to start with, both the jetty and tomcat plugins are using it, and
> not just for running the tests.
>
> Rick
>
>
>> thanks
>> david jencks
>>
>> On Oct 21, 2009, at 6:06 AM, Ivan wrote:
>>
>>  If no objection, I will try to recover those functions tomorrow,
>>> including the help methods in the ConfigurationUtil.
>>>
>>> 2009/10/21 Ivan <xhhsld@gmail.com <mailto:xhhsld@gmail.com>>
>>>
>>>    Seems that EditableConfigurationManager support is removed (at
>>>    least temporarily) in the Geronimo kernel. I am not sure that
>>>    adding the gbeans to the existed configuration in the runtime is
>>>    a good idea. But, IIRC, some plugins depend on it to add gbeans
>>>    dynamically.
>>>    I would suggest to recover it, the staff I could see is to change
>>>    some codes to adapt the OSGI environment in its implemetation.
>>>
>>>    2009/10/21 Rick McGuire <rickmcg@gmail.com
>>>    <mailto:rickmcg@gmail.com>>
>>>
>>>        The Tomcat unit tests are making use of
>>>        ConfigurationUtil.getEditableConfigurationManager, which has
>>>        been commented out.  Is that method no longer applicable, or
>>>        does it still require some work?  Is
>>>        getConfigurationManager() an workable replacement?
>>>
>>>        Rick
>>>
>>>
>>>
>>>
>>>    --    Ivan
>>>
>>>
>>>
>>>
>>> --
>>> Ivan
>>>
>>
>>
>


-- 
Ivan

Mime
View raw message