${geronimoVersion} is guaranteed to work like "2.1-SNAPSHOT". There
is no such guarantee for ${version}, i.e. it may evaluate to 2.1-nnnn.

Thanks
Anita

--- David Jencks <[EMAIL PROTECTED]> wrote:

> 
> On Dec 4, 2007, at 6:26 PM, Anita Kulshreshtha wrote:
> 
> >   Is the <version> temporary? Could you have used geronimoVersion
> > property instead of 2.1-SNAPSHOT?
> 
> I hope I can get rid of the version element again.  I'm hoping to fix
>  
> up the plugin repo code so that file system plugin repositories work 
> 
> at which point a lot of explicit versions won't be required.  If that
>  
> doesn't work then all the 2.1-SNAPSHOTS need to be turned into $ 
> {version} or ${geronimoVersion}.  I'm not sure which would work
> better?
> 
> thanks
> david jencks
> >
> > Thanks
> > Anita
> >
> > --- [EMAIL PROTECTED] wrote:
> >
> >> Modified:
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > ExplicitDefaultArtifactResolver.java
> >> URL:
> >>
> > http://svn.apache.org/viewvc/geronimo/server/trunk/framework/ 
> > modules/geronimo-system/src/main/java/org/apache/geronimo/system/ 
> > resolver/ExplicitDefaultArtifactResolver.java? 
> > rev=601152&r1=601151&r2=601152&view=diff
> >>
> >
>
======================================================================
> 
> > ========
> >> ---
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > ExplicitDefaultArtifactResolver.java
> >> (original)
> >> +++
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > ExplicitDefaultArtifactResolver.java
> >> Tue Dec  4 15:49:03 2007
> >> @@ -38,14 +38,14 @@
> >>  /**
> >>   * @version $Rev$ $Date$
> >>   */
> >> -public class ExplicitDefaultArtifactResolver extends
> >> DefaultArtifactResolver implements AliasedArtifactResolver {
> >> +public class ExplicitDefaultArtifactResolver extends
> >> DefaultArtifactResolver implements LocalAliasedArtifactResolver {
> >>      private static final String COMMENT = "#You can use this file
> to
> >> indicate that you want to substitute one module for another.\n" +
> >>              "#format is oldartifactid=newartifactId e.g.\n" +
> >>
> >>
> > "#org.apache.geronimo.configs/transaction// 
> >
> car=org.apache.geronimo.configs/transaction-jta11/1.2-SNAPSHOT/car\n"
> >> +
> >>              "#versions can be ommitted on the left side but not
> the
> >> right.\n" +
> >>              "#This can also specify explicit versions in the same
> >> format.";
> >>
> >> -    private final String versionMapLocation;
> >> +    private final String artifactAliasesFile;
> >>      private final ServerInfo serverInfo;
> >>
> >>      public ExplicitDefaultArtifactResolver(String
> >> versionMapLocation,
> >> @@ -53,10 +53,15 @@
> >>              Collection<? extends ListableRepository>
> repositories,
> >>              ServerInfo serverInfo ) throws IOException {
> >>          super(artifactManager, repositories,
> >> buildExplicitResolution(versionMapLocation, serverInfo));
> >> -        this.versionMapLocation = versionMapLocation;
> >> +        this.artifactAliasesFile = versionMapLocation;
> >>          this.serverInfo = serverInfo;
> >>      }
> >>
> >> +
> >> +    public String getArtifactAliasesFile() {
> >> +        return artifactAliasesFile;
> >> +    }
> >> +
> >>      private static Map<Artifact, Artifact>
> >> buildExplicitResolution(String versionMapLocation, ServerInfo
> >> serverInfo) throws IOException {
> >>          if (versionMapLocation == null) {
> >>              return null;
> >> @@ -123,7 +128,7 @@
> >>      public synchronized void addAliases(Properties properties)
> >> throws IOException {
> >>          Map<Artifact, Artifact> explicitResolutions =
> >> propertiesToArtifactMap(properties);
> >>          getExplicitResolution().putAll(explicitResolutions);
> >> -        saveExplicitResolution(getExplicitResolution(),
> >> versionMapLocation, serverInfo);
> >> +        saveExplicitResolution(getExplicitResolution(),
> >> artifactAliasesFile, serverInfo);
> >>      }
> >>
> >>      public static final GBeanInfo GBEAN_INFO;
> >>
> >> Added:
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > LocalAliasedArtifactResolver.java
> >> URL:
> >>
> > http://svn.apache.org/viewvc/geronimo/server/trunk/framework/ 
> > modules/geronimo-system/src/main/java/org/apache/geronimo/system/ 
> > resolver/LocalAliasedArtifactResolver.java?rev=601152&view=auto
> >>
> >
>
======================================================================
> 
> > ========
> >> ---
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > LocalAliasedArtifactResolver.java
> >> (added)
> >> +++
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > LocalAliasedArtifactResolver.java
> >> Tue Dec  4 15:49:03 2007
> >> @@ -0,0 +1,28 @@
> >> +/*
> >> + * Licensed to the Apache Software Foundation (ASF) under one
> >> + * or more contributor license agreements.  See the NOTICE file
> >> + * distributed with this work for additional information
> >> + * regarding copyright ownership.  The ASF licenses this file
> >> + * to you under the Apache License, Version 2.0 (the
> >> + * "License"); you may not use this file except in compliance
> >> + * with the License.  You may obtain a copy of the License at
> >> + *
> >> + *  http://www.apache.org/licenses/LICENSE-2.0
> >> + *
> >> + * Unless required by applicable law or agreed to in writing,
> >> + * software distributed under the License is distributed on an
> >> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> >> + * KIND, either express or implied.  See the License for the
> >> + * specific language governing permissions and limitations
> >> + * under the License.
> >> + */
> >> +
> >> +
> >> +package org.apache.geronimo.system.resolver;
> >> +
> >> +/**
> >> + * @version $Rev:$ $Date:$
> >> + */
> >> +public interface LocalAliasedArtifactResolver extends
> >> AliasedArtifactResolver {
> >> +    String getArtifactAliasesFile();
> >> +}
> >>
> >> Propchange:
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > LocalAliasedArtifactResolver.java
> >>
> >
>
----------------------------------------------------------------------
> 
> > --------
> >>     svn:eol-style = native
> >>
> >> Propchange:
> >>
> > geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
> > java/org/apache/geronimo/system/resolver/ 
> > LocalAliasedArtifactResolver.java
> >>
> >
>
----------------------------------------------------------------------
> 
> > --------
> >>     svn:keywords = Date Revision
> >>
> >> Propchange:
> >>
> 
=== message truncated ===



      
____________________________________________________________________________________
Looking for last minute shopping deals?  
Find them fast with Yahoo! Search.  
http://tools.search.yahoo.com/newsearch/category.php?category=shopping

Reply via email to