You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ep...@apache.org on 2005/12/01 09:28:23 UTC
svn commit: r350205 - in
/maven/repository-manager/trunk/maven-repository-reports-standard/src/main:
java/org/apache/maven/repository/
java/org/apache/maven/repository/reporting/ resources/META-INF/plexus/
Author: epunzalan
Date: Thu Dec 1 00:28:12 2005
New Revision: 350205
URL: http://svn.apache.org/viewcvs?rev=350205&view=rev
Log:
PR: MRM-16
Still incomplete. Requires unit tests, waiting for abstract unit test.
Added:
maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/RepositoryFileFilter.java (with props)
maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/BadMetadataReporter.java (with props)
Modified:
maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/ArtifactReporter.java
maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/DefaultArtifactReporter.java
maven/repository-manager/trunk/maven-repository-reports-standard/src/main/resources/META-INF/plexus/components.xml
Added: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/RepositoryFileFilter.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/RepositoryFileFilter.java?rev=350205&view=auto
==============================================================================
--- maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/RepositoryFileFilter.java (added)
+++ maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/RepositoryFileFilter.java Thu Dec 1 00:28:12 2005
@@ -0,0 +1,43 @@
+package org.apache.maven.repository;
+
+/*
+ * Copyright 2001-2005 The Apache Software Foundation.
+ *
+ * Licensed 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.
+ */
+
+/**
+ * This class is used to ignore several files that may be present inside the repository so that the other classes
+ * may not worry about them and then can concentrate on doing their tasks.
+ *
+ */
+public class RepositoryFileFilter implements java.io.FileFilter
+{
+ public boolean accept(java.io.File pathname)
+ {
+ if ( pathname.isDirectory() )
+ {
+ if ( ".svn".equals( pathname.getName() ) ) return false;
+ if ( "CVS".equals( pathname.getName() ) ) return false;
+ }
+ else
+ {
+ String name = pathname.getName();
+ if ( name.endsWith( ".md5" ) ) return false;
+ if ( name.endsWith( ".sha1" ) ) return false;
+ }
+
+ return true;
+ }
+}
Propchange: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/RepositoryFileFilter.java
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/RepositoryFileFilter.java
------------------------------------------------------------------------------
svn:keywords = "Author Date Id Revision"
Modified: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/ArtifactReporter.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/ArtifactReporter.java?rev=350205&r1=350204&r2=350205&view=diff
==============================================================================
--- maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/ArtifactReporter.java (original)
+++ maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/ArtifactReporter.java Thu Dec 1 00:28:12 2005
@@ -17,6 +17,7 @@
*/
import org.apache.maven.artifact.Artifact;
+import org.apache.maven.artifact.repository.metadata.RepositoryMetadata;
/**
* This interface is used by the single artifact processor.
@@ -34,4 +35,10 @@
void addSuccess( Artifact artifact );
void addWarning( Artifact artifact, String message );
+
+ void addFailure( RepositoryMetadata metadata, String reason );
+
+ void addSuccess( RepositoryMetadata metadata );
+
+ void addWarning( RepositoryMetadata metadata, String message );
}
Added: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/BadMetadataReporter.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/BadMetadataReporter.java?rev=350205&view=auto
==============================================================================
--- maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/BadMetadataReporter.java (added)
+++ maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/BadMetadataReporter.java Thu Dec 1 00:28:12 2005
@@ -0,0 +1,194 @@
+package org.apache.maven.repository.reporting;
+
+/*
+ * Copyright 2001-2005 The Apache Software Foundation.
+ *
+ * Licensed 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.
+ */
+
+import java.io.File;
+import java.util.Iterator;
+import org.apache.maven.artifact.Artifact;
+import org.apache.maven.artifact.factory.ArtifactFactory;
+import org.apache.maven.artifact.manager.WagonManager;
+import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.repository.metadata.RepositoryMetadata;
+import org.apache.maven.artifact.repository.metadata.Snapshot;
+import org.apache.maven.artifact.repository.metadata.Versioning;
+import org.apache.maven.repository.RepositoryFileFilter;
+import org.apache.maven.wagon.ResourceDoesNotExistException;
+import org.apache.maven.wagon.TransferFailedException;
+
+/**
+ * This class will report on bad metadata files. These include invalid version declarations and incomplete version
+ * information inside the metadata file. Plugin metadata will be checked for validity of the latest plugin artifacts.
+ *
+ */
+public class BadMetadataReporter implements MetadataReportProcessor
+{
+ private WagonManager wagon;
+ private ArtifactFactory artifactFactory;
+
+ public void processMetadata( RepositoryMetadata metadata, ArtifactRepository repository, ArtifactReporter reporter )
+ {
+ boolean hasFailures = false;
+
+ String lastUpdated = metadata.getMetadata().getVersioning().getLastUpdated();
+ if ( lastUpdated == null || lastUpdated.length() == 0 )
+ {
+ reporter.addFailure( metadata, "Missing lastUpdated element inside the metadata." );
+ hasFailures = true;
+ }
+
+ if ( metadata.storedInGroupDirectory() )
+ {
+ checkPluginMetadata( metadata, repository, reporter );
+ }
+ else if ( metadata.storedInArtifactVersionDirectory() )
+ {
+ //snapshot metadata
+ }
+ else
+ {
+ if ( !checkMetadataVersions( metadata, repository, reporter ) ) hasFailures = true;
+
+ if ( checkRepositoryVersions( metadata, repository, reporter ) ) hasFailures = true;
+ }
+
+ if ( !hasFailures ) reporter.addSuccess( metadata );
+ }
+
+ /**
+ * Checks the plugin metadata
+ */
+ public boolean checkPluginMetadata( RepositoryMetadata metadata, ArtifactRepository repository,
+ ArtifactReporter reporter )
+ {
+ boolean hasFailures = false;
+
+
+
+ return hasFailures;
+ }
+
+ /**
+ * Checks the snapshot metadata
+ */
+ public boolean checkSnapshotMetadata( RepositoryMetadata metadata, ArtifactRepository repository,
+ ArtifactReporter reporter )
+ {
+ boolean hasFailures = false;
+
+ Snapshot snapshot = metadata.getMetadata().getVersioning().getSnapshot();
+ String timestamp = snapshot.getTimestamp();
+ String buildNumber = String.valueOf( snapshot.getBuildNumber() );
+ String artifactName = metadata.getArtifactId() + "-" + timestamp + "-" + buildNumber + ".pom";
+
+ //@todo use wagon instead
+ Artifact artifact = createArtifact( metadata );
+ File artifactFile = new File ( repository.pathOf( artifact ) );
+ File snapshotFile = new File( artifactFile.getParentFile(), artifactName );
+ if ( !snapshotFile.exists() )
+ {
+ reporter.addFailure( metadata, "Snapshot artifact " + artifactName + " does not exist." );
+ hasFailures = true;
+ }
+
+ return hasFailures;
+ }
+
+ /**
+ * Checks the declared metadata versions if the artifacts are present in the repository
+ */
+ public boolean checkMetadataVersions( RepositoryMetadata metadata, ArtifactRepository repository,
+ ArtifactReporter reporter )
+ {
+ boolean hasFailures = false;
+ Versioning versioning = metadata.getMetadata().getVersioning();
+ for ( Iterator versions = versioning.getVersions().iterator(); versions.hasNext(); )
+ {
+ String version = (String) versions.next();
+
+ Artifact artifact = createArtifact( metadata, version );
+
+ try
+ {
+ wagon.getArtifact( artifact, repository );
+ }
+ catch ( TransferFailedException e )
+ {
+ reporter.addWarning( artifact, "An error occurred during the transfer of the artifact in " +
+ "the repository." );
+ }
+ catch ( ResourceDoesNotExistException e )
+ {
+ //do nothing, will check later that this artifact has not been resolved
+ }
+
+ if ( !artifact.isResolved() )
+ {
+ reporter.addFailure( metadata, "Artifact version " + version + " is present in metadata but " +
+ "missing in the repository." );
+ if ( !hasFailures ) hasFailures = true;
+ }
+ }
+ return hasFailures;
+ }
+
+ /**
+ * Searches the artifact repository directory for all versions and verifies that all of them are listed in the
+ * metadata file.
+ */
+ public boolean checkRepositoryVersions( RepositoryMetadata metadata, ArtifactRepository repository,
+ ArtifactReporter reporter )
+ {
+ boolean hasFailures = false;
+ Versioning versioning = metadata.getMetadata().getVersioning();
+ String repositoryPath = repository.getBasedir();
+ File versionsDir = new File( repositoryPath, formatAsDirectory( metadata.getGroupId() ) +
+ File.pathSeparator + metadata.getArtifactId() );
+ File[] versions = versionsDir.listFiles( new RepositoryFileFilter() );
+ for( int idx=0; idx<versions.length; idx++ )
+ {
+ String version = versions[ idx ].getName();
+ if ( !versioning.getVersions().contains( version ) )
+ {
+ reporter.addFailure( metadata, "Artifact version " + version + " found in the repository but " +
+ "missing in the metadata." );
+ if ( !hasFailures ) hasFailures = true;
+ }
+ }
+ return hasFailures;
+ }
+
+ /**
+ * Formats an artifact groupId to the directory structure format used for storage in repositories
+ */
+ private String formatAsDirectory( String directory )
+ {
+ return directory.replace( '.', File.pathSeparatorChar );
+ }
+
+ private Artifact createArtifact( RepositoryMetadata metadata )
+ {
+ return artifactFactory.createBuildArtifact( metadata.getGroupId(), metadata.getArtifactId(),
+ metadata.getBaseVersion(), "pom" );
+ }
+
+ private Artifact createArtifact( RepositoryMetadata metadata, String version )
+ {
+ return artifactFactory.createBuildArtifact( metadata.getGroupId(), metadata.getArtifactId(),
+ version, "pom" );
+ }
+}
Propchange: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/BadMetadataReporter.java
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/BadMetadataReporter.java
------------------------------------------------------------------------------
svn:keywords = "Author Date Id Revision"
Modified: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/DefaultArtifactReporter.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/DefaultArtifactReporter.java?rev=350205&r1=350204&r2=350205&view=diff
==============================================================================
--- maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/DefaultArtifactReporter.java (original)
+++ maven/repository-manager/trunk/maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/DefaultArtifactReporter.java Thu Dec 1 00:28:12 2005
@@ -1,10 +1,5 @@
package org.apache.maven.repository.reporting;
-import org.apache.maven.repository.discovery.AbstractArtifactDiscoverer;
-import org.apache.maven.artifact.Artifact;
-
-import java.util.List;
-
/*
* Copyright 2001-2005 The Apache Software Foundation.
*
@@ -21,6 +16,7 @@
* limitations under the License.
*/
+import org.apache.maven.artifact.Artifact;
/**
* @author <a href="mailto:jtolentino@mergere.com">John Tolentino</a>
@@ -38,6 +34,18 @@
}
public void addWarning( Artifact artifact, String message )
+ {
+ }
+
+ public void addWarning(org.apache.maven.artifact.repository.metadata.RepositoryMetadata metadata, String message)
+ {
+ }
+
+ public void addFailure(org.apache.maven.artifact.repository.metadata.RepositoryMetadata metadata, String reason)
+ {
+ }
+
+ public void addSuccess(org.apache.maven.artifact.repository.metadata.RepositoryMetadata metadata)
{
}
}
Modified: maven/repository-manager/trunk/maven-repository-reports-standard/src/main/resources/META-INF/plexus/components.xml
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-reports-standard/src/main/resources/META-INF/plexus/components.xml?rev=350205&r1=350204&r2=350205&view=diff
==============================================================================
--- maven/repository-manager/trunk/maven-repository-reports-standard/src/main/resources/META-INF/plexus/components.xml (original)
+++ maven/repository-manager/trunk/maven-repository-reports-standard/src/main/resources/META-INF/plexus/components.xml Thu Dec 1 00:28:12 2005
@@ -12,5 +12,18 @@
<implementation>org.apache.maven.repository.reporting.DefaultRepositoryQueryLayer</implementation>
<instantiation-strategy>per-lookup</instantiation-strategy>
</component>
+ <component>
+ <role>org.apache.maven.repository.reporting.BadMetadataReporter</role>
+ <role-hint>default</role-hint>
+ <implementation>org.apache.maven.repository.reporting.BadMetadataReporter</implementation>
+ <requirements>
+ <requirement>
+ <role>org.apache.maven.artifact.manager.WagonManager</role>
+ </requirement>
+ <requirement>
+ <role>org.apache.maven.artifact.factory.ArtifactFactory</role>
+ </requirement>
+ </requirements>
+ </component>
</components>
</component-set>