You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by Ate Douma <at...@douma.nu> on 2009/05/04 19:02:16 UTC

Re: svn commit: r771327 - unintended commit of unrelated changes

I just noticed I accidentally also committed some other local changes totally unrelated to JS2-985 but were just testing changes for 
determining Java6 build/test problems :(
I knew and planned not to commit those but somehow I forgot to exclude them from from commit list after all.
My bad and sorry about this. I'll rollback these unintended changes ASAP.

Regards,

Ate

ate@apache.org wrote:
> Author: ate
> Date: Mon May  4 15:02:23 2009
> New Revision: 771327
> 
> URL: http://svn.apache.org/viewvc?rev=771327&view=rev
> Log:
> JS2-985: Upgrade to new pluto-container-api requirements as needed for PLUTO-557
> See: http://issues.apache.org/jira/browse/JS2-985
> - adding new Pluto RequestDispatcherService and handling: as it needs to be passed on to PortletContext its needed in PortletFactory
>   as well as for the ServletContextProvider service
> - dropping OM ApplicationDefinition ServletURLMapping state and corresponding (OJB) mapping and schema table definition
> 
> Added:
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/JETSPEED-INF/spring/RequestDispatcherService.xml   (with props)
> Removed:
>     portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/util/
>     portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/ServletMappingURLPatternImpl.java
>     portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/om/portlet/ServletMappingURLPattern.java
> Modified:
>     portals/jetspeed-2/portal/trunk/components/jetspeed-db-tools/pom.xml
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/services/JetspeedPlutoServices.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/AbstractRequestContextTestCase.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/AbstractEngineTest.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/TestSpringEngine.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/tools/pamanager/PortletApplicationManagerServer.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/pom.xml
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedPortletContextImpl.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedServletContextProviderImpl.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/factory/JetspeedPortletFactory.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/test/java/org/apache/jetspeed/factory/TestPortletFactory.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-profiler/src/test/java/org/apache/jetspeed/profiler/TestProfiler.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/JETSPEED-INF/ojb/registry_repository.xml
>     portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/descriptor/JetspeedDescriptorServiceImpl.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/PortletApplicationDefinitionImpl.java
>     portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/test/resources/registry-test.xml
>     portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/ddl-schema/registry-schema.xml
>     portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/pluto-services.xml
>     portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/registry.xml
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org


Re: svn commit: r771327 - unintended commit of unrelated changes

Posted by Randy Watler <wa...@wispertel.net>.
Ate,

Those tests are somewhat sensitive to machine load and timing cross 
process. I think i need to loosen up the response time constraints to 
some extent to allow for concurrent developer activities. Of course, 
there could also be a relatively rare bug in there that only gets teased 
out once in a while. I am tracking it... hopefully it shows itself 
sometime when I am ready to look at it.

Randy

Ate Douma wrote:
> Randy Watler wrote:
>> Ate,
>>
>> Please try to rerun. I have noticed some sporadic failures with that 
>> test.
> Just did without errors, weird.
>>
>> Randy
>>
>> Ate Douma wrote:
>>> Ate Douma wrote:
>>>> I just noticed I accidentally also committed some other local 
>>>> changes totally unrelated to JS2-985 but were just testing changes 
>>>> for determining Java6 build/test problems :(
>>>> I knew and planned not to commit those but somehow I forgot to 
>>>> exclude them from from commit list after all.
>>>> My bad and sorry about this. I'll rollback these unintended changes 
>>>> ASAP.
>>>
>>> Done: r771376 contains the rolled back versions of 
>>> AbstractEngineTest and TestSpringEngine again (no modifications in 
>>> between).
>>>
>>> Note: the page-manager/TestDatabasePageManagerCache now suddenly 
>>> fails on line #248.
>>> I have no idea yet why but AFAIK its unrelated to my other changes 
>>> for JS2-985 which passed testing during the weekend and I didn't 
>>> make changes afterwards.
>>>
>>>>
>>>> Regards,
>>>>
>>>> Ate
>>>>
>>>> ate@apache.org wrote:
>>>>> Author: ate
>>>>> Date: Mon May  4 15:02:23 2009
>>>>> New Revision: 771327
>>>>>
>>>>> URL: http://svn.apache.org/viewvc?rev=771327&view=rev
>>>>> Log:
>>>>> JS2-985: Upgrade to new pluto-container-api requirements as needed 
>>>>> for PLUTO-557
>>>>> See: http://issues.apache.org/jira/browse/JS2-985
>>>>> - adding new Pluto RequestDispatcherService and handling: as it 
>>>>> needs to be passed on to PortletContext its needed in PortletFactory
>>>>>   as well as for the ServletContextProvider service
>>>>> - dropping OM ApplicationDefinition ServletURLMapping state and 
>>>>> corresponding (OJB) mapping and schema table definition
>>>>>
>>>>> Added:
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/JETSPEED-INF/spring/RequestDispatcherService.xml   
>>>>> (with props)
>>>>> Removed:
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/util/ 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/ServletMappingURLPatternImpl.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/om/portlet/ServletMappingURLPattern.java 
>>>>>
>>>>> Modified:
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-db-tools/pom.xml
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/services/JetspeedPlutoServices.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/AbstractRequestContextTestCase.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/AbstractEngineTest.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/TestSpringEngine.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/tools/pamanager/PortletApplicationManagerServer.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/pom.xml 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedPortletContextImpl.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedServletContextProviderImpl.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/factory/JetspeedPortletFactory.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/test/java/org/apache/jetspeed/factory/TestPortletFactory.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-profiler/src/test/java/org/apache/jetspeed/profiler/TestProfiler.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/JETSPEED-INF/ojb/registry_repository.xml 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/descriptor/JetspeedDescriptorServiceImpl.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/PortletApplicationDefinitionImpl.java 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/test/resources/registry-test.xml 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/ddl-schema/registry-schema.xml 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/pluto-services.xml 
>>>>>
>>>>>     
>>>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/registry.xml 
>>>>>
>>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
>>>> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>>>>
>>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
>>> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>>>
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
>> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>>
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org


Re: svn commit: r771327 - unintended commit of unrelated changes

Posted by Ate Douma <at...@douma.nu>.
Randy Watler wrote:
> Ate,
> 
> Please try to rerun. I have noticed some sporadic failures with that test.
Just did without errors, weird.
> 
> Randy
> 
> Ate Douma wrote:
>> Ate Douma wrote:
>>> I just noticed I accidentally also committed some other local changes 
>>> totally unrelated to JS2-985 but were just testing changes for 
>>> determining Java6 build/test problems :(
>>> I knew and planned not to commit those but somehow I forgot to 
>>> exclude them from from commit list after all.
>>> My bad and sorry about this. I'll rollback these unintended changes 
>>> ASAP.
>>
>> Done: r771376 contains the rolled back versions of AbstractEngineTest 
>> and TestSpringEngine again (no modifications in between).
>>
>> Note: the page-manager/TestDatabasePageManagerCache now suddenly fails 
>> on line #248.
>> I have no idea yet why but AFAIK its unrelated to my other changes for 
>> JS2-985 which passed testing during the weekend and I didn't make 
>> changes afterwards.
>>
>>>
>>> Regards,
>>>
>>> Ate
>>>
>>> ate@apache.org wrote:
>>>> Author: ate
>>>> Date: Mon May  4 15:02:23 2009
>>>> New Revision: 771327
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=771327&view=rev
>>>> Log:
>>>> JS2-985: Upgrade to new pluto-container-api requirements as needed 
>>>> for PLUTO-557
>>>> See: http://issues.apache.org/jira/browse/JS2-985
>>>> - adding new Pluto RequestDispatcherService and handling: as it 
>>>> needs to be passed on to PortletContext its needed in PortletFactory
>>>>   as well as for the ServletContextProvider service
>>>> - dropping OM ApplicationDefinition ServletURLMapping state and 
>>>> corresponding (OJB) mapping and schema table definition
>>>>
>>>> Added:
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/JETSPEED-INF/spring/RequestDispatcherService.xml   
>>>> (with props)
>>>> Removed:
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/util/ 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/ServletMappingURLPatternImpl.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/om/portlet/ServletMappingURLPattern.java 
>>>>
>>>> Modified:
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-db-tools/pom.xml
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/services/JetspeedPlutoServices.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/AbstractRequestContextTestCase.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/AbstractEngineTest.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/TestSpringEngine.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/tools/pamanager/PortletApplicationManagerServer.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/pom.xml 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedPortletContextImpl.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedServletContextProviderImpl.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/factory/JetspeedPortletFactory.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/test/java/org/apache/jetspeed/factory/TestPortletFactory.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-profiler/src/test/java/org/apache/jetspeed/profiler/TestProfiler.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/JETSPEED-INF/ojb/registry_repository.xml 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/descriptor/JetspeedDescriptorServiceImpl.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/PortletApplicationDefinitionImpl.java 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/test/resources/registry-test.xml 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/ddl-schema/registry-schema.xml 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/pluto-services.xml 
>>>>
>>>>     
>>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/registry.xml 
>>>>
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
>>> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>>>
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
>> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>>
>>
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org


Re: svn commit: r771327 - unintended commit of unrelated changes

Posted by Randy Watler <wa...@wispertel.net>.
Ate,

Please try to rerun. I have noticed some sporadic failures with that test.

Randy

Ate Douma wrote:
> Ate Douma wrote:
>> I just noticed I accidentally also committed some other local changes 
>> totally unrelated to JS2-985 but were just testing changes for 
>> determining Java6 build/test problems :(
>> I knew and planned not to commit those but somehow I forgot to 
>> exclude them from from commit list after all.
>> My bad and sorry about this. I'll rollback these unintended changes 
>> ASAP.
>
> Done: r771376 contains the rolled back versions of AbstractEngineTest 
> and TestSpringEngine again (no modifications in between).
>
> Note: the page-manager/TestDatabasePageManagerCache now suddenly fails 
> on line #248.
> I have no idea yet why but AFAIK its unrelated to my other changes for 
> JS2-985 which passed testing during the weekend and I didn't make 
> changes afterwards.
>
>>
>> Regards,
>>
>> Ate
>>
>> ate@apache.org wrote:
>>> Author: ate
>>> Date: Mon May  4 15:02:23 2009
>>> New Revision: 771327
>>>
>>> URL: http://svn.apache.org/viewvc?rev=771327&view=rev
>>> Log:
>>> JS2-985: Upgrade to new pluto-container-api requirements as needed 
>>> for PLUTO-557
>>> See: http://issues.apache.org/jira/browse/JS2-985
>>> - adding new Pluto RequestDispatcherService and handling: as it 
>>> needs to be passed on to PortletContext its needed in PortletFactory
>>>   as well as for the ServletContextProvider service
>>> - dropping OM ApplicationDefinition ServletURLMapping state and 
>>> corresponding (OJB) mapping and schema table definition
>>>
>>> Added:
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/JETSPEED-INF/spring/RequestDispatcherService.xml   
>>> (with props)
>>> Removed:
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/util/ 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/ServletMappingURLPatternImpl.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/om/portlet/ServletMappingURLPattern.java 
>>>
>>> Modified:
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-db-tools/pom.xml
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/services/JetspeedPlutoServices.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/AbstractRequestContextTestCase.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/AbstractEngineTest.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/TestSpringEngine.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/tools/pamanager/PortletApplicationManagerServer.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/pom.xml 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedPortletContextImpl.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedServletContextProviderImpl.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/factory/JetspeedPortletFactory.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/test/java/org/apache/jetspeed/factory/TestPortletFactory.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-profiler/src/test/java/org/apache/jetspeed/profiler/TestProfiler.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/JETSPEED-INF/ojb/registry_repository.xml 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/descriptor/JetspeedDescriptorServiceImpl.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/PortletApplicationDefinitionImpl.java 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/test/resources/registry-test.xml 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/ddl-schema/registry-schema.xml 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/pluto-services.xml 
>>>
>>>     
>>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/registry.xml 
>>>
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
>> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>>
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org


Re: svn commit: r771327 - unintended commit of unrelated changes

Posted by Ate Douma <at...@douma.nu>.
Ate Douma wrote:
> I just noticed I accidentally also committed some other local changes 
> totally unrelated to JS2-985 but were just testing changes for 
> determining Java6 build/test problems :(
> I knew and planned not to commit those but somehow I forgot to exclude 
> them from from commit list after all.
> My bad and sorry about this. I'll rollback these unintended changes ASAP.

Done: r771376 contains the rolled back versions of AbstractEngineTest and TestSpringEngine again (no modifications in between).

Note: the page-manager/TestDatabasePageManagerCache now suddenly fails on line #248.
I have no idea yet why but AFAIK its unrelated to my other changes for JS2-985 which passed testing during the weekend and I didn't make 
changes afterwards.

> 
> Regards,
> 
> Ate
> 
> ate@apache.org wrote:
>> Author: ate
>> Date: Mon May  4 15:02:23 2009
>> New Revision: 771327
>>
>> URL: http://svn.apache.org/viewvc?rev=771327&view=rev
>> Log:
>> JS2-985: Upgrade to new pluto-container-api requirements as needed for 
>> PLUTO-557
>> See: http://issues.apache.org/jira/browse/JS2-985
>> - adding new Pluto RequestDispatcherService and handling: as it needs 
>> to be passed on to PortletContext its needed in PortletFactory
>>   as well as for the ServletContextProvider service
>> - dropping OM ApplicationDefinition ServletURLMapping state and 
>> corresponding (OJB) mapping and schema table definition
>>
>> Added:
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/JETSPEED-INF/spring/RequestDispatcherService.xml   
>> (with props)
>> Removed:
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/util/ 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/ServletMappingURLPatternImpl.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/om/portlet/ServletMappingURLPattern.java 
>>
>> Modified:
>>     portals/jetspeed-2/portal/trunk/components/jetspeed-db-tools/pom.xml
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/services/JetspeedPlutoServices.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/AbstractRequestContextTestCase.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/AbstractEngineTest.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/engine/TestSpringEngine.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/test/java/org/apache/jetspeed/tools/pamanager/PortletApplicationManagerServer.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/pom.xml 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedPortletContextImpl.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/container/JetspeedServletContextProviderImpl.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/main/java/org/apache/jetspeed/factory/JetspeedPortletFactory.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-portlet-factory/src/test/java/org/apache/jetspeed/factory/TestPortletFactory.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-profiler/src/test/java/org/apache/jetspeed/profiler/TestProfiler.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/JETSPEED-INF/ojb/registry_repository.xml 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/descriptor/JetspeedDescriptorServiceImpl.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/om/portlet/impl/PortletApplicationDefinitionImpl.java 
>>
>>     
>> portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/test/resources/registry-test.xml 
>>
>>     
>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/ddl-schema/registry-schema.xml 
>>
>>     
>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/pluto-services.xml 
>>
>>     
>> portals/jetspeed-2/portal/trunk/jetspeed-portal-resources/src/main/resources/assembly/registry.xml 
>>
>>
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org