You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@excalibur.apache.org by fa...@apache.org on 2005/04/07 17:33:31 UTC
svn commit: r160410 -
excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress/util/ContextManager.java
Author: farra
Date: Thu Apr 7 08:33:30 2005
New Revision: 160410
URL: http://svn.apache.org/viewcvs?view=rev&rev=160410
Log:
Fixed bug in creating new RoleManager. ContextManager now passes the classloader to the role manager in the constructor. This is necessary to get Fortress to run inside an Eclipse plugin, for example
Modified:
excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress/util/ContextManager.java
Modified: excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress/util/ContextManager.java
URL: http://svn.apache.org/viewcvs/excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress/util/ContextManager.java?view=diff&r1=160409&r2=160410
==============================================================================
--- excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress/util/ContextManager.java (original)
+++ excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress/util/ContextManager.java Thu Apr 7 08:33:30 2005
@@ -164,7 +164,7 @@
* Either supplied via rootContext or created locally.
*/
protected InstrumentManager m_instrumentManager;
-
+
/**
* The components that are "owned" by this context and should
* be disposed by it. Any manager that is created as a result
@@ -606,7 +606,7 @@
else
{
// Create a role manager with the configured roles
- rm = new ConfigurableRoleManager( frm );
+ rm = new ConfigurableRoleManager( frm, classLcp oader );
}
ContainerUtil.enableLogging(rm, rmLogger );
@@ -918,7 +918,7 @@
final ThreadPolicy policy = buildThreadPolicy();
policy.setThreadFactory( factory );
-
+
final DefaultThreadManager tm = new DefaultThreadManager(policy);
assumeOwnership( tm );
---------------------------------------------------------------------
To unsubscribe, e-mail: scm-unsubscribe@excalibur.apache.org
For additional commands, e-mail: scm-help@excalibur.apache.org
Re: svn commit: r160410 - excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress /util/ContextManager.java
Posted by J Aaron Farr <ja...@gmail.com>.
On 4/13/05, Leo Simons <ma...@leosimons.com> wrote:
> On 07-04-2005 17:33, "farra@apache.org" <fa...@apache.org> wrote:
> > @@ -606,7 +606,7 @@
> > else
> > {
> > // Create a role manager with the configured roles
> > - rm = new ConfigurableRoleManager( frm );
> > + rm = new ConfigurableRoleManager( frm, classLcp oader );
> ^^^^^
> Typo! Gump barfed about it. Please don't commit non-compiling code :-D
Sorry about that. I must have inadvertantly edited the code between
testing it and committing it. I think that "cp " was supposed to be a
copy command in my editor. :)
Committing a fix now (or whenever my proxy server allows an SSL connection... )
--
jaaron
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@excalibur.apache.org
For additional commands, e-mail: dev-help@excalibur.apache.org
Re: svn commit: r160410 -
excalibur/trunk/fortress/container-impl/src/java/org/apache/avalon/fortress
/util/ContextManager.java
Posted by Leo Simons <ma...@leosimons.com>.
On 07-04-2005 17:33, "farra@apache.org" <fa...@apache.org> wrote:
> @@ -606,7 +606,7 @@
> else
> {
> // Create a role manager with the configured roles
> - rm = new ConfigurableRoleManager( frm );
> + rm = new ConfigurableRoleManager( frm, classLcp oader );
^^^^^
Typo! Gump barfed about it. Please don't commit non-compiling code :-D
- Leo
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@excalibur.apache.org
For additional commands, e-mail: dev-help@excalibur.apache.org