maven-surefire git commit: JDK 5 should be the min requirements in surefire project

2014-12-27 Thread tibordigana
Repository: maven-surefire
Updated Branches:
  refs/heads/master f003abd6f - 4c07642ad


JDK 5 should be the min requirements in surefire project


Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/4c07642a
Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/4c07642a
Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/4c07642a

Branch: refs/heads/master
Commit: 4c07642adb892c2b8a34d8640fa6d571fbe84775
Parents: f003abd
Author: Tibor17 tibo...@lycos.com
Authored: Sat Dec 27 00:16:35 2014 +0100
Committer: Tibor17 tibo...@lycos.com
Committed: Sat Dec 27 00:16:35 2014 +0100

--
 maven-failsafe-plugin/pom.xml| 5 -
 maven-surefire-plugin/pom.xml| 5 -
 maven-surefire-report-plugin/pom.xml | 5 -
 3 files changed, 15 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/4c07642a/maven-failsafe-plugin/pom.xml
--
diff --git a/maven-failsafe-plugin/pom.xml b/maven-failsafe-plugin/pom.xml
index 8d55eac..16ae10d 100644
--- a/maven-failsafe-plugin/pom.xml
+++ b/maven-failsafe-plugin/pom.xml
@@ -215,11 +215,6 @@
 groupIdorg.apache.maven.plugins/groupId
 artifactIdmaven-plugin-plugin/artifactId
 version${mavenPluginPluginVersion}/version
-configuration
-  requirements
-jdk1.4/jdk
-  /requirements
-/configuration
   /plugin
 /plugins
   /reporting

http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/4c07642a/maven-surefire-plugin/pom.xml
--
diff --git a/maven-surefire-plugin/pom.xml b/maven-surefire-plugin/pom.xml
index 2b1d440..c3f3fa7 100644
--- a/maven-surefire-plugin/pom.xml
+++ b/maven-surefire-plugin/pom.xml
@@ -129,11 +129,6 @@
 groupIdorg.apache.maven.plugins/groupId
 artifactIdmaven-plugin-plugin/artifactId
 version${mavenPluginPluginVersion}/version
-configuration
-  requirements
-jdk1.4/jdk
-  /requirements
-/configuration
   /plugin
 /plugins
   /reporting

http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/4c07642a/maven-surefire-report-plugin/pom.xml
--
diff --git a/maven-surefire-report-plugin/pom.xml 
b/maven-surefire-report-plugin/pom.xml
index 96b3cbf..2a13789 100644
--- a/maven-surefire-report-plugin/pom.xml
+++ b/maven-surefire-report-plugin/pom.xml
@@ -164,11 +164,6 @@
 groupIdorg.apache.maven.plugins/groupId
 artifactIdmaven-plugin-plugin/artifactId
 version${mavenPluginPluginVersion}/version
-configuration
-  requirements
-jdk1.4/jdk
-  /requirements
-/configuration
   /plugin
 /plugins
   /reporting



maven-surefire git commit: [SUREFIRE-1128] Fix mvn 2.2.1 build process https://builds.apache.org/view/All/job/maven-surefire-mvn-2.2.1

2014-12-27 Thread tibordigana
Repository: maven-surefire
Updated Branches:
  refs/heads/master 4c07642ad - 12d2c4b4a


[SUREFIRE-1128] Fix mvn 2.2.1 build process 
https://builds.apache.org/view/All/job/maven-surefire-mvn-2.2.1


Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/12d2c4b4
Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/12d2c4b4
Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/12d2c4b4

Branch: refs/heads/master
Commit: 12d2c4b4a44402f36fbc66ebabf1c400b5c6451a
Parents: 4c07642
Author: unknown digana.ti...@sk-za-04278.vsb
Authored: Fri Dec 26 21:35:50 2014 +0100
Committer: unknown digana.ti...@sk-za-04278.vsb
Committed: Sat Dec 27 11:07:16 2014 +0100

--
 .../it/jetty-war-test-failing/invoker.properties  |  3 +++
 .../it/jetty-war-test-passing/invoker.properties  |  3 +++
 .../surefire/report/StatelessXmlReporterTest.java |  4 +++-
 .../surefire-1055-parallelTestCount/pom.xml   | 17 -
 .../java/listeners/MarkAsFailureListener.java | 18 +-
 5 files changed, 30 insertions(+), 15 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/12d2c4b4/maven-failsafe-plugin/src/it/jetty-war-test-failing/invoker.properties
--
diff --git 
a/maven-failsafe-plugin/src/it/jetty-war-test-failing/invoker.properties 
b/maven-failsafe-plugin/src/it/jetty-war-test-failing/invoker.properties
index ab6608f..82e0bc1 100644
--- a/maven-failsafe-plugin/src/it/jetty-war-test-failing/invoker.properties
+++ b/maven-failsafe-plugin/src/it/jetty-war-test-failing/invoker.properties
@@ -17,3 +17,6 @@
 # under the License.
 #
 invoker.buildResult=failure
+
+# build project if JRE version is 1.7 or higher
+invoker.java.version = 1.7+

http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/12d2c4b4/maven-failsafe-plugin/src/it/jetty-war-test-passing/invoker.properties
--
diff --git 
a/maven-failsafe-plugin/src/it/jetty-war-test-passing/invoker.properties 
b/maven-failsafe-plugin/src/it/jetty-war-test-passing/invoker.properties
index c56128f..dca9bcd 100644
--- a/maven-failsafe-plugin/src/it/jetty-war-test-passing/invoker.properties
+++ b/maven-failsafe-plugin/src/it/jetty-war-test-passing/invoker.properties
@@ -17,3 +17,6 @@
 # under the License.
 #
 invoker.buildResult=success
+
+# build project if JRE version is 1.7 or higher
+invoker.java.version = 1.7+

http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/12d2c4b4/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporterTest.java
--
diff --git 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporterTest.java
 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporterTest.java
index 6c2982a..dad0ac2 100644
--- 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporterTest.java
+++ 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporterTest.java
@@ -60,6 +60,7 @@ public class StatelessXmlReporterTest
 private final static String TEST_TWO = bTestMethod;
 private final static String TEST_THREE = cTestMethod;
 
+@Override
 protected void setUp()
 throws Exception
 {
@@ -71,7 +72,8 @@ public class StatelessXmlReporterTest
 reporter.cleanTestHistoryMap();
 }
 
-@Override protected void tearDown()
+@Override
+protected void tearDown()
 throws Exception
 {
 super.tearDown();

http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/12d2c4b4/surefire-integration-tests/src/test/resources/surefire-1055-parallelTestCount/pom.xml
--
diff --git 
a/surefire-integration-tests/src/test/resources/surefire-1055-parallelTestCount/pom.xml
 
b/surefire-integration-tests/src/test/resources/surefire-1055-parallelTestCount/pom.xml
index 56f6457..d405a8c 100644
--- 
a/surefire-integration-tests/src/test/resources/surefire-1055-parallelTestCount/pom.xml
+++ 
b/surefire-integration-tests/src/test/resources/surefire-1055-parallelTestCount/pom.xml
@@ -20,9 +20,16 @@
  xsi:schemaLocation=http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xsd/maven-4.0.0.xsd;
 modelVersion4.0.0/modelVersion
 
+parent
+groupIdorg.apache.maven.surefire/groupId
+artifactIdit-parent/artifactId
+version1.0/version
+relativePath../pom.xml/relativePath
+/parent
+
 groupIdfoo/groupId
 artifactIdfoo/artifactId
-

svn commit: r1648045 - /maven/plugins/trunk/maven-pmd-plugin/pom.xml

2014-12-27 Thread hboutemy
Author: hboutemy
Date: Sat Dec 27 10:54:59 2014
New Revision: 1648045

URL: http://svn.apache.org/r1648045
Log:
fixed build with mvn 2.2.1

Modified:
maven/plugins/trunk/maven-pmd-plugin/pom.xml

Modified: maven/plugins/trunk/maven-pmd-plugin/pom.xml
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/pom.xml?rev=1648045r1=1648044r2=1648045view=diff
==
--- maven/plugins/trunk/maven-pmd-plugin/pom.xml (original)
+++ maven/plugins/trunk/maven-pmd-plugin/pom.xml Sat Dec 27 10:54:59 2014
@@ -120,18 +120,6 @@ under the License.
   scopeprovided/scope
 /dependency
 
-!-- shared --
-dependency
-  groupIdorg.apache.maven.reporting/groupId
-  artifactIdmaven-reporting-api/artifactId
-  version3.0/version
-/dependency
-dependency
-  groupIdorg.apache.maven.reporting/groupId
-  artifactIdmaven-reporting-impl/artifactId
-  version2.3/version
-/dependency
-
 !-- doxia --
 dependency
   groupIdorg.apache.maven.doxia/groupId
@@ -171,6 +159,18 @@ under the License.
   /exclusions
 /dependency
 
+!-- shared --
+dependency
+  groupIdorg.apache.maven.reporting/groupId
+  artifactIdmaven-reporting-api/artifactId
+  version3.0/version
+/dependency
+dependency
+  groupIdorg.apache.maven.reporting/groupId
+  artifactIdmaven-reporting-impl/artifactId
+  version2.3/version
+/dependency
+
 !-- plexus --
 dependency
   groupIdorg.codehaus.plexus/groupId
@@ -224,13 +224,19 @@ under the License.
   version1.52/version
   scopetest/scope
   exclusions
-  !-- wiremock brings slf4j 1.7.6, but maven-plugin-testing-harness 
brings 1.5.6 --
-  exclusion
-  artifactIdslf4j-api/artifactId
-  groupIdorg.slf4j/groupId
-  /exclusion
+!-- wiremock brings slf4j 1.7.6, but maven-plugin-testing-harness 
brings 1.5.6 --
+exclusion
+  groupIdorg.slf4j/groupId
+  artifactIdslf4j-api/artifactId
+/exclusion
   /exclusions
 /dependency
+dependency!-- wiremock requires 4.3.5 but Doxia brings 4.0.2 and 
httpcore 4.0.1 --
+  groupIdorg.apache.httpcomponents/groupId
+  artifactIdhttpclient/artifactId
+  version4.3.5/version
+  scopetest/scope
+/dependency
   /dependencies
 
   build




svn commit: r1648046 - /maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/JarModule.java

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 11:07:02 2014
New Revision: 1648046

URL: http://svn.apache.org/r1648046
Log:
Removed accidently added comment.

Modified:

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/JarModule.java

Modified: 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/JarModule.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/JarModule.java?rev=1648046r1=1648045r2=1648046view=diff
==
--- 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/JarModule.java
 (original)
+++ 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/JarModule.java
 Sat Dec 27 11:07:02 2014
@@ -37,9 +37,6 @@ import java.util.Set;
  * @author a href=snic...@apache.orgStephane Nicoll/a
  * @version $Id$
  */
-/**
- * @author kama
- */
 public class JarModule
 extends AbstractEarModule
 {




svn commit: r1648051 - in /maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear: AbstractEarMojo.java GenerateApplicationXmlMojo.java

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 12:55:58 2014
New Revision: 1648051

URL: http://svn.apache.org/r1648051
Log:
[MEAR-188] Project property cannot be resolved inside env-entry element
 Introduced interpolation within the envEntries elements its no possible
 having properties etc. within those elements.

Modified:

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/AbstractEarMojo.java

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/GenerateApplicationXmlMojo.java

Modified: 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/AbstractEarMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/AbstractEarMojo.java?rev=1648051r1=1648050r2=1648051view=diff
==
--- 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/AbstractEarMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/AbstractEarMojo.java
 Sat Dec 27 12:55:58 2014
@@ -31,8 +31,10 @@ import org.apache.maven.plugin.MojoExecu
 import org.apache.maven.plugin.MojoFailureException;
 import org.apache.maven.plugin.ear.util.ArtifactTypeMappingService;
 import org.apache.maven.plugin.ear.util.JavaEEVersion;
+import org.apache.maven.plugins.annotations.Component;
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
+import org.apache.maven.shared.filtering.MavenResourcesFiltering;
 import org.codehaus.plexus.configuration.PlexusConfiguration;
 import org.codehaus.plexus.configuration.PlexusConfigurationException;
 
@@ -159,6 +161,9 @@ public abstract class AbstractEarMojo
 @Parameter( defaultValue = ${project.build.directory}, required = true )
 private File tempFolder;
 
+@Component
+private MavenResourcesFiltering mavenResourcesFiltering;
+
 private ListEarModule earModules;
 
 private ListEarModule allModules;

Modified: 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/GenerateApplicationXmlMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/GenerateApplicationXmlMojo.java?rev=1648051r1=1648050r2=1648051view=diff
==
--- 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/GenerateApplicationXmlMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/GenerateApplicationXmlMojo.java
 Sat Dec 27 12:55:58 2014
@@ -19,6 +19,12 @@ package org.apache.maven.plugin.ear;
  * under the License.
  */
 
+import java.io.File;
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.List;
+
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugin.MojoFailureException;
 import org.apache.maven.plugin.ear.util.JavaEEVersion;
@@ -28,14 +34,13 @@ import org.apache.maven.plugins.annotati
 import org.apache.maven.plugins.annotations.ResolutionScope;
 import org.codehaus.plexus.configuration.PlexusConfiguration;
 import org.codehaus.plexus.configuration.PlexusConfigurationException;
+import org.codehaus.plexus.interpolation.InterpolationException;
+import org.codehaus.plexus.interpolation.Interpolator;
+import org.codehaus.plexus.interpolation.MapBasedValueSource;
+import org.codehaus.plexus.interpolation.StringSearchInterpolator;
+import org.codehaus.plexus.interpolation.ValueSource;
 import org.codehaus.plexus.util.FileUtils;
 
-import java.io.File;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.List;
-
 /**
  * Generates the EAR deployment descriptor file(s).
  * 
@@ -44,7 +49,7 @@ import java.util.List;
  */
 // CHECKSTYLE_OFF: LineLength
 @Mojo( name = generate-application-xml, defaultPhase = 
LifecyclePhase.GENERATE_RESOURCES, threadSafe = true, 
requiresDependencyResolution = ResolutionScope.TEST )
-//CHECKSTYLE_ON: LineLength
+// CHECKSTYLE_ON: LineLength
 public class GenerateApplicationXmlMojo
 extends AbstractEarMojo
 {
@@ -304,6 +309,28 @@ public class GenerateApplicationXmlMojo
 }
 
 /**
+ * This help method was needed otherwise the interpolate method of 
interpolator will make an empty string of a
+ * {@code null} element which results in supplemental elements for 
env-entry.
+ * 
+ * @param interpolator The interpolator
+ * @param element The element
+ * @return The interpolated elements.
+ * @throws InterpolationException in case of an error.
+ */
+private String interpolate( Interpolator interpolator, String element )
+throws InterpolationException
+{
+if ( element == null )
+{
+return element;
+}

svn commit: r1648055 - in /maven/plugins/trunk/maven-ear-plugin/src: main/java/org/apache/maven/plugin/ear/ test/java/org/apache/maven/plugin/ear/it/ test/resources/projects/project-085/ test/resource

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 14:59:45 2014
New Revision: 1648055

URL: http://svn.apache.org/r1648055
Log:
[MEAR-181] Adding ejb-ref in application.xml
 Added ejb-ref first implementation.

Added:

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/EjbRef.java

maven/plugins/trunk/maven-ear-plugin/src/test/resources/projects/project-085/

maven/plugins/trunk/maven-ear-plugin/src/test/resources/projects/project-085/expected-META-INF/

maven/plugins/trunk/maven-ear-plugin/src/test/resources/projects/project-085/expected-META-INF/application.xml

maven/plugins/trunk/maven-ear-plugin/src/test/resources/projects/project-085/pom.xml
Modified:

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriter.java

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriterContext.java

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/EnvEntry.java

maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/GenerateApplicationXmlMojo.java

maven/plugins/trunk/maven-ear-plugin/src/test/java/org/apache/maven/plugin/ear/it/EarMojoIT.java

Modified: 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriter.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriter.java?rev=1648055r1=1648054r2=1648055view=diff
==
--- 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriter.java
 (original)
+++ 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriter.java
 Sat Dec 27 14:59:45 2014
@@ -130,6 +130,10 @@ final class ApplicationXmlWriter
 {
 envEntry.appendEnvEntry( writer );
 }
+for ( EjbRef ejbEntry : context.getEjbEntries() )
+{
+ejbEntry.appendEjbRefEntry( writer );
+}
 }
 
 writer.endElement();

Modified: 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriterContext.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriterContext.java?rev=1648055r1=1648054r2=1648055view=diff
==
--- 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriterContext.java
 (original)
+++ 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/ApplicationXmlWriterContext.java
 Sat Dec 27 14:59:45 2014
@@ -40,6 +40,8 @@ class ApplicationXmlWriterContext
 private final ListSecurityRole securityRoles;
 
 private final ListEnvEntry envEntries;
+
+private final ListEjbRef ejbEntries;
 
 private final String displayName;
 
@@ -53,6 +55,7 @@ class ApplicationXmlWriterContext
 
 public ApplicationXmlWriterContext( File destinationFile, ListEarModule 
earModules,
 ListSecurityRole securityRoles, 
ListEnvEntry envEntries,
+ListEjbRef ejbEntries,
 String displayName, String 
description, String libraryDirectory,
 String applicationName, Boolean 
initializeInOrder )
 {
@@ -60,6 +63,7 @@ class ApplicationXmlWriterContext
 this.earModules = earModules;
 this.securityRoles = securityRoles;
 this.envEntries = envEntries;
+this.ejbEntries = ejbEntries;
 this.displayName = displayName;
 this.description = description;
 this.libraryDirectory = libraryDirectory;
@@ -119,6 +123,16 @@ class ApplicationXmlWriterContext
 }
 
 /**
+ * Returns the list of {@link EjbRef}.
+ * 
+ * @return the env-ref elements
+ */
+public ListEjbRef getEjbEntries()
+{
+return ejbEntries;
+}
+
+/**
  * Returns the display name.
  * 
  * @return the display name

Added: 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/EjbRef.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/EjbRef.java?rev=1648055view=auto
==
--- 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/EjbRef.java
 (added)
+++ 
maven/plugins/trunk/maven-ear-plugin/src/main/java/org/apache/maven/plugin/ear/EjbRef.java
 Sat Dec 27 14:59:45 2014
@@ -0,0 +1,147 @@
+package org.apache.maven.plugin.ear;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or 

svn commit: r1648057 - /maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 15:50:36 2014
New Revision: 1648057

URL: http://svn.apache.org/r1648057
Log:
Refactored code to get shorter methods.

Modified:

maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

Modified: 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java?rev=1648057r1=1648056r2=1648057view=diff
==
--- 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 Sat Dec 27 15:50:36 2014
@@ -277,16 +277,7 @@ public class EjbMojo
 File deploymentDescriptor = new File( outputDirectory, ejbJar );
 
 /* test EJB version compliance */
-if ( !ejbVersion.matches( \\A[2-3]\\.[0-9]\\z ) )
-{
-throw new MojoExecutionException( ejbVersion is not valid:  + 
ejbVersion
-+ . Must be 2.x or 3.x (where x is a digit) );
-}
-
-if ( ejbVersion.matches( \\A2\\.[0-9]\\z )  
!deploymentDescriptor.exists() )
-{
-throw new MojoExecutionException( Error assembling EJB:  + 
ejbJar +  is required for ejbVersion 2.x );
-}
+ejbVersionCompliance( deploymentDescriptor );
 
 try
 {
@@ -306,20 +297,7 @@ public class EjbMojo
 // EJB-34 Filter ejb-jar.xml
 if ( filterDeploymentDescriptor )
 {
-getLog().debug( Filtering deployment descriptor. );
-MavenResourcesExecution mavenResourcesExecution = new 
MavenResourcesExecution();
-mavenResourcesExecution.setEscapeString( escapeString );
-ListFilterWrapper filterWrappers =
-mavenFileFilter.getDefaultFilterWrappers( project, 
filters, escapeBackslashesInFilePath,
-  
this.session, mavenResourcesExecution );
-
-// Create a temporary file that we can copy-and-filter
-File unfilteredDeploymentDescriptor = new File( 
outputDirectory, ejbJar + .unfiltered );
-FileUtils.copyFile( deploymentDescriptor, 
unfilteredDeploymentDescriptor );
-mavenFileFilter.copyFile( unfilteredDeploymentDescriptor, 
deploymentDescriptor, true,
-  filterWrappers, getEncoding( 
unfilteredDeploymentDescriptor ) );
-// Remove the temporary file
-FileUtils.forceDelete( unfilteredDeploymentDescriptor );
+filterDeploymentDescriptor( deploymentDescriptor );
 }
 archiver.getArchiver().addFile( deploymentDescriptor, ejbJar );
 }
@@ -361,74 +339,114 @@ public class EjbMojo
 
 if ( generateClient )
 {
-String clientJarName = jarName;
-if ( classifier != null )
-{
-clientJarName += - + classifier;
-}
+generateEjbClient();
+}
+}
 
-getLog().info( Building EJB client  + clientJarName + -client 
);
+private void generateEjbClient()
+throws MojoExecutionException
+{
+String clientJarName = jarName;
+if ( classifier != null )
+{
+clientJarName += - + classifier;
+}
 
-String[] excludes = DEFAULT_CLIENT_EXCLUDES;
-String[] includes = DEFAULT_INCLUDES;
+getLog().info( Building EJB client  + clientJarName + -client );
 
-if ( clientIncludes != null  !clientIncludes.isEmpty() )
-{
-includes = (String[]) clientIncludes.toArray( new 
String[clientIncludes.size()] );
-}
+String[] excludes = DEFAULT_CLIENT_EXCLUDES;
+String[] includes = DEFAULT_INCLUDES;
 
-if ( clientExcludes != null  !clientExcludes.isEmpty() )
-{
-excludes = (String[]) clientExcludes.toArray( new 
String[clientExcludes.size()] );
-}
+if ( clientIncludes != null  !clientIncludes.isEmpty() )
+{
+includes = (String[]) clientIncludes.toArray( new 
String[clientIncludes.size()] );
+}
 
-File clientJarFile = new File( basedir, clientJarName + 
-client.jar );
+if ( clientExcludes != null  !clientExcludes.isEmpty() )
+{
+excludes = (String[]) clientExcludes.toArray( new 
String[clientExcludes.size()] );
+}
 
-MavenArchiver clientArchiver = new MavenArchiver();
+File clientJarFile = new File( basedir, clientJarName + -client.jar 
);
 
-

svn commit: r1648059 - /maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 15:54:24 2014
New Revision: 1648059

URL: http://svn.apache.org/r1648059
Log:
Fixed warning based on missing type.

Modified:

maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

Modified: 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java?rev=1648059r1=1648058r2=1648059view=diff
==
--- 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 Sat Dec 27 15:54:24 2014
@@ -231,7 +231,7 @@ public class EjbMojo
  * @since 2.3
  */
 @Parameter
-private List filters;
+private ListString filters;
 
 /**
  * @since 2.3




svn commit: r1648062 - in /maven/plugins/trunk/maven-ejb-plugin/src: main/java/org/apache/maven/plugin/ejb/EjbMojo.java test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 16:10:33 2014
New Revision: 1648062

URL: http://svn.apache.org/r1648062
Log:
Polished code.

Modified:

maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

maven/plugins/trunk/maven-ejb-plugin/src/test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java

Modified: 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java?rev=1648062r1=1648061r2=1648062view=diff
==
--- 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 Sat Dec 27 16:10:33 2014
@@ -94,7 +94,7 @@ public class EjbMojo
  * You can define the location of codeejb-jar.xml/code file.
  */
 @Parameter( property = ejb.ejbJar, defaultValue = META-INF/ejb-jar.xml 
)
-// The initalization is needed to get the unit tests running which seemed 
to lack lookup for the defaultValue.
+// The initialization is needed to get the unit tests running which seemed 
to lack lookup for the defaultValue.
 private String ejbJar = META-INF/ejb-jar.xml;
 
 /**
@@ -277,7 +277,7 @@ public class EjbMojo
 File deploymentDescriptor = new File( outputDirectory, ejbJar );
 
 /* test EJB version compliance */
-ejbVersionCompliance( deploymentDescriptor );
+checEJBVersionCompliance( deploymentDescriptor );
 
 try
 {
@@ -415,7 +415,7 @@ public class EjbMojo
 }
 }
 
-private void ejbVersionCompliance( File deploymentDescriptor )
+private void checEJBVersionCompliance( File deploymentDescriptor )
 throws MojoExecutionException
 {
 if ( !ejbVersion.matches( \\A[2-3]\\.[0-9]\\z ) )

Modified: 
maven/plugins/trunk/maven-ejb-plugin/src/test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ejb-plugin/src/test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java?rev=1648062r1=1648061r2=1648062view=diff
==
--- 
maven/plugins/trunk/maven-ejb-plugin/src/test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java
 (original)
+++ 
maven/plugins/trunk/maven-ejb-plugin/src/test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java
 Sat Dec 27 16:10:33 2014
@@ -19,6 +19,12 @@ package org.apache.maven.plugin.ejb;
  * under the License.
  */
 
+import java.io.File;
+import java.io.IOException;
+import java.util.LinkedList;
+import java.util.List;
+import java.util.jar.JarFile;
+
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugin.ejb.stub.MavenProjectResourcesStub;
 import org.apache.maven.plugin.ejb.utils.JarContentChecker;
@@ -26,12 +32,6 @@ import org.apache.maven.plugin.testing.A
 import org.apache.maven.project.MavenProject;
 import org.codehaus.plexus.util.FileUtils;
 
-import java.io.File;
-import java.io.IOException;
-import java.util.LinkedList;
-import java.util.jar.JarFile;
-
-
 /**
  * EJB plugin Test Case
  */
@@ -183,10 +183,10 @@ public class EjbMojoTest
 mojo.execute();
 
 assertJarCreation( project, true, false );
-assertJarContent( project, new String[]{META-INF/MANIFEST.MF, 
META-INF/ejb-jar.xml,
+assertJarContent( project, new String[] { META-INF/MANIFEST.MF, 
META-INF/ejb-jar.xml,
 META-INF/maven/org.apache.maven.test/maven-test-plugin/pom.xml,
 
META-INF/maven/org.apache.maven.test/maven-test-plugin/pom.properties, 
org/sample/ejb/AppBean.class,
-org/sample/ejb/AppCMP.class, org/sample/ejb/AppSession.class}, 
null );
+org/sample/ejb/AppCMP.class, org/sample/ejb/AppSession.class 
}, null );
 }
 
 /**
@@ -220,11 +220,12 @@ public class EjbMojoTest
 mojo.execute();
 
 assertJarCreation( project, true, true );
-assertClientJarContent( project, new String[]{META-INF/MANIFEST.MF,
-META-INF/maven/org.apache.maven.test/maven-test-plugin/pom.xml,
-
META-INF/maven/org.apache.maven.test/maven-test-plugin/pom.properties, 
org/sample/ejb/AppStub.class},
- new String[]{META-INF/ejb-jar.xml, 
org/sample/ejb/AppBean.class,
- org/sample/ejb/AppCMP.class, 
org/sample/ejb/AppSession.class} );
+assertClientJarContent( project, new String[] { META-INF/MANIFEST.MF,
+
META-INF/maven/org.apache.maven.test/maven-test-plugin/pom.xml,
+
META-INF/maven/org.apache.maven.test/maven-test-plugin/pom.properties,
+org/sample/ejb/AppStub.class 

svn commit: r1648069 - /maven/plugins/trunk/maven-deploy-plugin/src/site/apt/examples/deploy-ssh-external.apt

2014-12-27 Thread michaelo
Author: michaelo
Date: Sat Dec 27 16:55:13 2014
New Revision: 1648069

URL: http://svn.apache.org/r1648069
Log:
[MDEPLOY-194] Deploy SSH external: copy and paste failure in POM snippet comment

Modified:

maven/plugins/trunk/maven-deploy-plugin/src/site/apt/examples/deploy-ssh-external.apt

Modified: 
maven/plugins/trunk/maven-deploy-plugin/src/site/apt/examples/deploy-ssh-external.apt
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-deploy-plugin/src/site/apt/examples/deploy-ssh-external.apt?rev=1648069r1=1648068r2=1648069view=diff
==
--- 
maven/plugins/trunk/maven-deploy-plugin/src/site/apt/examples/deploy-ssh-external.apt
 (original)
+++ 
maven/plugins/trunk/maven-deploy-plugin/src/site/apt/examples/deploy-ssh-external.apt
 Sat Dec 27 16:55:13 2014
@@ -45,7 +45,7 @@ Deployment of artifacts in an external S
 
   build
 extensions
-  !-- Enabling the use of FTP --
+  !-- Enabling the use of SSH --
   extension
 groupIdorg.apache.maven.wagon/groupId
  artifactIdwagon-ssh-external/artifactId




svn commit: r1648080 - in /maven/plugins/trunk/maven-javadoc-plugin/src/it: MJAVADOC-257/pom.xml MJAVADOC-259/src/main/java/com/mycompany/myapp/App.java MJAVADOC-262/src/main/java/com/mycompany/myapp/

2014-12-27 Thread hboutemy
Author: hboutemy
Date: Sat Dec 27 18:13:23 2014
New Revision: 1648080

URL: http://svn.apache.org/r1648080
Log:
fixed IT issues with JDK 8

Modified:
maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-257/pom.xml

maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-259/src/main/java/com/mycompany/myapp/App.java

maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-262/src/main/java/com/mycompany/myapp/App.java

Modified: maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-257/pom.xml
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-257/pom.xml?rev=1648080r1=1648079r2=1648080view=diff
==
--- maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-257/pom.xml 
(original)
+++ maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-257/pom.xml Sat 
Dec 27 18:13:23 2014
@@ -80,6 +80,15 @@
 groupIdorg.apache.maven.plugins/groupId
 artifactIdmaven-javadoc-plugin/artifactId
 version@pom.version@/version
+configuration
+  tagletArtifacts
+tagletArtifact
+  groupIdorg.apache.maven.plugin-tools/groupId
+  artifactIdmaven-plugin-tools-javadoc/artifactId
+  version3.3/version
+/tagletArtifact
+  /tagletArtifacts
+/configuration
 reportSets
   reportSet
 idaggregate/id

Modified: 
maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-259/src/main/java/com/mycompany/myapp/App.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-259/src/main/java/com/mycompany/myapp/App.java?rev=1648080r1=1648079r2=1648080view=diff
==
--- 
maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-259/src/main/java/com/mycompany/myapp/App.java
 (original)
+++ 
maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-259/src/main/java/com/mycompany/myapp/App.java
 Sat Dec 27 18:13:23 2014
@@ -22,7 +22,7 @@ package com.mycompany.myapp;
 /**
  * Hello world!
  *
- * img src=doc-files/maven-feather.png alt=Maven/
+ * img src=doc-files/maven-feather.png alt=Maven
  */
 public class App
 {

Modified: 
maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-262/src/main/java/com/mycompany/myapp/App.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-262/src/main/java/com/mycompany/myapp/App.java?rev=1648080r1=1648079r2=1648080view=diff
==
--- 
maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-262/src/main/java/com/mycompany/myapp/App.java
 (original)
+++ 
maven/plugins/trunk/maven-javadoc-plugin/src/it/MJAVADOC-262/src/main/java/com/mycompany/myapp/App.java
 Sat Dec 27 18:13:23 2014
@@ -22,7 +22,7 @@ package com.mycompany.myapp;
 /**
  * Hello world!
  *
- * img src=doc-files/maven-feather.png alt=Maven/
+ * img src=doc-files/maven-feather.png alt=Maven
  */
 public class App
 {




svn commit: r1648088 - /maven/plugins/trunk/maven-compiler-plugin/src/it/error-prone-compiler/invoker.properties

2014-12-27 Thread hboutemy
Author: hboutemy
Date: Sat Dec 27 18:44:16 2014
New Revision: 1648088

URL: http://svn.apache.org/r1648088
Log:
disabling error-prone compiler test with JDK 8: this should be tested in 
plexus-compiler

Modified:

maven/plugins/trunk/maven-compiler-plugin/src/it/error-prone-compiler/invoker.properties

Modified: 
maven/plugins/trunk/maven-compiler-plugin/src/it/error-prone-compiler/invoker.properties
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-compiler-plugin/src/it/error-prone-compiler/invoker.properties?rev=1648088r1=1648087r2=1648088view=diff
==
--- 
maven/plugins/trunk/maven-compiler-plugin/src/it/error-prone-compiler/invoker.properties
 (original)
+++ 
maven/plugins/trunk/maven-compiler-plugin/src/it/error-prone-compiler/invoker.properties
 Sat Dec 27 18:44:16 2014
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.java.version = 1.7+
+invoker.java.version = 1.7
 invoker.goals = clean compile
 invoker.buildResult = failure




Git Push Summary

2014-12-27 Thread hboutemy
Repository: maven
Updated Branches:
  refs/heads/MNG- [deleted] 1295d41c7


Git Push Summary

2014-12-27 Thread hboutemy
Repository: maven
Updated Branches:
  refs/heads/maven-3.0.4-RCx [deleted] 63beaa103


svn commit: r1648121 - /maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 21:59:08 2014
New Revision: 1648121

URL: http://svn.apache.org/r1648121
Log:
Slight improvments.

Modified:

maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

Modified: 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java?rev=1648121r1=1648120r2=1648121view=diff
==
--- 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 Sat Dec 27 21:59:08 2014
@@ -277,7 +277,7 @@ public class EjbMojo
 File deploymentDescriptor = new File( outputDirectory, ejbJar );
 
 /* test EJB version compliance */
-checEJBVersionCompliance( deploymentDescriptor );
+checkEJBVersionCompliance( deploymentDescriptor );
 
 try
 {
@@ -328,6 +328,7 @@ public class EjbMojo
 }
 
 // Handle the classifier if necessary
+// TODO: For 3.0 this should be changed having a separate classifier 
for main artifact and ejb-client.
 if ( classifier != null )
 {
 projectHelper.attachArtifact( project, ejb, classifier, jarFile 
);
@@ -352,7 +353,8 @@ public class EjbMojo
 clientJarName += - + classifier;
 }
 
-getLog().info( Building EJB client  + clientJarName + -client );
+String resultingClientJarNameWithClassifier = clientJarName + 
-client;
+getLog().info( Building EJB client  + 
resultingClientJarNameWithClassifier );
 
 String[] excludes = DEFAULT_CLIENT_EXCLUDES;
 String[] includes = DEFAULT_INCLUDES;
@@ -367,7 +369,7 @@ public class EjbMojo
 excludes = (String[]) clientExcludes.toArray( new 
String[clientExcludes.size()] );
 }
 
-File clientJarFile = new File( basedir, clientJarName + -client.jar 
);
+File clientJarFile = new File( basedir, 
resultingClientJarNameWithClassifier + .jar );
 
 MavenArchiver clientArchiver = new MavenArchiver();
 
@@ -405,6 +407,7 @@ public class EjbMojo
 }
 
 // TODO: shouldn't need classifer
+// TODO: For 3.0 this should be changed having a separate classifier 
for main artifact and ejb-client.
 if ( classifier != null )
 {
 projectHelper.attachArtifact( project, ejb-client, classifier + 
-client, clientJarFile );
@@ -415,7 +418,7 @@ public class EjbMojo
 }
 }
 
-private void checEJBVersionCompliance( File deploymentDescriptor )
+private void checkEJBVersionCompliance( File deploymentDescriptor )
 throws MojoExecutionException
 {
 if ( !ejbVersion.matches( \\A[2-3]\\.[0-9]\\z ) )




maven-wagon git commit: [WAGON-429] add the missing WagonScp.getTransfer() to support 2G file

2014-12-27 Thread dantran
Repository: maven-wagon
Updated Branches:
  refs/heads/master bcaf7300d - dd623e806


[WAGON-429] add the missing WagonScp.getTransfer() to support  2G file


Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/dd623e80
Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/dd623e80
Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/dd623e80

Branch: refs/heads/master
Commit: dd623e8069221940439c8d4dbc867be91c75885d
Parents: bcaf730
Author: dantran dant...@gmail.com
Authored: Sat Dec 27 15:35:38 2014 -0800
Committer: dantran dant...@gmail.com
Committed: Sat Dec 27 15:35:38 2014 -0800

--
 .../wagon/providers/ssh/jsch/ScpWagon.java  | 894 ++-
 1 file changed, 451 insertions(+), 443 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/dd623e80/wagon-providers/wagon-ssh/src/main/java/org/apache/maven/wagon/providers/ssh/jsch/ScpWagon.java
--
diff --git 
a/wagon-providers/wagon-ssh/src/main/java/org/apache/maven/wagon/providers/ssh/jsch/ScpWagon.java
 
b/wagon-providers/wagon-ssh/src/main/java/org/apache/maven/wagon/providers/ssh/jsch/ScpWagon.java
index 9ce7561..c5290b2 100644
--- 
a/wagon-providers/wagon-ssh/src/main/java/org/apache/maven/wagon/providers/ssh/jsch/ScpWagon.java
+++ 
b/wagon-providers/wagon-ssh/src/main/java/org/apache/maven/wagon/providers/ssh/jsch/ScpWagon.java
@@ -1,443 +1,451 @@
-package org.apache.maven.wagon.providers.ssh.jsch;
-
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
- * to you 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 com.jcraft.jsch.ChannelExec;
-import com.jcraft.jsch.JSchException;
-import org.apache.maven.wagon.CommandExecutionException;
-import org.apache.maven.wagon.InputData;
-import org.apache.maven.wagon.OutputData;
-import org.apache.maven.wagon.ResourceDoesNotExistException;
-import org.apache.maven.wagon.TransferFailedException;
-import org.apache.maven.wagon.events.TransferEvent;
-import org.apache.maven.wagon.providers.ssh.ScpHelper;
-import org.apache.maven.wagon.repository.RepositoryPermissions;
-import org.apache.maven.wagon.resource.Resource;
-
-import java.io.EOFException;
-import java.io.IOException;
-import java.io.InputStream;
-import java.io.OutputStream;
-
-/**
- * SCP protocol wagon.
- * p/
- * Note that this implementation is inot/i thread-safe, and multiple 
channels can not be used on the session at
- * the same time.
- * p/
- * See a href=http://blogs.sun.com/janp/entry/how_the_scp_protocol_works;
- * http://blogs.sun.com/janp/entry/how_the_scp_protocol_works/a
- * for information on how the SCP protocol works.
- *
- *
- * @todo [BP] add compression flag
- * @plexus.component role=org.apache.maven.wagon.Wagon
- * role-hint=scp
- * instantiation-strategy=per-lookup
- */
-public class ScpWagon
-extends AbstractJschWagon
-{
-private static final char COPY_START_CHAR = 'C';
-
-private static final char ACK_SEPARATOR = ' ';
-
-private static final String END_OF_FILES_MSG = E\n;
-
-private static final int LINE_BUFFER_SIZE = 8192;
-
-private static final byte LF = '\n';
-
-private ChannelExec channel;
-
-private InputStream channelInputStream;
-
-private OutputStream channelOutputStream;
-
-private void setFileGroup( RepositoryPermissions permissions, String 
basedir, Resource resource )
-throws CommandExecutionException
-{
-if ( permissions != null  permissions.getGroup() != null )
-{
-//executeCommand( chgrp -f  + permissions.getGroup() +   + 
getPath( basedir, resource.getName() ) );
-executeCommand( chgrp -f  + permissions.getGroup() +  \ + 
getPath( basedir, resource.getName() ) + \ );
-}
-}
-
-protected void cleanupPutTransfer( Resource resource )
-{
-if ( channel != null )
-{
-channel.disconnect();
-channel = null;
-}
-}
-
-protected void finishPutTransfer( Resource resource, InputStream input, 
OutputStream output )
-

svn commit: r1648129 - in /maven/plugins/trunk/maven-ejb-plugin/src: it/reactor/sub/pom.xml main/java/org/apache/maven/plugin/ejb/EjbMojo.java test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java

2014-12-27 Thread khmarbaise
Author: khmarbaise
Date: Sat Dec 27 23:35:16 2014
New Revision: 1648129

URL: http://svn.apache.org/r1648129
Log:
[MEJB-63] outputDirectory is treated differently to ear, war plugins
 Now we have a sourceDirectory and a outputDirectory which behave the
 way you expect them to behave.
 The sourceDirectory contains those elements which will be archived
 and the outputDirectory contains the created archives.

Modified:
maven/plugins/trunk/maven-ejb-plugin/src/it/reactor/sub/pom.xml

maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java

maven/plugins/trunk/maven-ejb-plugin/src/test/java/org/apache/maven/plugin/ejb/EjbMojoTest.java

Modified: maven/plugins/trunk/maven-ejb-plugin/src/it/reactor/sub/pom.xml
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ejb-plugin/src/it/reactor/sub/pom.xml?rev=1648129r1=1648128r2=1648129view=diff
==
--- maven/plugins/trunk/maven-ejb-plugin/src/it/reactor/sub/pom.xml (original)
+++ maven/plugins/trunk/maven-ejb-plugin/src/it/reactor/sub/pom.xml Sat Dec 27 
23:35:16 2014
@@ -44,7 +44,7 @@ under the License.
 version@project.version@/version
 configuration
   !-- Note the relative path here --
-  outputDirectorytarget/classes/outputDirectory
+  sourceDirectorytarget/classes/sourceDirectory
 /configuration
   /plugin
 /plugins

Modified: 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java?rev=1648129r1=1648128r2=1648129view=diff
==
--- 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-ejb-plugin/src/main/java/org/apache/maven/plugin/ejb/EjbMojo.java
 Sat Dec 27 23:35:16 2014
@@ -67,16 +67,17 @@ public class EjbMojo
 **/*Session.class, **/package.html };
 
 /**
- * The directory for the generated EJB.
+ * The directory location for the generated EJB.
  */
 @Parameter( defaultValue = ${project.build.directory}, required = true, 
readonly = true )
-private File basedir;
+private File outputDirectory;
 
 /**
- * Directory that resources are copied to during the build.
+ * Directory that contains the resources which are packaged into
+ * the created archive {@code target/classes}.
  */
-@Parameter( property = outputDirectory, defaultValue = 
${project.build.outputDirectory} )
-private File outputDirectory;
+@Parameter( defaultValue = ${project.build.outputDirectory}, required = 
true )
+private File sourceDirectory;
 
 /**
  * The name of the EJB file to generate.
@@ -254,11 +255,11 @@ public class EjbMojo
 throws MojoExecutionException
 {
 
-if ( !outputDirectory.exists() )
+if ( !sourceDirectory.exists() )
 {
-getLog().warn( The created EJB jar will be empty cause the  + 
outputDirectory.getPath()
+getLog().warn( The created EJB jar will be empty cause the  + 
sourceDirectory.getPath()
+  did not exist. );
-outputDirectory.mkdirs();
+sourceDirectory.mkdirs();
 }
 
 if ( getLog().isInfoEnabled() )
@@ -266,7 +267,7 @@ public class EjbMojo
 getLog().info( Building EJB  + jarName +  with EJB version  + 
ejbVersion );
 }
 
-File jarFile = getEJBJarFile( basedir, jarName, classifier );
+File jarFile = getEJBJarFile( outputDirectory, jarName, classifier );
 
 MavenArchiver archiver = new MavenArchiver();
 
@@ -274,7 +275,7 @@ public class EjbMojo
 
 archiver.setOutputFile( jarFile );
 
-File deploymentDescriptor = new File( outputDirectory, ejbJar );
+File deploymentDescriptor = new File( sourceDirectory, ejbJar );
 
 /* test EJB version compliance */
 checkEJBVersionCompliance( deploymentDescriptor );
@@ -290,7 +291,7 @@ public class EjbMojo
 mainJarExcludes = (String[]) excludes.toArray( new 
String[excludes.size()] );
 }
 
-archiver.getArchiver().addDirectory( outputDirectory, 
DEFAULT_INCLUDES, mainJarExcludes );
+archiver.getArchiver().addDirectory( sourceDirectory, 
DEFAULT_INCLUDES, mainJarExcludes );
 
 if ( deploymentDescriptor.exists() )
 {
@@ -369,7 +370,7 @@ public class EjbMojo
 excludes = (String[]) clientExcludes.toArray( new 
String[clientExcludes.size()] );
 }
 
-File clientJarFile = new File( basedir, 
resultingClientJarNameWithClassifier + .jar );
+File clientJarFile = new File( outputDirectory, 
resultingClientJarNameWithClassifier + .jar );
 
 

svn commit: r1648130 - in /maven/plugins/trunk/maven-gpg-plugin/src: it/settings.xml main/java/org/apache/maven/plugin/gpg/AbstractGpgMojo.java

2014-12-27 Thread dantran
Author: dantran
Date: Sat Dec 27 23:45:59 2014
New Revision: 1648130

URL: http://svn.apache.org/r1648130
Log:
[MGPG-31] fix typos from previous commit

Modified:
maven/plugins/trunk/maven-gpg-plugin/src/it/settings.xml

maven/plugins/trunk/maven-gpg-plugin/src/main/java/org/apache/maven/plugin/gpg/AbstractGpgMojo.java

Modified: maven/plugins/trunk/maven-gpg-plugin/src/it/settings.xml
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-gpg-plugin/src/it/settings.xml?rev=1648130r1=1648129r2=1648130view=diff
==
--- maven/plugins/trunk/maven-gpg-plugin/src/it/settings.xml (original)
+++ maven/plugins/trunk/maven-gpg-plugin/src/it/settings.xml Sat Dec 27 
23:45:59 2014
@@ -58,7 +58,7 @@ under the License.
 
   servers
 server
-  idgpg.passphase/id
+  idgpg.passphrase/id
   passphraseTEST/passphrase
 /server
   /servers

Modified: 
maven/plugins/trunk/maven-gpg-plugin/src/main/java/org/apache/maven/plugin/gpg/AbstractGpgMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-gpg-plugin/src/main/java/org/apache/maven/plugin/gpg/AbstractGpgMojo.java?rev=1648130r1=1648129r2=1648130view=diff
==
--- 
maven/plugins/trunk/maven-gpg-plugin/src/main/java/org/apache/maven/plugin/gpg/AbstractGpgMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-gpg-plugin/src/main/java/org/apache/maven/plugin/gpg/AbstractGpgMojo.java
 Sat Dec 27 23:45:59 2014
@@ -62,8 +62,8 @@ public abstract class AbstractGpgMojo
  * Server id to lookup the passphase under Maven settings.
  * @since 1.6
  */
-@Parameter( property = gpg.passphaseServerKey, defaultValue = 
gpg.passphase )
-private String passphaseServerKey;
+@Parameter( property = gpg.passphraseServerKey, defaultValue = 
gpg.passphrase )
+private String passphraseServerKey;
 
 /**
  * The name of the key to sign with. Passed to gpg as 
code--local-user/code.
@@ -213,7 +213,7 @@ public abstract class AbstractGpgMojo
 {
 if ( StringUtils.isEmpty( this.passphrase ) )
 {
-Server server = this.settings.getServer( passphaseServerKey );
+Server server = this.settings.getServer( passphraseServerKey );
 
 if ( server != null )
 {