Agreed, I specifically put it in runtime for that reason. On 12 March 2010 22:48, Alasdair Nottingham <[email protected]> wrote:
> 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] >
