[jboss-cvs] jboss-cvs-commits Digest, Vol 44, Issue 151

[Thomas Diesler]

I might need to add: When you do put stuff in the installer that 
overwrites critical components in
AS, please make sure the AS starts up at least.

cheers
-thoams

On 02/10/2010 02:18 PM, Thomas Diesler wrote:
> I tried hard to make it work for you. Unfortunately its not just the 
> missing update you were talking about.
> Believe me, I also rather not rollback work. However those commits 
> broke fundamental service handling functionality to a degree where 
> everything stopped working in AS. A working AS integration is however 
> needed for much of the work I do.
>
> We don't need to get political on this. A valid commit needs to pass 
> the local testsuite, which is a prerequisite of TCK runs. It should 
> also not break the AS integration in a fundamental way. If something 
> needs to be installed in AS, the installer can do that.
>
> I don't ask you to run the AS tests for every commit, but I do ask you 
> to respect the nightly Hudson runs that pick up broken functionality 
> that might work standalone.
>
> All I am asking is: Put the stuff in the installer that you want to 
> have in AS. The day after you will know whether it really worked in AS 
> as well.
>
> On 02/10/2010 11:57 AM, Ales Justin wrote:
>> Can you please revert this, as this is getting childish and ridiculous.
>> I said that Kernel 2.2.0.Alpha6 is gonna be in AS in a day -- today.
>>
>> The framework itself is not broken,
>> and AS - afaik - can easily be updated via installer with new Kernel 
>> version,
>> if you're really in such a hurry.
>>
>> Your constant unjustified reverting/refactoring is very annoying,
>> Adrian expressed it already, now I'm expressing it as well.
>> If you want us to help you, this is not the way to go, and needs an 
>> immediate change of attitude.
>>
>> I would understand if the framework itself was broken, which is not.
>> When you broke it a couple of days ago, I worked around the issue.
>> Not to mention the lack of willingness to support other OS envs with 
>> your native code -- which was all likely candidate for reverting as 
>> well.
>>
>> OSGi+MC is not WS or jBPM, hence needs different approach on how to 
>> tests it. ;-)
>> * http://community.jboss.org/message/525198#525198
>>
>>> Message: 4
>>> Date: Wed, 10 Feb 2010 05:39:46 -0500
>>> From: jboss-cvs-commits at lists.jboss.org
>>> Subject: [jboss-cvs] JBossAS SVN: r100793 - in projects/jboss-osgi:
>>>     projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle 
>>>
>>>     and 9 other directories.
>>> To: jboss-cvs-commits at lists.jboss.org
>>> Message-ID:
>>> <201002101039.o1AAdk9a030469 at svn01.web.mwc.hst.phx2.redhat.com>
>>> Content-Type: text/plain; charset=UTF-8
>>>
>>> Author: thomas.diesler at jboss.com
>>> Date: 2010-02-10 05:39:45 -0500 (Wed, 10 Feb 2010)
>>> New Revision: 100793
>>>
>>> Removed:
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/AbstractDictionaryFactory.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/JMXDictionaryFactory.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/service/support/a/AMBean.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/service/support/c/CMBean.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/resources/bundles/service/service-jmx1/ 
>>>
>>> Modified:
>>>    projects/jboss-osgi/projects/runtime/framework/trunk/pom.xml
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/AbstractBundleState.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/DescribeAction.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/KernelDictionaryFactory.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/OSGiBundleState.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/OSGiServiceReferenceWrapper.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/PreInstallAction.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/bundle/ServiceManagerPluginImpl.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/main/java/org/jboss/osgi/framework/plugins/ServiceManagerPlugin.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/DeployersTest.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/service/JMXServicesUnitTestCase.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/service/ServiceMixUnitTestCase.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/service/ServicesTest.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/service/support/a/A.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/java/org/jboss/test/osgi/service/support/c/C.java 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/resources/bootstrap/jboss-osgi-bootstrap.xml 
>>>
>>>    
>>> projects/jboss-osgi/projects/runtime/framework/trunk/src/test/resources/org/jboss/test/osgi/service/JMXServicesUnitTestCase.xml 
>>>
>>>    
>>> projects/jboss-osgi/trunk/distribution/installer/src/main/resources/installer/install-definition.xml 
>>>
>>> Log:
>>> [JBOSGI-141] Restore AS integration to a working state
>

-- 
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Thomas Diesler
JBoss OSGi Lead
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx