Author: hboutemy
Date: Thu Jul 11 23:01:07 2013
New Revision: 1502390
URL: http://svn.apache.org/r1502390
Log:
added src+bin format
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ConfigurationLineInfo.java
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/resources/db/mavendb.txt
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/test/java/org/apache/maven/dist/tools/PatternTest.java
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ConfigurationLineInfo.java
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ConfigurationLineInfo.java?rev=1502390&r1=1502389&r2=1502390&view=diff
==============================================================================
---
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ConfigurationLineInfo.java
(original)
+++
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ConfigurationLineInfo.java
Thu Jul 11 23:01:07 2013
@@ -22,6 +22,8 @@ package org.apache.maven.dist.tools;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.Date;
+import java.util.LinkedList;
+import java.util.List;
import org.apache.maven.artifact.repository.metadata.Metadata;
@@ -33,8 +35,9 @@ class ConfigurationLineInfo
{
private static final String URLSEP = "/";
- private final String groupId;
private final String directory;
+ private final String groupId;
+ private final boolean srcBin;
private final String artifactId;
private final String forceVersion;
@@ -45,6 +48,7 @@ class ConfigurationLineInfo
{
this.directory = infos[0];
this.groupId = infos[1];
+ this.srcBin = ( infos.length > 2 ) && "src+bin".equals( infos[2] );
this.artifactId = null;
this.forceVersion = null;
@@ -54,6 +58,7 @@ class ConfigurationLineInfo
{
this.directory = group.getDirectory();
this.groupId = group.getGroupId();
+ this.srcBin = group.isSrcBin();
this.artifactId = infos[0];
this.forceVersion = ( infos.length > 1 ) ? infos[1] : null;
@@ -88,6 +93,11 @@ class ConfigurationLineInfo
return directory;
}
+ public boolean isSrcBin()
+ {
+ return srcBin;
+ }
+
String getBaseURL( String repoBaseUrl, String folder )
{
return repoBaseUrl + groupId.replaceAll( "\\.", URLSEP ) + URLSEP +
artifactId + URLSEP + folder;
@@ -129,4 +139,22 @@ class ConfigurationLineInfo
}
}
+
+ String getSourceReleaseFilename( String version, boolean dist )
+ {
+ return artifactId + "-" + version
+ + ( srcBin && ( dist || !"maven-ant-tasks".equals( artifactId ) )
? "-src" : "-source-release" ) + ".zip";
+ }
+
+ List<String> getExpectedFilenames( String version, boolean dist )
+ {
+ String sourceReleaseFilename = getSourceReleaseFilename( version, dist
);
+
+ List<String> expectedFiles = new LinkedList<>();
+ expectedFiles.add( sourceReleaseFilename );
+ expectedFiles.add( sourceReleaseFilename + ".asc" );
+ expectedFiles.add( sourceReleaseFilename + ".md5" );
+
+ return expectedFiles;
+ }
}
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java?rev=1502390&r1=1502389&r2=1502390&view=diff
==============================================================================
---
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java
(original)
+++
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java
Thu Jul 11 23:01:07 2013
@@ -189,8 +189,9 @@ public class DistCheckSourceReleaseMojo
// dist column
sink.tableCell();
- sink.link( DIST_AREA + cli.getDirectory() );
- sink.text( cli.getDirectory() );
+ String directory = cli.getDirectory() + ( cli.isSrcBin() ? ( "/" +
csr.getVersion() + "/source" ) : "" );
+ sink.link( DIST_AREA + directory );
+ sink.text( directory );
sink.link_();
sink.text( "source-release" );
if ( csr.dist.isEmpty() && csr.distOlder.isEmpty() )
@@ -433,11 +434,7 @@ public class DistCheckSourceReleaseMojo
}
}
- List<String> expectedFiles = new LinkedList<>();
-
- expectedFiles.add( configLine.getArtifactId() + "-" + version +
"-source-release.zip" );
- expectedFiles.add( configLine.getArtifactId() + "-" + version +
"-source-release.zip.asc" );
- expectedFiles.add( configLine.getArtifactId() + "-" + version +
"-source-release.zip.md5" );
+ List<String> expectedFiles = configLine.getExpectedFilenames( version,
true );
retrievedFile.removeAll( expectedFiles );
@@ -463,7 +460,7 @@ public class DistCheckSourceReleaseMojo
* @return missing files
* @throws IOException
*/
- private List<String> checkDirectoryIndex( String url,
ConfigurationLineInfo configLine, String version )
+ private List<String> checkDirectoryIndex( String url,
ConfigurationLineInfo configLine, String version, boolean dist )
throws IOException
{
List<String> retrievedFile = new LinkedList<>();
@@ -473,11 +470,9 @@ public class DistCheckSourceReleaseMojo
retrievedFile.add( e.attr( "href" ) );
}
- List<String> missingFiles = new LinkedList<>();
+ List<String> missingFiles;
// initialize missing files with expected release file names
- missingFiles.add( configLine.getArtifactId() + "-" + version +
"-source-release.zip" );
- missingFiles.add( configLine.getArtifactId() + "-" + version +
"-source-release.zip.asc" );
- missingFiles.add( configLine.getArtifactId() + "-" + version +
"-source-release.zip.md5" );
+ missingFiles = configLine.getExpectedFilenames( version, dist );
// removed retrieved files
missingFiles.removeAll( retrievedFile );
@@ -505,11 +500,12 @@ public class DistCheckSourceReleaseMojo
// central
String centralUrl = configLine.getVersionnedFolderURL(
repoBaseUrl, version );
- result.setMissingCentralSourceRelease( checkDirectoryIndex(
centralUrl, configLine, version ) );
+ result.setMissingCentralSourceRelease( checkDirectoryIndex(
centralUrl, configLine, version, false ) );
// dist
- String distUrl = DIST_AREA + configLine.getDirectory();
- result.setMissingDistSourceRelease( checkDirectoryIndex( distUrl,
configLine, version ) );
+ String distUrl =
+ DIST_AREA + configLine.getDirectory() + (
configLine.isSrcBin() ? ( "/" + version + "/source" ) : "" );
+ result.setMissingDistSourceRelease( checkDirectoryIndex( distUrl,
configLine, version, true ) );
result.setDistOlderSourceRelease( checkContainsOld( distUrl,
configLine, version ) );
}
catch ( IOException ex )
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/resources/db/mavendb.txt
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/resources/db/mavendb.txt?rev=1502390&r1=1502389&r2=1502390&view=diff
==============================================================================
---
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/resources/db/mavendb.txt
(original)
+++
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/resources/db/mavendb.txt
Thu Jul 11 23:01:07 2013
@@ -1,8 +1,8 @@
## line format:
-## directory groupId
+## directory groupId [src+bin]
## artifactId [forced version]
##
-ant-tasks org.apache.maven
+ant-tasks org.apache.maven src+bin
maven-ant-tasks
archetype org.apache.maven.archetype
maven-archetype
@@ -21,8 +21,12 @@ enforcer org.apache.maven.enforcer
indexer org.apache.maven.indexer
maven-indexer
## maven-1
-## maven-2
-## maven-3
+maven-2 org.apache.maven src+bin
+ apache-maven 2.0.11
+ apache-maven 2.2.1
+maven-3 org.apache.maven src+bin
+ apache-maven 3.0.5
+ apache-maven
jxr org.apache.maven.jxr
jxr
plugin-testing org.apache.maven.plugin-testing
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/test/java/org/apache/maven/dist/tools/PatternTest.java
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/test/java/org/apache/maven/dist/tools/PatternTest.java?rev=1502390&r1=1502389&r2=1502390&view=diff
==============================================================================
---
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/test/java/org/apache/maven/dist/tools/PatternTest.java
(original)
+++
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/test/java/org/apache/maven/dist/tools/PatternTest.java
Thu Jul 11 23:01:07 2013
@@ -33,7 +33,7 @@ public class PatternTest
@Test
public void testGetGroupId()
{
- String q = DistCheckSourceReleaseMojo.getArtifactPattern( "doxia" );
+ String q = DistCheckSourceReleaseMojo.getSourceReleasePattern( "doxia"
);
assertTrue( "doxia-1.4-source-release.zip.asc".matches( q ) );
assertTrue( "doxia-1.4-source-release.zip.md5".matches( q ) );
@@ -42,7 +42,7 @@ public class PatternTest
assertFalse( "doxia-sitetools-1.4-source-release.zip.md5".matches( q )
);
assertFalse( "doxia-sitetools-1.4-source-release.zip".matches( q ) );
- String r = DistCheckSourceReleaseMojo.getArtifactPattern(
"doxia-sitetools" );
+ String r = DistCheckSourceReleaseMojo.getSourceReleasePattern(
"doxia-sitetools" );
assertFalse( "doxia-1.4-source-release.zip.asc".matches( r ) );
assertFalse( "doxia-1.4-source-release.zip.md5".matches( r ) );
