Author: evenisse
Date: Mon Feb 19 16:04:04 2007
New Revision: 509391
URL: http://svn.apache.org/viewvc?view=rev&rev=509391
Log:
Allow to set the version of dependencies in the release process then set
dependencies versions to snapshots if they was snapshots before the release.
Submitted by: Pete Marvin King
Modified:
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/config/PropertiesReleaseDescriptorStore.java
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhase.java
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhase.java
maven/release/trunk/maven-release-manager/src/main/mdo/release-descriptor.mdo
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/config/PropertiesReleaseDescriptorStoreTest.java
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhaseTest.java
maven/release/trunk/maven-release-manager/src/test/resources/release.properties
Modified:
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/config/PropertiesReleaseDescriptorStore.java
URL:
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/config/PropertiesReleaseDescriptorStore.java?view=diff&rev=509391&r1=509390&r2=509391
==============================================================================
---
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/config/PropertiesReleaseDescriptorStore.java
(original)
+++
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/config/PropertiesReleaseDescriptorStore.java
Mon Feb 19 16:04:04 2007
@@ -30,9 +30,12 @@
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
+import java.util.HashMap;
import java.util.Iterator;
import java.util.Map;
+import java.util.Map.Entry;
import java.util.Properties;
+import java.util.Set;
/**
* Read and write release configuration and state from a properties file.
@@ -95,6 +98,8 @@
releaseDescriptor.setPomFileName( properties.getProperty(
"exec.pomFileName" ) );
releaseDescriptor.setPreparationGoals( properties.getProperty(
"preparationGoals" ) );
+ loadResolvedDependencies( properties, releaseDescriptor );
+
// boolean properties are not written to the properties file because
the value from the caller is always used
for ( Iterator i = properties.keySet().iterator(); i.hasNext(); )
@@ -253,6 +258,12 @@
}
}
+ if ( ( config.getResolvedSnapshotDependencies() != null ) &&
+ ( config.getResolvedSnapshotDependencies().size() > 0 ) )
+ {
+ processResolvedDependencies( properties,
config.getResolvedSnapshotDependencies() );
+ }
+
OutputStream outStream = null;
//noinspection OverlyBroadCatchBlock
try
@@ -273,8 +284,82 @@
}
+ private void processResolvedDependencies( Properties prop, Map
resolvedDependencies )
+ {
+ Set entries = resolvedDependencies.entrySet();
+ Iterator iterator = entries.iterator();
+ Entry currentEntry;
+
+ while ( iterator.hasNext() )
+ {
+ currentEntry = (Entry) iterator.next();
+
+ Map versionMap = (Map) currentEntry.getValue();
+
+ prop.setProperty( "dependency." + currentEntry.getKey() +
".release",
+ (String) versionMap.get(
ReleaseDescriptor.RELEASE_KEY ) );
+ prop.setProperty( "dependency." + currentEntry.getKey() +
".development",
+ (String) versionMap.get(
ReleaseDescriptor.DEVELOPMENT_KEY ) );
+ }
+ }
+
private static File getDefaultReleasePropertiesFile( ReleaseDescriptor
mergeDescriptor )
{
return new File( mergeDescriptor.getWorkingDirectory(),
"release.properties" );
+ }
+
+ private void loadResolvedDependencies( Properties prop, ReleaseDescriptor
descriptor )
+ {
+ Map resolvedDependencies = new HashMap();
+
+ Set entries = prop.entrySet();
+ Iterator iterator = entries.iterator();
+ String propertyName;
+ Entry currentEntry;
+
+ while ( iterator.hasNext() )
+ {
+ currentEntry = (Entry) iterator.next();
+ propertyName = (String) currentEntry.getKey();
+
+ if ( propertyName.startsWith( "dependency." ) )
+ {
+ Map versionMap;
+ String artifactVersionlessKey;
+ int startIndex;
+ int endIndex;
+ String versionType;
+
+ versionMap = new HashMap();
+ startIndex = propertyName.lastIndexOf( "dependency." );
+
+ if ( propertyName.indexOf( ".development" ) != -1 )
+ {
+ endIndex = propertyName.indexOf( ".development" );
+ versionType = ReleaseDescriptor.DEVELOPMENT_KEY;
+ }
+ else
+ {
+ endIndex = propertyName.indexOf( ".release" );
+ versionType = ReleaseDescriptor.RELEASE_KEY;
+ }
+
+ artifactVersionlessKey = propertyName.substring( startIndex,
endIndex );
+
+ if ( resolvedDependencies.containsKey( artifactVersionlessKey
) )
+ {
+ versionMap = (Map) resolvedDependencies.get(
artifactVersionlessKey );
+ }
+ else
+ {
+ versionMap = new HashMap();
+ resolvedDependencies.put( artifactVersionlessKey,
versionMap );
+ }
+
+ versionMap.put( versionType, currentEntry.getValue() );
+ }
+ }
+
+ descriptor.setResolvedSnapshotDependencies( resolvedDependencies );
}
}
Modified:
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
URL:
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java?view=diff&rev=509391&r1=509390&r2=509391
==============================================================================
---
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
(original)
+++
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
Mon Feb 19 16:04:04 2007
@@ -207,6 +207,7 @@
Namespace namespace = rootElement.getNamespace();
Map mappedVersions = getNextVersionMap( releaseDescriptor );
Map originalVersions = getOriginalVersionMap( releaseDescriptor,
reactorProjects );
+ Map resolvedSnapshotDependencies =
releaseDescriptor.getResolvedSnapshotDependencies();
Element properties = rootElement.getChild( "properties", namespace );
String parentVersion = rewriteParent( project, rootElement, namespace,
mappedVersions, originalVersions );
@@ -215,8 +216,8 @@
rewriteVersion( rootElement, namespace, mappedVersions, projectId,
project, parentVersion );
- rewriteDependencies( project.getDependencies(), rootElement,
mappedVersions, originalVersions, projectId,
- properties, result );
+ rewriteDependencies( project.getDependencies(), rootElement,
mappedVersions, resolvedSnapshotDependencies,
+ originalVersions, projectId, properties,
releaseDescriptor, result );
if ( project.getDependencyManagement() != null )
{
@@ -224,7 +225,8 @@
if ( dependencyRoot != null )
{
rewriteDependencies(
project.getDependencyManagement().getDependencies(), dependencyRoot,
- mappedVersions, originalVersions,
projectId, properties, result );
+ mappedVersions,
resolvedSnapshotDependencies, originalVersions, projectId,
+ properties, releaseDescriptor, result );
}
}
@@ -233,19 +235,19 @@
Element buildRoot = rootElement.getChild( "build", namespace );
if ( buildRoot != null )
{
- rewritePlugins( project.getBuildPlugins(), buildRoot,
mappedVersions, originalVersions, projectId,
- properties, result );
+ rewritePlugins( project.getBuildPlugins(), buildRoot,
mappedVersions, resolvedSnapshotDependencies,
+ originalVersions, projectId, properties,
result );
if ( project.getPluginManagement() != null )
{
Element pluginsRoot = buildRoot.getChild(
"pluginManagement", namespace );
if ( pluginsRoot != null )
{
rewritePlugins(
project.getPluginManagement().getPlugins(), pluginsRoot, mappedVersions,
- originalVersions, projectId,
properties, result );
+ resolvedSnapshotDependencies,
originalVersions, projectId, properties, result );
}
}
- rewriteExtensions( project.getBuildExtensions(), buildRoot,
mappedVersions, originalVersions, projectId,
- properties, result );
+ rewriteExtensions( project.getBuildExtensions(), buildRoot,
mappedVersions,
+ resolvedSnapshotDependencies,
originalVersions, projectId, properties, result );
}
}
@@ -254,8 +256,8 @@
Element pluginsRoot = rootElement.getChild( "reporting", namespace
);
if ( pluginsRoot != null )
{
- rewriteReportPlugins( project.getReportPlugins(), pluginsRoot,
mappedVersions, originalVersions,
- projectId, properties, result );
+ rewriteReportPlugins( project.getReportPlugins(), pluginsRoot,
mappedVersions,
+ resolvedSnapshotDependencies,
originalVersions, projectId, properties, result );
}
}
@@ -321,7 +323,8 @@
}
private void rewriteDependencies( List dependencies, Element
dependencyRoot, Map mappedVersions,
- Map originalVersions, String projectId,
Element properties, ReleaseResult result )
+ Map resolvedSnapshotDependencies, Map
originalVersions, String projectId,
+ Element properties, ReleaseDescriptor
releaseDescriptor, ReleaseResult result )
throws ReleaseExecutionException, ReleaseFailureException
{
if ( dependencies != null )
@@ -330,15 +333,15 @@
{
Dependency dep = (Dependency) i.next();
- updateDomVersion( dep.getGroupId(), dep.getArtifactId(),
mappedVersions, dep.getVersion(),
- originalVersions, "dependencies",
"dependency", dependencyRoot, projectId,
- properties, result );
+ updateDomVersion( dep.getGroupId(), dep.getArtifactId(),
mappedVersions, resolvedSnapshotDependencies,
+ dep.getVersion(), originalVersions,
"dependencies", "dependency", dependencyRoot,
+ projectId, properties, result );
}
}
}
- private void rewritePlugins( List plugins, Element pluginRoot, Map
mappedVersions, Map originalVersions,
- String projectId, Element properties,
ReleaseResult result )
+ private void rewritePlugins( List plugins, Element pluginRoot, Map
mappedVersions, Map resolvedSnapshotDependencies,
+ Map originalVersions, String projectId,
Element properties, ReleaseResult result )
throws ReleaseExecutionException, ReleaseFailureException
{
if ( plugins != null )
@@ -350,16 +353,17 @@
// We can ignore plugins whose version is assumed, they are
only written into the release pom
if ( plugin.getVersion() != null )
{
- updateDomVersion( plugin.getGroupId(),
plugin.getArtifactId(), mappedVersions, plugin.getVersion(),
- originalVersions, "plugins", "plugin",
pluginRoot, projectId,
- properties, result );
+ updateDomVersion( plugin.getGroupId(),
plugin.getArtifactId(), mappedVersions,
+ resolvedSnapshotDependencies,
plugin.getVersion(), originalVersions, "plugins",
+ "plugin", pluginRoot, projectId,
properties, result );
}
}
}
}
- private void rewriteExtensions( List extensions, Element extensionRoot,
Map mappedVersions, Map originalVersions,
- String projectId, Element properties,
ReleaseResult result )
+ private void rewriteExtensions( List extensions, Element extensionRoot,
Map mappedVersions,
+ Map resolvedSnapshotDependencies, Map
originalVersions, String projectId,
+ Element properties, ReleaseResult result )
throws ReleaseExecutionException, ReleaseFailureException
{
if ( extensions != null )
@@ -369,14 +373,15 @@
Extension extension = (Extension) i.next();
updateDomVersion( extension.getGroupId(),
extension.getArtifactId(), mappedVersions,
- extension.getVersion(), originalVersions,
"extensions", "extension", extensionRoot,
- projectId, properties, result );
+ resolvedSnapshotDependencies,
extension.getVersion(), originalVersions, "extensions",
+ "extension", extensionRoot, projectId,
properties, result );
}
}
}
- private void rewriteReportPlugins( List plugins, Element pluginRoot, Map
mappedVersions, Map originalVersions,
- String projectId, Element properties,
ReleaseResult result )
+ private void rewriteReportPlugins( List plugins, Element pluginRoot, Map
mappedVersions,
+ Map resolvedSnapshotDependencies, Map
originalVersions, String projectId,
+ Element properties, ReleaseResult
result )
throws ReleaseExecutionException, ReleaseFailureException
{
if ( plugins != null )
@@ -388,28 +393,38 @@
// We can ignore plugins whose version is assumed, they are
only written into the release pom
if ( plugin.getVersion() != null )
{
- updateDomVersion( plugin.getGroupId(),
plugin.getArtifactId(), mappedVersions, plugin.getVersion(),
- originalVersions, "plugins", "plugin",
pluginRoot, projectId,
- properties, result );
+ updateDomVersion( plugin.getGroupId(),
plugin.getArtifactId(), mappedVersions,
+ resolvedSnapshotDependencies,
plugin.getVersion(), originalVersions, "plugins",
+ "plugin", pluginRoot, projectId,
properties, result );
}
}
}
}
- private void updateDomVersion( String groupId, String artifactId, Map
mappedVersions, String version,
- Map originalVersions, String groupTagName,
String tagName, Element dependencyRoot,
- String projectId, Element properties,
ReleaseResult result )
+ private void updateDomVersion( String groupId, String artifactId, Map
mappedVersions,
+ Map resolvedSnapshotDepedencies, String
version, Map originalVersions,
+ String groupTagName, String tagName,
Element dependencyRoot, String projectId,
+ Element properties, ReleaseResult result )
throws ReleaseExecutionException, ReleaseFailureException
{
String key = ArtifactUtils.versionlessKey( groupId, artifactId );
String mappedVersion = (String) mappedVersions.get( key );
+ String resolvedSnapshotVersion = getResolvedSnapshotVersion( key,
resolvedSnapshotDepedencies );
Object originalVersion = originalVersions.get( key );
+
+ // workaround
+ if ( originalVersion == null )
+ {
+ originalVersion = getOriginalResolvedSnapshotVersion( key,
resolvedSnapshotDepedencies );
+ }
+
if ( version.equals( originalVersion ) )
{
- if ( mappedVersion != null )
+ if ( ( mappedVersion != null ) || ( resolvedSnapshotVersion !=
null ) )
{
- logInfo( result, "Updating " + artifactId + " to " +
mappedVersion );
+ logInfo( result, "Updating " + artifactId + " to " +
+ ( ( mappedVersion != null ) ? mappedVersion :
resolvedSnapshotVersion ) );
try
{
@@ -435,6 +450,12 @@
// avoid if in management
if ( versionElement != null )
{
+ if ( mappedVersion == null )
+ {
+ versionElement.setText(
resolvedSnapshotVersion );
+ return;
+ }
+
String versionText = versionElement.getTextTrim();
// avoid if it was not originally set to the
original value (it may be an expression), unless mapped version differs
@@ -443,8 +464,8 @@
{
versionElement.setText( mappedVersion );
}
- else if ( versionText.matches(
"\\$\\{project.+\\}" )
- || versionText.matches( "\\$\\{pom.+\\}" ) )
+ else if ( versionText.matches(
"\\$\\{project.+\\}" ) ||
+ versionText.matches( "\\$\\{pom.+\\}" ) )
{
logInfo( result, "Ignoring artifact version
update for expression: " + versionText );
//ignore... we cannot update this expression
@@ -475,8 +496,8 @@
else
{
// the expression used to define the
version of this artifact may be inherited
- throw new ReleaseFailureException( "The
version could not be updated: " +
- versionText );
+ throw new ReleaseFailureException(
+ "The version could not be updated: " +
versionText );
}
}
else
@@ -623,6 +644,8 @@
return result;
}
+ protected abstract String getResolvedSnapshotVersion( String
artifactVersionlessKey, Map resolvedSnapshots );
+
protected abstract Map getOriginalVersionMap( ReleaseDescriptor
releaseDescriptor, List reactorProjects );
protected abstract Map getNextVersionMap( ReleaseDescriptor
releaseDescriptor );
@@ -631,6 +654,20 @@
ReleaseDescriptor releaseDescriptor,
String projectId,
ScmRepository scmRepository,
ReleaseResult result )
throws ReleaseExecutionException;
+
+ protected String getOriginalResolvedSnapshotVersion( String
artifactVersionlessKey, Map resolvedSnapshots )
+ {
+ Map versionsMap = (Map) resolvedSnapshots.get( artifactVersionlessKey
);
+
+ if ( versionsMap != null )
+ {
+ return (String) ( versionsMap.get(
ReleaseDescriptor.ORIGINAL_VERSION ) );
+ }
+ else
+ {
+ return null;
+ }
+ }
protected Element rewriteElement( String name, String value, Element root,
Namespace namespace )
{
Modified:
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java
URL:
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java?view=diff&rev=509391&r1=509390&r2=509391
==============================================================================
---
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java
(original)
+++
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java
Mon Feb 19 16:04:04 2007
@@ -29,12 +29,16 @@
import org.apache.maven.shared.release.ReleaseFailureException;
import org.apache.maven.shared.release.ReleaseResult;
import org.apache.maven.shared.release.config.ReleaseDescriptor;
+import org.apache.maven.shared.release.versions.DefaultVersionInfo;
+import org.apache.maven.shared.release.versions.VersionInfo;
+import org.apache.maven.shared.release.versions.VersionParseException;
import org.codehaus.plexus.components.interactivity.Prompter;
import org.codehaus.plexus.components.interactivity.PrompterException;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
+import java.util.HashMap;
import java.util.HashSet;
import java.util.Iterator;
import java.util.List;
@@ -51,6 +55,26 @@
public class CheckDependencySnapshotsPhase
extends AbstractReleasePhase
{
+ public static final String RESOLVE_SNAPSHOT_MESSAGE = "There are still
some remaining snapshot dependencies.";
+
+ public static final String RESOLVE_SNAPSHOT_PROMPT = "Do you want to
resolve them now?";
+
+ public static final String RESOLVE_SNAPSHOT_TYPE_MESSAGE = "Dependency
type to resolve,";
+
+ public static final String RESOLVE_SNAPSHOT_TYPE_PROMPT =
+ "specify the selection number ( 0:All 1:Project Dependencies 2:Plugins
3:Reports 4:Extensions ):";
+
+ public static final String RESOLVE_ALL_SNAPSHOT_MESSAGE = "Resolve All
Snapshots.";
+
+ public static final String
RESOLVE_ALL_PROJECT_DEPENDENCIES_SNAPSHOT_MESSAGE =
+ "Resolve Project Dependency Snapshots.";
+
+ public static final String RESOLVE_ALL_REPORTS_SNAPSHOT_MESSAGE = "Resolve
Report Dependency Snapshots.";
+
+ public static final String RESOLVE_ALL_EXTENSIONS_SNAPSHOT_MESSAGE =
"Resolve Extension Dependency Snapshots.";
+
+ public static final String RESOLVE_ALL_PLUGIN_SNAPSHOT_MESSAGE = "Resolve
Plugin Dependency Snapshots.";
+
/**
* Component used to prompt for input.
*/
@@ -86,6 +110,9 @@
throws ReleaseFailureException, ReleaseExecutionException
{
Set snapshotDependencies = new HashSet();
+ Set snapshotReportDependencies = new HashSet();
+ Set snapshotExtensionsDependencies = new HashSet();
+ Set snapshotPluginDependencies = new HashSet();
if ( project.getParentArtifact() != null )
{
@@ -149,7 +176,7 @@
if ( addToFailures )
{
- snapshotDependencies.add( artifact );
+ snapshotPluginDependencies.add( artifact );
}
}
}
@@ -160,7 +187,8 @@
if ( checkArtifact( artifact, originalVersions ) )
{
- snapshotDependencies.add( artifact );
+ //snapshotDependencies.add( artifact );
+ snapshotReportDependencies.add( artifact );
}
}
@@ -170,32 +198,33 @@
if ( checkArtifact( artifact, originalVersions ) )
{
- snapshotDependencies.add( artifact );
+ snapshotExtensionsDependencies.add( artifact );
}
}
- if ( !snapshotDependencies.isEmpty() )
+ if ( !snapshotDependencies.isEmpty() ||
!snapshotReportDependencies.isEmpty() ||
+ !snapshotExtensionsDependencies.isEmpty() ||
!snapshotPluginDependencies.isEmpty() )
{
- List snapshotsList = new ArrayList( snapshotDependencies );
-
- Collections.sort( snapshotsList );
-
- StringBuffer message = new StringBuffer();
-
- for ( Iterator i = snapshotsList.iterator(); i.hasNext(); )
+ if ( releaseDescriptor.isInteractive() )
{
- Artifact artifact = (Artifact) i.next();
+ resolveSnapshots( snapshotDependencies,
snapshotReportDependencies, snapshotExtensionsDependencies,
+ snapshotPluginDependencies,
releaseDescriptor );
+ }
- message.append( " " );
+ if ( !snapshotDependencies.isEmpty() ||
!snapshotReportDependencies.isEmpty() ||
+ !snapshotExtensionsDependencies.isEmpty() ||
!snapshotPluginDependencies.isEmpty() )
+ {
+ StringBuffer message = new StringBuffer();
- message.append( artifact );
+ printSnapshotDependencies( snapshotDependencies, message );
+ printSnapshotDependencies( snapshotReportDependencies, message
);
+ printSnapshotDependencies( snapshotExtensionsDependencies,
message );
+ printSnapshotDependencies( snapshotPluginDependencies, message
);
+ message.append( "in project '" + project.getName() + "' (" +
project.getId() + ")" );
- message.append( "\n" );
+ throw new ReleaseFailureException(
+ "Can't release project due to non released dependencies
:\n" + message );
}
-
- message.append( "in project '" + project.getName() + "' (" +
project.getId() + ")" );
-
- throw new ReleaseFailureException( "Can't release project due to
non released dependencies :\n" + message );
}
}
@@ -219,5 +248,132 @@
public void setPrompter( Prompter prompter )
{
this.prompter = prompter;
+ }
+
+ private StringBuffer printSnapshotDependencies( Set snapshotsSet,
StringBuffer message )
+ {
+ List snapshotsList = new ArrayList( snapshotsSet );
+
+ Collections.sort( snapshotsList );
+
+ for ( Iterator i = snapshotsList.iterator(); i.hasNext(); )
+ {
+ Artifact artifact = (Artifact) i.next();
+
+ message.append( " " );
+
+ message.append( artifact );
+
+ message.append( "\n" );
+ }
+
+ return message;
+ }
+
+ private void resolveSnapshots( Set projectDependencies, Set
reportDependencies, Set extensionDependencies,
+ Set pluginDependencies, ReleaseDescriptor
releaseDescriptor )
+ throws ReleaseExecutionException
+ {
+ try
+ {
+ prompter.showMessage( RESOLVE_SNAPSHOT_MESSAGE );
+ String result =
+ prompter.prompt( RESOLVE_SNAPSHOT_PROMPT, Arrays.asList( new
String[]{"yes", "no"} ), "no" );
+
+ if ( result.toLowerCase().startsWith( "y" ) )
+ {
+ Set snapshotSet = new HashSet();
+ Map resolvedSnapshots = null;
+ prompter.showMessage( RESOLVE_SNAPSHOT_TYPE_MESSAGE );
+ result = prompter.prompt( RESOLVE_SNAPSHOT_TYPE_PROMPT,
+ Arrays.asList( new String[]{"0",
"1", "2", "3"} ), "1" );
+
+ switch ( Integer.parseInt( result.toLowerCase() ) )
+ {
+ // all
+ case 0:
+ prompter.showMessage( RESOLVE_ALL_SNAPSHOT_MESSAGE );
+ snapshotSet.addAll( projectDependencies );
+ snapshotSet.addAll( reportDependencies );
+ snapshotSet.addAll( extensionDependencies );
+ snapshotSet.addAll( pluginDependencies );
+ resolvedSnapshots = processSnapshot( snapshotSet );
+ break;
+
+ // project dependencies
+ case 1:
+ prompter.showMessage(
RESOLVE_ALL_PROJECT_DEPENDENCIES_SNAPSHOT_MESSAGE );
+ resolvedSnapshots = processSnapshot(
projectDependencies );
+ break;
+
+ // plugins
+ case 2:
+ prompter.showMessage(
RESOLVE_ALL_PLUGIN_SNAPSHOT_MESSAGE );
+ resolvedSnapshots = processSnapshot(
pluginDependencies );
+ break;
+
+ // reports
+ case 3:
+ prompter.showMessage(
RESOLVE_ALL_REPORTS_SNAPSHOT_MESSAGE );
+ resolvedSnapshots = processSnapshot(
reportDependencies );
+ break;
+
+ // extensions
+ case 4:
+ prompter.showMessage(
RESOLVE_ALL_EXTENSIONS_SNAPSHOT_MESSAGE );
+ resolvedSnapshots = processSnapshot(
extensionDependencies );
+ break;
+ }
+
+ releaseDescriptor.setResolvedSnapshotDependencies(
resolvedSnapshots );
+ }
+ }
+ catch ( PrompterException e )
+ {
+ throw new ReleaseExecutionException( e.getMessage(), e );
+ }
+ catch ( VersionParseException e )
+ {
+ throw new ReleaseExecutionException( e.getMessage(), e );
+ }
+ }
+
+ private Map processSnapshot( Set snapshotSet )
+ throws PrompterException, VersionParseException
+ {
+ Map resolvedSnapshots = new HashMap();
+ Iterator iterator = snapshotSet.iterator();
+ Artifact currentArtifact;
+ String result;
+ VersionInfo version;
+
+ while ( iterator.hasNext() )
+ {
+ currentArtifact = (Artifact) iterator.next();
+ version = new DefaultVersionInfo( currentArtifact.getVersion() );
+
+ result = prompter.prompt( "'" + ArtifactUtils.versionlessKey(
currentArtifact ) + "' set to release?",
+ Arrays.asList( new String[]{"yes", "no"}
), "yes" );
+
+ if ( result.toLowerCase().startsWith( "y" ) )
+ {
+ VersionInfo nextDevelopmentVersion;
+ Map versionMap = new HashMap();
+
+ iterator.remove();
+ result = prompter.prompt( "What is the next development
version?", Collections.singletonList(
+ version.getNextVersion().getSnapshotVersionString() ),
+
version.getNextVersion().getSnapshotVersionString() );
+
+ nextDevelopmentVersion = new DefaultVersionInfo( result );
+ versionMap.put( ReleaseDescriptor.ORIGINAL_VERSION,
version.toString() );
+ versionMap.put( ReleaseDescriptor.DEVELOPMENT_KEY,
nextDevelopmentVersion.getSnapshotVersionString() );
+ versionMap.put( ReleaseDescriptor.RELEASE_KEY,
version.getReleaseVersionString() );
+
+ resolvedSnapshots.put( ArtifactUtils.versionlessKey(
currentArtifact ), versionMap );
+ }
+ }
+
+ return resolvedSnapshots;
}
}
Modified:
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhase.java
URL:
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhase.java?view=diff&rev=509391&r1=509390&r2=509391
==============================================================================
---
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhase.java
(original)
+++
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhase.java
Mon Feb 19 16:04:04 2007
@@ -101,4 +101,18 @@
{
return releaseDescriptor.getDevelopmentVersions();
}
+
+ protected String getResolvedSnapshotVersion( String
artifactVersionlessKey, Map resolvedSnapshotsMap )
+ {
+ Map versionsMap = (Map) resolvedSnapshotsMap.get(
artifactVersionlessKey );
+
+ if ( versionsMap != null )
+ {
+ return (String) ( versionsMap.get(
ReleaseDescriptor.DEVELOPMENT_KEY ) );
+ }
+ else
+ {
+ return null;
+ }
+ }
}
Modified:
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhase.java
URL:
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhase.java?view=diff&rev=509391&r1=509390&r2=509391
==============================================================================
---
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhase.java
(original)
+++
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhase.java
Mon Feb 19 16:04:04 2007
@@ -162,4 +162,18 @@
{
return releaseDescriptor.getReleaseVersions();
}
-}
\ No newline at end of file
+
+ protected String getResolvedSnapshotVersion( String
artifactVersionlessKey, Map resolvedSnapshotsMap )
+ {
+ Map versionsMap = (Map) resolvedSnapshotsMap.get(
artifactVersionlessKey );
+
+ if ( versionsMap != null )
+ {
+ return (String) ( versionsMap.get( ReleaseDescriptor.RELEASE_KEY )
);
+ }
+ else
+ {
+ return null;
+ }
+ }
+}
Modified:
maven/release/trunk/maven-release-manager/src/main/mdo/release-descriptor.mdo
URL:
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/mdo/release-descriptor.mdo?view=diff&rev=509391&r1=509390&r2=509391
==============================================================================
---
maven/release/trunk/maven-release-manager/src/main/mdo/release-descriptor.mdo
(original)
+++
maven/release/trunk/maven-release-manager/src/main/mdo/release-descriptor.mdo
Mon Feb 19 16:04:04 2007
@@ -161,6 +161,18 @@
</description>
</field>
<field>
+ <name>resolvedSnapshotDependencies</name>
+ <version>1.0.0</version>
+ <type>Map</type>
+ <association stash.keyType="String">
+ <type>String</type>
+ <multiplicity>*</multiplicity>
+ </association>
+ <description>
+ A map of resolved snapshot dependencies versions.
+ </description>
+ </field>
+ <field>
<name>originalScmInfo</name>
<version>1.0.0</version>
<type>Map</type>
@@ -257,6 +269,31 @@
<codeSegment>
<version>1.0.0</version>
<code><