I'm not so sure about moving the NoOpResolver to the util bundle. It isn't something I would encourage people to consume and having it in util will mean everyone gets it, which could cause it to be chosen over say the OBR resolver.
Alasdair On 12 March 2010 18:30, <[email protected]> wrote: > Author: gawor > Date: Fri Mar 12 18:30:33 2010 > New Revision: 922381 > > URL: http://svn.apache.org/viewvc?rev=922381&view=rev > Log: > ARIES-238: move NoOpResolver and DefaultLocalPlatform to application-utils > > Added: > > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/DefaultLocalPlatform.java > (contents, props changed) > - copied, changed from r922335, > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/DefaultLocalPlatform.java > > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/NoOpResolver.java > (contents, props changed) > - copied, changed from r922335, > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/NoOpResolver.java > Removed: > > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/DefaultLocalPlatform.java > > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/NoOpResolver.java > Modified: > > incubator/aries/trunk/application/application-runtime/src/main/resources/OSGI-INF/blueprint/app-context-management.xml > > Modified: > incubator/aries/trunk/application/application-runtime/src/main/resources/OSGI-INF/blueprint/app-context-management.xml > URL: > http://svn.apache.org/viewvc/incubator/aries/trunk/application/application-runtime/src/main/resources/OSGI-INF/blueprint/app-context-management.xml?rev=922381&r1=922380&r2=922381&view=diff > ============================================================================== > --- > incubator/aries/trunk/application/application-runtime/src/main/resources/OSGI-INF/blueprint/app-context-management.xml > (original) > +++ > incubator/aries/trunk/application/application-runtime/src/main/resources/OSGI-INF/blueprint/app-context-management.xml > Fri Mar 12 18:30:33 2010 > @@ -28,8 +28,8 @@ > <!-- Default implementations of services that must be implemented against a > particular application server environment follow --> > > - <bean id="default-local-platform" > class="org.apache.aries.application.runtime.defaults.DefaultLocalPlatform"/> > - <bean id="no-op-resolver" > class="org.apache.aries.application.runtime.defaults.NoOpResolver"/> > + <bean id="default-local-platform" > class="org.apache.aries.application.utils.management.DefaultLocalPlatform"/> > + <bean id="no-op-resolver" > class="org.apache.aries.application.utils.management.NoOpResolver"/> > > <service interface="org.apache.aries.application.management.LocalPlatform" > ref="default-local-platform" > > Copied: > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/DefaultLocalPlatform.java > (from r922335, > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/DefaultLocalPlatform.java) > URL: > http://svn.apache.org/viewvc/incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/DefaultLocalPlatform.java?p2=incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/DefaultLocalPlatform.java&p1=incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/DefaultLocalPlatform.java&r1=922335&r2=922381&rev=922381&view=diff > ============================================================================== > --- > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/DefaultLocalPlatform.java > (original) > +++ > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/DefaultLocalPlatform.java > Fri Mar 12 18:30:33 2010 > @@ -16,7 +16,7 @@ > * specific language governing permissions and limitations > * under the License. > */ > -package org.apache.aries.application.runtime.defaults; > +package org.apache.aries.application.utils.management; > > import java.io.File; > import java.io.IOException; > > Propchange: > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/DefaultLocalPlatform.java > ------------------------------------------------------------------------------ > svn:mergeinfo = > > Copied: > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/NoOpResolver.java > (from r922335, > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/NoOpResolver.java) > URL: > http://svn.apache.org/viewvc/incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/NoOpResolver.java?p2=incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/NoOpResolver.java&p1=incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/NoOpResolver.java&r1=922335&r2=922381&rev=922381&view=diff > ============================================================================== > --- > incubator/aries/trunk/application/application-runtime/src/main/java/org/apache/aries/application/runtime/defaults/NoOpResolver.java > (original) > +++ > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/NoOpResolver.java > Fri Mar 12 18:30:33 2010 > @@ -16,7 +16,7 @@ > * specific language governing permissions and limitations > * under the License. > */ > -package org.apache.aries.application.runtime.defaults; > +package org.apache.aries.application.utils.management; > > import java.util.Set; > > > Propchange: > incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/utils/management/NoOpResolver.java > ------------------------------------------------------------------------------ > svn:mergeinfo = > > > -- Alasdair Nottingham [email protected]
