Incompatible change to JBossASKernel

[wolfc]

With changes r82684 & r82689 JBossASKernel has become incompatible with ejb3-core. This leads to the following problem when trying to deploy service beans into a patched AS GA:
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4200935#4200935

[alrubinger]

I should have posted this before.

To clarify, JBossASKernel is now incompatible w/ ejb3-core:1.0.0-Beta11 and under, (used in AS 5.0.0.GA). So this in turn means that the Plugin is no longer compatible with AS 5.0.0GA.

This is begging for a better solution, both to the "patching 5.0.0.GA" problem as well as patching AS looking forward.

For the latter I'll be starting to remove the "ejb3" module from AS when TestSuite work is completed; will post a new Thread when this is to start.

S,
ALR

[wolfc]

I say revert JBossASKernel and have ServiceContainer use existing working construct to get the same functionality.

[alrubinger]

"wolfc" wrote:
I say revert JBossASKernel and have ServiceContainer use existing working construct to get the same functionality.

To keep this Thread current, what have we decided here in light of AS 5.0.1?

My position is that we're under no obligation for the first release of Plugin to patch AS 5.0.0.GA. However, once we start releasing, we must be able to patch all the way back to whatever version of AS we start from.

Meaning if Plugin 1.0.0.GA is targeted for AS 5.0.1, all subsequent Plugin 1.0.x must work against AS 5.0.1(+x).

S,
ALR

[alrubinger]

Here's what's been done:

* Reverted the changes made to JBossASKernel to put it back to state used in AS 5.0.0.GA (Branch_5_x only) http://fisheye.jboss.org/changelog/JBossAS/?cs=82891
* Made ServiceContainer emulate the stuff I'd previously removed from JBossASKernel http://fisheye.jboss.org/changelog/JBossAS/?cs=82890

So Plugin should again be compatible w/ JBossAS 5.0.0.GA.

S,
ALR

[alrubinger]

"ALRubinger" wrote:
* Reverted the changes made to JBossASKernel to put it back to state used in AS 5.0.0.GA (Branch_5_x only)

I did this in Branch_5_0 in r82895 as well. Too many branches now. :)

S,
ALR