[prev in list] [next in list] [prev in thread] [next in thread]
List: jboss-user
Subject: Re: [jboss-user] [JBoss Microcontainer Development] -
From: Ales Justin <do-not-reply () jboss ! com>
Date: 2010-03-31 11:46:24
Message-ID: 25065504.913881270035991885.JavaMail.jive () clearspace02 ! app ! mwc ! hst ! phx2 ! redhat ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Ales Justin [http://community.jboss.org/people/alesj] replied to the discussion
"setRequiredStage should also setRequiredState"
To view the discussion, visit: http://community.jboss.org/message/534881#534881
--------------------------------------------------------------
> This patch in OSGi shows that there is something wrong with \
> AbstractDeploymentContext.setRequiredStage(). You shouldn't have to explicitly do \
> the ControllerContext.setRequiredState(), that is an implementation detail.
I think this is how it already was, but you changed it at some point. ;-)
* http://fisheye.jboss.org/browse/JBossAS/projects/jboss-deployers/trunk/deployers-st \
ructure-spi/src/main/java/org/jboss/deployers/structure/spi/helpers/AbstractDeploymentContext.java?r1=85312&r2=89967 \
http://fisheye.jboss.org/browse/JBossAS/projects/jboss-deployers/trunk/deployers-struc \
ture-spi/src/main/java/org/jboss/deployers/structure/spi/helpers/AbstractDeploymentContext.java?r1=85312&r2=89967
So, I guess we need to change both things?
--------------------------------------------------------------
Reply to this message by going to Community
[http://community.jboss.org/message/534881#534881]
Start a new discussion in JBoss Microcontainer Development at Community
[http://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2115]
[Attachment #5 (text/html)]
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<html>
<body link="#355491" alink="#4262a1" vlink="#355491" style="background: #e2e2e2; \
margin: 0; padding: 20px;">
<div>
<table cellpadding="0" bgcolor="#FFFFFF" border="0" cellspacing="0" style="border: \
1px solid #dadada; margin-bottom: 30px; width: 100%; -moz-border-radius: 6px; \
-webkit-border-radius: 6px;"> <tbody>
<tr>
<td>
<table border="0" cellpadding="0" cellspacing="0" bgcolor="#FFFFFF" \
style="border: solid 2px #ccc; background: #dadada; width: 100%; -moz-border-radius: \
6px; -webkit-border-radius: 6px;"> <tbody>
<tr>
<td bgcolor="#000000" valign="middle" height="58px" style="border-bottom: 1px \
solid #ccc; padding: 20px; -moz-border-radius-topleft: 3px; \
-moz-border-radius-topright: 3px; -webkit-border-top-right-radius: 5px; \
-webkit-border-top-left-radius: 5px;">
<h1 style="color: #333333; font: bold 22px Arial, Helvetica, sans-serif; \
margin: 0; display: block !important;">
<!-- To have a header image/logo replace the name below with your img tag \
-->
<!-- Email clients will render the images when the message is read so any \
image -->
<!-- must be made available on a public server, so that all recipients can \
load the image. -->
<a href="http://community.jboss.org/index.jspa" style="text-decoration: \
none; color: #E1E1E1">Community</a></h1> </td>
</tr>
<tr>
<td bgcolor="#FFFFFF" style="font: normal 12px Arial, Helvetica, sans-serif; \
color:#333333; padding: 20px; -moz-border-radius-bottomleft: 4px; \
-moz-border-radius-bottomright: 4px; -webkit-border-bottom-right-radius: 5px; \
-webkit-border-bottom-left-radius: 5px;"><h3 style="margin: 10px 0 5px; font-size: \
17px; font-weight: normal;"> setRequiredStage should also setRequiredState
</h3>
<span style="margin-bottom: 10px;">
reply from <a href="http://community.jboss.org/people/alesj">Ales Justin</a> in \
<i>JBoss Microcontainer Development</i> - <a \
href="http://community.jboss.org/message/534881#534881">View the full discussion</a> \
</span> <hr style="margin: 20px 0; border: none; background-color: #dadada; height: \
1px;">
<div class="jive-rendered-content"><blockquote class="jive-quote">This patch in OSGi \
shows that there is something wrong with \
AbstractDeploymentContext.setRequiredStage().<p style="min-height: 8pt; height: 8pt; \
padding: 0px;"> </p><p>You shouldn't have to explicitly do the \
ControllerContext.setRequiredState(), that is an implementation \
detail.</p></blockquote><p>I think this is how it already was, but you changed it at \
some point. ;-)</p><p><span>* </span><a class="jive-link-external-small" \
href="http://fisheye.jboss.org/browse/JBossAS/projects/jboss-deployers/trunk/deployers \
-structure-spi/src/main/java/org/jboss/deployers/structure/spi/helpers/AbstractDeploymentContext.java?r1=85312&r2=89967" \
target="_blank">http://fisheye.jboss.org/browse/JBossAS/projects/jboss-deployers/trunk \
/deployers-structure-spi/src/main/java/org/jboss/deployers/structure/spi/helpers/AbstractDeploymentContext.java?r1=85312&r2=89967</a></p><p \
style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>So, I guess we need \
to change both things?</p></div>
<div style="background-color: #f4f4f4; padding: 10px; margin-top: 20px;">
<p style="margin: 0;">Reply to this message by <a \
href="http://community.jboss.org/message/534881#534881">going to Community</a></p> \
<p style="margin: 0;">Start a new discussion in JBoss Microcontainer Development at \
<a href="http://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2115">Community</a></p>
</div></td>
</tr>
</tbody>
</table>
</td>
</tr>
</tbody>
</table>
</div>
</body>
</html>
_______________________________________________
jboss-user mailing list
jboss-user@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-user
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic