Author: brett
Date: Tue Sep  6 02:52:37 2005
New Revision: 278961

URL: http://svn.apache.org/viewcvs?rev=278961&view=rev
Log:
PR: MNG-613
make sure versions are aggregated

Modified:
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/LatestArtifactMetadata.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/ReleaseArtifactMetadata.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/SnapshotArtifactMetadata.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/ArtifactRepositoryMetadata.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/GroupRepositoryMetadata.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/SnapshotArtifactRepositoryMetadata.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/AbstractVersionTransformation.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/LatestArtifactTransformation.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/ReleaseArtifactTransformation.java
    
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/Artifact.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/DefaultArtifact.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadata.java
    
maven/components/trunk/maven-plugins/maven-deploy-plugin/src/main/java/org/apache/maven/plugin/deploy/DeployMojo.java
    
maven/components/trunk/maven-plugins/maven-install-plugin/src/main/java/org/apache/maven/plugin/install/InstallMojo.java
    
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ActiveProjectArtifact.java
    
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ProjectArtifactMetadata.java

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/LatestArtifactMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/LatestArtifactMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/LatestArtifactMetadata.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/LatestArtifactMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -116,23 +116,4 @@
         return false;
     }
 
-    public int getBuildNumber()
-    {
-        return 0;
-    }
-
-    public String getTimestamp()
-    {
-        return null;
-    }
-
-    public String getLatestVersion()
-    {
-        return null;
-    }
-
-    public String getReleaseVersion()
-    {
-        return null;
-    }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/ReleaseArtifactMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/ReleaseArtifactMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/ReleaseArtifactMetadata.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/ReleaseArtifactMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -123,23 +123,4 @@
         return false;
     }
 
-    public int getBuildNumber()
-    {
-        return 0;
-    }
-
-    public String getTimestamp()
-    {
-        return null;
-    }
-
-    public String getLatestVersion()
-    {
-        return null;
-    }
-
-    public String getReleaseVersion()
-    {
-        return null;
-    }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/SnapshotArtifactMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/SnapshotArtifactMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/SnapshotArtifactMetadata.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/metadata/SnapshotArtifactMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -97,11 +97,6 @@
         }
     }
 
-    public String getTimestamp()
-    {
-        return timestamp;
-    }
-
     public int getBuildNumber()
     {
         return buildNumber;
@@ -180,13 +175,4 @@
         return artifact.getBaseVersion();
     }
 
-    public String getLatestVersion()
-    {
-        return null;
-    }
-
-    public String getReleaseVersion()
-    {
-        return null;
-    }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/ArtifactRepositoryMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/ArtifactRepositoryMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/ArtifactRepositoryMetadata.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/ArtifactRepositoryMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -207,23 +207,4 @@
         return null;
     }
 
-    public String getLatestVersion()
-    {
-        return versioning.getLatest();
-    }
-
-    public String getReleaseVersion()
-    {
-        return versioning.getRelease();
-    }
-
-    public int getBuildNumber()
-    {
-        return 0;
-    }
-
-    public String getTimestamp()
-    {
-        return null;
-    }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/GroupRepositoryMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/GroupRepositoryMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/GroupRepositoryMetadata.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/GroupRepositoryMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -200,23 +200,4 @@
         return false;
     }
 
-    public int getBuildNumber()
-    {
-        return 0;
-    }
-
-    public String getTimestamp()
-    {
-        return null;
-    }
-
-    public String getLatestVersion()
-    {
-        return null;
-    }
-
-    public String getReleaseVersion()
-    {
-        return null;
-    }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/SnapshotArtifactRepositoryMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/SnapshotArtifactRepositoryMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/SnapshotArtifactRepositoryMetadata.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/repository/metadata/SnapshotArtifactRepositoryMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -196,23 +196,4 @@
         return artifact.isSnapshot();
     }
 
-    public int getBuildNumber()
-    {
-        return snapshot != null ? snapshot.getBuildNumber() : 0;
-    }
-
-    public String getTimestamp()
-    {
-        return snapshot != null ? snapshot.getTimestamp() : null;
-    }
-
-    public String getLatestVersion()
-    {
-        return null;
-    }
-
-    public String getReleaseVersion()
-    {
-        return null;
-    }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/AbstractVersionTransformation.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/AbstractVersionTransformation.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/AbstractVersionTransformation.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/AbstractVersionTransformation.java
 Tue Sep  6 02:52:37 2005
@@ -21,13 +21,13 @@
 import org.apache.maven.artifact.metadata.ArtifactMetadata;
 import org.apache.maven.artifact.metadata.ArtifactMetadataRetrievalException;
 import org.apache.maven.artifact.metadata.LegacyArtifactMetadata;
-import org.apache.maven.artifact.metadata.SnapshotArtifactMetadata;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.artifact.repository.ArtifactRepositoryPolicy;
 import 
org.apache.maven.artifact.repository.metadata.ArtifactRepositoryMetadata;
 import org.apache.maven.artifact.repository.metadata.Metadata;
 import org.apache.maven.artifact.repository.metadata.RepositoryMetadataManager;
 import 
org.apache.maven.artifact.repository.metadata.SnapshotArtifactRepositoryMetadata;
+import org.apache.maven.artifact.repository.metadata.Versioning;
 import 
org.apache.maven.artifact.repository.metadata.io.xpp3.MetadataXpp3Reader;
 import org.apache.maven.wagon.ResourceDoesNotExistException;
 import org.codehaus.plexus.logging.AbstractLogEnabled;
@@ -69,7 +69,7 @@
     {
         // TODO: can we improve on this?
         ArtifactMetadata metadata = null;
-        if ( artifact.isSnapshot() )
+        if ( !artifact.isSnapshot() )
         {
             metadata = new ArtifactRepositoryMetadata( artifact );
         }
@@ -94,23 +94,23 @@
 
         String version = selectVersion( versioning, artifact.getVersion() );
 */
-        ArtifactMetadata localMetadata = null;
+        Versioning versioning = null;
         for ( Iterator i = remoteRepositories.iterator(); i.hasNext(); )
         {
             ArtifactRepository repository = (ArtifactRepository) i.next();
 
-            localMetadata = loadVersioningInformation( metadata, repository, 
localRepository, artifact );
-            if ( localMetadata != null )
+            versioning = loadVersioningInformation( metadata, repository, 
localRepository, artifact );
+            if ( versioning != null )
             {
                 artifact.setRepository( repository );
                 // TODO: merge instead (see above)
                 break;
             }
         }
-        ArtifactMetadata m = loadVersioningInformation( metadata, 
localRepository, localRepository, artifact );
-        if ( m != null )
+        Versioning v = loadVersioningInformation( metadata, localRepository, 
localRepository, artifact );
+        if ( v != null )
         {
-            localMetadata = m;
+            versioning = v;
             // TODO: figure out way to avoid duplicated message
             if ( getLogger().isDebugEnabled() /*&& !alreadyResolved*/ )
             {
@@ -120,9 +120,9 @@
         }
 
         String version = null;
-        if ( localMetadata != null )
+        if ( versioning != null )
         {
-            version = constructVersion( localMetadata );
+            version = constructVersion( versioning, artifact.getBaseVersion() 
);
         }
 
         if ( version == null )
@@ -154,43 +154,7 @@
         return version;
     }
 
-    protected int resolveLatestSnapshotBuildNumber( Artifact artifact, 
ArtifactRepository localRepository,
-                                                    ArtifactRepository 
remoteRepository )
-        throws ArtifactMetadataRetrievalException
-    {
-        // TODO: can we improve on this?
-        ArtifactMetadata metadata = new SnapshotArtifactRepositoryMetadata( 
artifact );
-
-        getLogger().info( "Retrieving previous build number from " + 
remoteRepository.getId() );
-        repositoryMetadataManager.resolveAlways( metadata, localRepository, 
remoteRepository );
-
-        ArtifactMetadata m = loadVersioningInformation( metadata, 
remoteRepository, localRepository, artifact );
-        int buildNumber = 0;
-        if ( m == null )
-        {
-            try
-            {
-                SnapshotArtifactMetadata snapshotMetadata = new 
SnapshotArtifactMetadata( artifact );
-                snapshotMetadata.retrieveFromRemoteRepository( 
remoteRepository, wagonManager,
-                                                               
ArtifactRepositoryPolicy.CHECKSUM_POLICY_WARN );
-                getLogger().warn( "Using old-style versioning metadata from 
remote repo for " + artifact );
-
-                buildNumber = snapshotMetadata.getBuildNumber();
-            }
-            catch ( ResourceDoesNotExistException e1 )
-            {
-                // safe to ignore, use default snapshot data
-                getLogger().debug( "Unable to find legacy metadata - ignoring" 
);
-            }
-        }
-        else
-        {
-            buildNumber = m.getBuildNumber();
-        }
-        return buildNumber;
-    }
-
-    protected abstract String constructVersion( ArtifactMetadata metadata );
+    protected abstract String constructVersion( Versioning versioning, String 
baseVersion );
 
 /* TODO
     private void mergeVersioning( Versioning dest, Versioning source )
@@ -349,35 +313,20 @@
         return artifact.getGroupId() + ":" + artifact.getArtifactId() + ":" + 
artifact.getBaseVersion();
     }
 
-    private ArtifactMetadata loadVersioningInformation( ArtifactMetadata 
repoMetadata,
-                                                        ArtifactRepository 
remoteRepository,
-                                                        ArtifactRepository 
localRepository, Artifact artifact )
+    protected Versioning loadVersioningInformation( ArtifactMetadata 
repoMetadata, ArtifactRepository remoteRepository,
+                                                    ArtifactRepository 
localRepository, Artifact artifact )
         throws ArtifactMetadataRetrievalException
     {
         File metadataFile = new File( localRepository.getBasedir(),
                                       
localRepository.pathOfLocalRepositoryMetadata( repoMetadata, remoteRepository ) 
);
 
-        ArtifactMetadata newMetadata = null;
+        Versioning versioning = null;
         if ( metadataFile.exists() )
         {
             Metadata metadata = readMetadata( metadataFile );
-            if ( metadata.getVersioning() != null )
-            {
-                if ( artifact.isSnapshot() )
-                {
-                    if ( metadata.getVersioning().getSnapshot() != null )
-                    {
-                        newMetadata = new SnapshotArtifactRepositoryMetadata( 
artifact,
-                                                                              
metadata.getVersioning().getSnapshot() );
-                    }
-                }
-                else
-                {
-                    newMetadata = new ArtifactRepositoryMetadata( artifact, 
metadata.getVersioning() );
-                }
-            }
+            versioning = metadata.getVersioning();
         }
-        return newMetadata;
+        return versioning;
     }
 
     /**

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/LatestArtifactTransformation.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/LatestArtifactTransformation.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/LatestArtifactTransformation.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/LatestArtifactTransformation.java
 Tue Sep  6 02:52:37 2005
@@ -68,8 +68,8 @@
         return metadata;
     }
 
-    protected String constructVersion( ArtifactMetadata metadata )
+    protected String constructVersion( Versioning versioning, String 
baseVersion )
     {
-        return metadata.getLatestVersion();
+        return versioning.getLatest();
     }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/ReleaseArtifactTransformation.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/ReleaseArtifactTransformation.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/ReleaseArtifactTransformation.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/ReleaseArtifactTransformation.java
 Tue Sep  6 02:52:37 2005
@@ -22,6 +22,8 @@
 import org.apache.maven.artifact.metadata.LegacyArtifactMetadata;
 import org.apache.maven.artifact.metadata.ReleaseArtifactMetadata;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import 
org.apache.maven.artifact.repository.metadata.ArtifactRepositoryMetadata;
+import org.apache.maven.artifact.repository.metadata.Versioning;
 
 import java.util.List;
 
@@ -54,14 +56,40 @@
     public void transformForInstall( Artifact artifact, ArtifactRepository 
localRepository )
         throws ArtifactMetadataRetrievalException
     {
-        // metadata is added at install time
+        Versioning versioning = new Versioning();
+        versioning.addVersion( artifact.getVersion() );
+
+        if ( artifact.isRelease() )
+        {
+            versioning.setRelease( artifact.getVersion() );
+        }
+
+        // TODO: need to create?
+        ArtifactMetadata metadata = new ArtifactRepositoryMetadata( artifact, 
versioning );
+
+        artifact.addMetadata( metadata );
     }
 
     public void transformForDeployment( Artifact artifact, ArtifactRepository 
remoteRepository,
                                         ArtifactRepository localRepository )
         throws ArtifactMetadataRetrievalException
     {
-        // metadata is added at deploy time
+        Versioning versioning = new Versioning();
+        versioning.addVersion( artifact.getVersion() );
+
+        if ( artifact.isRelease() )
+        {
+            versioning.setRelease( artifact.getVersion() );
+        }
+
+        // TODO: need to create?
+        ArtifactMetadata metadata = new ArtifactRepositoryMetadata( artifact, 
versioning );
+
+        artifact.addMetadata( metadata );
+
+        // TODO: this should be in the part that actually merges instead
+        getLogger().info( "Retrieving previous metadata from " + 
remoteRepository.getId() );
+        repositoryMetadataManager.resolveAlways( metadata, localRepository, 
remoteRepository );
     }
 
     protected LegacyArtifactMetadata createLegacyMetadata( Artifact artifact )
@@ -69,8 +97,8 @@
         return new ReleaseArtifactMetadata( artifact );
     }
 
-    protected String constructVersion( ArtifactMetadata metadata )
+    protected String constructVersion( Versioning versioning, String bS )
     {
-        return metadata.getReleaseVersion();
+        return versioning.getRelease();
     }
 }

Modified: 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java
 (original)
+++ 
maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java
 Tue Sep  6 02:52:37 2005
@@ -22,8 +22,11 @@
 import org.apache.maven.artifact.metadata.LegacyArtifactMetadata;
 import org.apache.maven.artifact.metadata.SnapshotArtifactMetadata;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.repository.ArtifactRepositoryPolicy;
 import org.apache.maven.artifact.repository.metadata.Snapshot;
 import 
org.apache.maven.artifact.repository.metadata.SnapshotArtifactRepositoryMetadata;
+import org.apache.maven.artifact.repository.metadata.Versioning;
+import org.apache.maven.wagon.ResourceDoesNotExistException;
 import org.codehaus.plexus.util.StringUtils;
 
 import java.util.Date;
@@ -78,7 +81,9 @@
 
             ArtifactMetadata metadata = new 
SnapshotArtifactRepositoryMetadata( artifact, snapshot );
 
-            artifact.setResolvedVersion( constructVersion( metadata ) );
+            Versioning versioning = new Versioning();
+            versioning.setSnapshot( snapshot );
+            artifact.setResolvedVersion( constructVersion( versioning, 
artifact.getBaseVersion() ) );
 
             artifact.addMetadata( metadata );
         }
@@ -98,21 +103,62 @@
         return new SnapshotArtifactMetadata( artifact );
     }
 
-    protected String constructVersion( ArtifactMetadata metadata )
+    protected String constructVersion( Versioning versioning, String 
baseVersion )
     {
-        String version = metadata.getBaseVersion();
-        if ( metadata.getTimestamp() != null && metadata.getBuildNumber() > 0 )
+        String version = baseVersion;
+        Snapshot snapshot = versioning.getSnapshot();
+        if ( snapshot != null )
         {
-            String newVersion = metadata.getTimestamp() + "-" + 
metadata.getBuildNumber();
-            if ( version != null )
+            if ( snapshot.getTimestamp() != null && snapshot.getBuildNumber() 
> 0 )
             {
-                version = StringUtils.replace( version, "SNAPSHOT", newVersion 
);
+                String newVersion = snapshot.getTimestamp() + "-" + 
snapshot.getBuildNumber();
+                if ( version != null )
+                {
+                    version = StringUtils.replace( version, "SNAPSHOT", 
newVersion );
+                }
+                else
+                {
+                    version = newVersion;
+                }
             }
-            else
+        }
+        return version;
+    }
+
+    private int resolveLatestSnapshotBuildNumber( Artifact artifact, 
ArtifactRepository localRepository,
+                                                  ArtifactRepository 
remoteRepository )
+        throws ArtifactMetadataRetrievalException
+    {
+        // TODO: can we improve on this?
+        ArtifactMetadata metadata = new SnapshotArtifactRepositoryMetadata( 
artifact );
+
+        getLogger().info( "Retrieving previous build number from " + 
remoteRepository.getId() );
+        repositoryMetadataManager.resolveAlways( metadata, localRepository, 
remoteRepository );
+
+        Versioning versioning = loadVersioningInformation( metadata, 
remoteRepository, localRepository, artifact );
+        int buildNumber = 0;
+        if ( versioning == null )
+        {
+            try
+            {
+                SnapshotArtifactMetadata snapshotMetadata = new 
SnapshotArtifactMetadata( artifact );
+                snapshotMetadata.retrieveFromRemoteRepository( 
remoteRepository, wagonManager,
+                                                               
ArtifactRepositoryPolicy.CHECKSUM_POLICY_WARN );
+                getLogger().warn( "Using old-style versioning metadata from 
remote repo for " + artifact );
+
+                buildNumber = snapshotMetadata.getBuildNumber();
+            }
+            catch ( ResourceDoesNotExistException e1 )
             {
-                version = newVersion;
+                // safe to ignore, use default snapshot data
+                getLogger().debug( "Unable to find legacy metadata - ignoring" 
);
             }
         }
-        return version;
+        else if ( versioning.getSnapshot() != null )
+        {
+            buildNumber = versioning.getSnapshot().getBuildNumber();
+        }
+        return buildNumber;
     }
+
 }

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/Artifact.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/Artifact.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/Artifact.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/Artifact.java
 Tue Sep  6 02:52:37 2005
@@ -49,7 +49,7 @@
     String SCOPE_RUNTIME = "runtime";
 
     String SCOPE_PROVIDED = "provided";
-    
+
     String SCOPE_SYSTEM = "system";
 
     String getGroupId();
@@ -141,4 +141,8 @@
      * @todo remove, a quick hack for the lifecycle executor
      */
     void setArtifactHandler( ArtifactHandler handler );
+
+    boolean isRelease();
+
+    void setRelease( boolean release );
 }

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/DefaultArtifact.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/DefaultArtifact.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/DefaultArtifact.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/DefaultArtifact.java
 Tue Sep  6 02:52:37 2005
@@ -74,6 +74,8 @@
 
     private boolean resolved;
 
+    private boolean release = false;
+
     public DefaultArtifact( String groupId, String artifactId, VersionRange 
versionRange, String scope, String type,
                             String classifier, ArtifactHandler artifactHandler 
)
     {
@@ -462,4 +464,13 @@
         this.artifactHandler = artifactHandler;
     }
 
+    public void setRelease( boolean release )
+    {
+        this.release = release;
+    }
+
+    public boolean isRelease()
+    {
+        return release;
+    }
 }

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadata.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -74,11 +74,4 @@
      */
     String getRemoteFilename();
 
-    int getBuildNumber();
-
-    String getTimestamp();
-
-    String getLatestVersion();
-
-    String getReleaseVersion();
 }

Modified: 
maven/components/trunk/maven-plugins/maven-deploy-plugin/src/main/java/org/apache/maven/plugin/deploy/DeployMojo.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-deploy-plugin/src/main/java/org/apache/maven/plugin/deploy/DeployMojo.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-plugins/maven-deploy-plugin/src/main/java/org/apache/maven/plugin/deploy/DeployMojo.java
 (original)
+++ 
maven/components/trunk/maven-plugins/maven-deploy-plugin/src/main/java/org/apache/maven/plugin/deploy/DeployMojo.java
 Tue Sep  6 02:52:37 2005
@@ -21,8 +21,6 @@
 import org.apache.maven.artifact.deployer.ArtifactDeploymentException;
 import org.apache.maven.artifact.metadata.ArtifactMetadata;
 import org.apache.maven.artifact.repository.ArtifactRepository;
-import 
org.apache.maven.artifact.repository.metadata.ArtifactRepositoryMetadata;
-import org.apache.maven.artifact.repository.metadata.Versioning;
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.project.artifact.ProjectArtifactMetadata;
@@ -129,15 +127,10 @@
             artifact.addMetadata( metadata );
         }
 
-        // TODO: clean up
-        Versioning versioning = new Versioning();
-        versioning.addVersion( artifact.getVersion() );
         if ( updateReleaseInfo )
         {
-            versioning.setRelease( artifact.getVersion() );
+            artifact.setRelease( true );
         }
-        ArtifactRepositoryMetadata metadata = new ArtifactRepositoryMetadata( 
artifact, versioning );
-        artifact.addMetadata( metadata );
 
         try
         {

Modified: 
maven/components/trunk/maven-plugins/maven-install-plugin/src/main/java/org/apache/maven/plugin/install/InstallMojo.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-install-plugin/src/main/java/org/apache/maven/plugin/install/InstallMojo.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-plugins/maven-install-plugin/src/main/java/org/apache/maven/plugin/install/InstallMojo.java
 (original)
+++ 
maven/components/trunk/maven-plugins/maven-install-plugin/src/main/java/org/apache/maven/plugin/install/InstallMojo.java
 Tue Sep  6 02:52:37 2005
@@ -19,8 +19,6 @@
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.installer.ArtifactInstallationException;
 import org.apache.maven.artifact.metadata.ArtifactMetadata;
-import 
org.apache.maven.artifact.repository.metadata.ArtifactRepositoryMetadata;
-import org.apache.maven.artifact.repository.metadata.Versioning;
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.project.artifact.ProjectArtifactMetadata;
 
@@ -88,6 +86,7 @@
     public void execute()
         throws MojoExecutionException
     {
+        // TODO: push into transformation
         boolean isPomArtifact = "pom".equals( packaging );
 
         File pom = new File( basedir, "pom.xml" );
@@ -97,15 +96,10 @@
             artifact.addMetadata( metadata );
         }
 
-        // TODO: clean up
-        Versioning versioning = new Versioning();
-        versioning.addVersion( artifact.getVersion() );
         if ( updateReleaseInfo )
         {
-            versioning.setRelease( artifact.getVersion() );
+            artifact.setRelease( true );
         }
-        ArtifactRepositoryMetadata metadata = new ArtifactRepositoryMetadata( 
artifact, versioning );
-        artifact.addMetadata( metadata );
 
         try
         {

Modified: 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ActiveProjectArtifact.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ActiveProjectArtifact.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ActiveProjectArtifact.java
 (original)
+++ 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ActiveProjectArtifact.java
 Tue Sep  6 02:52:37 2005
@@ -250,4 +250,14 @@
     {
         return "active project artifact:\n\tartifact = " + artifact + 
";\n\tproject: " + project;
     }
+
+    public boolean isRelease()
+    {
+        return artifact.isRelease();
+    }
+
+    public void setRelease( boolean release )
+    {
+        artifact.setResolved( release );
+    }
 }

Modified: 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ProjectArtifactMetadata.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ProjectArtifactMetadata.java?rev=278961&r1=278960&r2=278961&view=diff
==============================================================================
--- 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ProjectArtifactMetadata.java
 (original)
+++ 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/ProjectArtifactMetadata.java
 Tue Sep  6 02:52:37 2005
@@ -146,23 +146,4 @@
         return artifact.isSnapshot();
     }
 
-    public String getLatestVersion()
-    {
-        return null;
-    }
-
-    public String getReleaseVersion()
-    {
-        return null;
-    }
-
-    public int getBuildNumber()
-    {
-        return 0;
-    }
-
-    public String getTimestamp()
-    {
-        return null;
-    }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to