You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jv...@apache.org on 2015/04/16 15:11:56 UTC
[1/3] maven git commit: Add another deprecated method
Repository: maven
Updated Branches:
refs/heads/master 46e6c4215 -> 6c53bbb29
Add another deprecated method
Project: http://git-wip-us.apache.org/repos/asf/maven/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/7d5ca3e5
Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/7d5ca3e5
Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/7d5ca3e5
Branch: refs/heads/master
Commit: 7d5ca3e5b3be0ad96d1018849182def1802edc46
Parents: a2563b0
Author: Jason van Zyl <ja...@tesla.io>
Authored: Thu Apr 2 13:57:05 2015 -0400
Committer: Jason van Zyl <ja...@tesla.io>
Committed: Thu Apr 16 08:39:35 2015 -0400
----------------------------------------------------------------------
.../java/org/apache/maven/execution/MavenSession.java | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/maven/blob/7d5ca3e5/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java
----------------------------------------------------------------------
diff --git a/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java b/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java
index e4bfc80..235691a 100644
--- a/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java
+++ b/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java
@@ -272,13 +272,6 @@ public class MavenSession
this.projectMap = projectMap;
}
- @Deprecated
- /** @deprecated This appears to only be used in the ReactorReader and we can do any processing required there */
- public Map<String, MavenProject> getProjectMap()
- {
- return projectMap;
- }
-
/** This is a provisional method and may be removed */
public List<MavenProject> getAllProjects()
{
@@ -302,6 +295,13 @@ public class MavenSession
private final Settings settings;
@Deprecated
+ /** @deprecated This appears to only be used in the ReactorReader and we can do any processing required there */
+ public Map<String, MavenProject> getProjectMap()
+ {
+ return projectMap;
+ }
+
+ @Deprecated
public MavenSession( PlexusContainer container, RepositorySystemSession repositorySession,
MavenExecutionRequest request, MavenExecutionResult result )
{
[3/3] maven git commit: Remove the use of more deprecated methods
Posted by jv...@apache.org.
Remove the use of more deprecated methods
Project: http://git-wip-us.apache.org/repos/asf/maven/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/6c53bbb2
Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/6c53bbb2
Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/6c53bbb2
Branch: refs/heads/master
Commit: 6c53bbb293f7453cfb53f4c6da6b0926f6092cb3
Parents: 7d5ca3e
Author: Jason van Zyl <ja...@tesla.io>
Authored: Thu Apr 16 08:39:14 2015 -0400
Committer: Jason van Zyl <ja...@tesla.io>
Committed: Thu Apr 16 08:39:36 2015 -0400
----------------------------------------------------------------------
.../maven/lifecycle/internal/MojoExecutor.java | 20 ++++++++++----------
.../PluginParameterExpressionEvaluator.java | 12 +++++++++++-
2 files changed, 21 insertions(+), 11 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/maven/blob/6c53bbb2/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoExecutor.java
----------------------------------------------------------------------
diff --git a/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoExecutor.java b/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoExecutor.java
index fedb462..a6ba752 100644
--- a/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoExecutor.java
+++ b/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoExecutor.java
@@ -19,6 +19,15 @@ package org.apache.maven.lifecycle.internal;
* under the License.
*/
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.TreeSet;
+
import org.apache.maven.artifact.Artifact;
import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
import org.apache.maven.artifact.resolver.filter.CumulativeScopeArtifactFilter;
@@ -40,15 +49,6 @@ import org.codehaus.plexus.component.annotations.Component;
import org.codehaus.plexus.component.annotations.Requirement;
import org.codehaus.plexus.util.StringUtils;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.Collections;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-import java.util.TreeSet;
-
/**
* Executes an individual mojo
*
@@ -169,7 +169,7 @@ public class MojoExecutor
throw new LifecycleExecutionException( mojoExecution, session.getCurrentProject(), e );
}
- if ( mojoDescriptor.isProjectRequired() && !session.isUsingPOMsFromFilesystem() )
+ if ( mojoDescriptor.isProjectRequired() && !session.getRequest().isProjectPresent() )
{
Throwable cause =
new MissingProjectException( "Goal requires a project to execute"
http://git-wip-us.apache.org/repos/asf/maven/blob/6c53bbb2/maven-core/src/main/java/org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
----------------------------------------------------------------------
diff --git a/maven-core/src/main/java/org/apache/maven/plugin/PluginParameterExpressionEvaluator.java b/maven-core/src/main/java/org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
index b5cdc18..9199ba5 100644
--- a/maven-core/src/main/java/org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
+++ b/maven-core/src/main/java/org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
@@ -100,9 +100,16 @@ public class PluginParameterExpressionEvaluator
{
this.session = session;
this.mojoExecution = mojoExecution;
- this.properties = session.getExecutionProperties();
+ this.properties = new Properties();
this.project = session.getCurrentProject();
+ //
+ // Maven4: We may want to evaluate how this is used but we add these separate as the
+ // getExecutionProperties is deprecated in MavenSession.
+ //
+ this.properties.putAll( session.getUserProperties() );
+ this.properties.putAll( session.getSystemProperties() );
+
String basedir = null;
if ( project != null )
@@ -129,12 +136,14 @@ public class PluginParameterExpressionEvaluator
this.basedir = basedir;
}
+ @Override
public Object evaluate( String expr )
throws ExpressionEvaluationException
{
return evaluate( expr, null );
}
+ @Override
public Object evaluate( String expr, Class<?> type )
throws ExpressionEvaluationException
{
@@ -445,6 +454,7 @@ public class PluginParameterExpressionEvaluator
return expr;
}
+ @Override
public File alignToBaseDirectory( File file )
{
// TODO: Copied from the DefaultInterpolator. We likely want to resurrect the PathTranslator or at least a
[2/3] maven git commit: Don't add header if already present
Posted by jv...@apache.org.
Don't add header if already present
Project: http://git-wip-us.apache.org/repos/asf/maven/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/a2563b08
Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/a2563b08
Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/a2563b08
Branch: refs/heads/master
Commit: a2563b08c64c813c287fe3f51f18c7e0b1207ff9
Parents: 46e6c42
Author: Jason van Zyl <ja...@tesla.io>
Authored: Thu Apr 2 13:56:41 2015 -0400
Committer: Jason van Zyl <ja...@tesla.io>
Committed: Thu Apr 16 08:39:35 2015 -0400
----------------------------------------------------------------------
.../org/apache/maven/repository/legacy/DefaultWagonManager.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/maven/blob/a2563b08/maven-compat/src/main/java/org/apache/maven/repository/legacy/DefaultWagonManager.java
----------------------------------------------------------------------
diff --git a/maven-compat/src/main/java/org/apache/maven/repository/legacy/DefaultWagonManager.java b/maven-compat/src/main/java/org/apache/maven/repository/legacy/DefaultWagonManager.java
index 7f91d53..2c4a60e 100644
--- a/maven-compat/src/main/java/org/apache/maven/repository/legacy/DefaultWagonManager.java
+++ b/maven-compat/src/main/java/org/apache/maven/repository/legacy/DefaultWagonManager.java
@@ -243,11 +243,11 @@ public class DefaultWagonManager
// See org.eclipse.aether.connector.wagon.WagonRepositoryConnector.connectWagon(Wagon)
if( legacySupport.getRepositorySession() != null )
{
- String userAgent = ConfigUtils.getString( legacySupport.getRepositorySession(), ConfigurationProperties.USER_AGENT );
+ String userAgent = ConfigUtils.getString( legacySupport.getRepositorySession(), null, ConfigurationProperties.USER_AGENT );
if( userAgent == null)
{
Properties headers = new Properties();
-
+
headers.put( "User-Agent", ConfigUtils.getString( legacySupport.getRepositorySession(), "Maven",
ConfigurationProperties.USER_AGENT ) );
try