svn commit: r1430738 - in /maven/plugins/trunk/maven-doap-plugin/src: it/MDOAP-38/verify.groovy main/java/org/apache/maven/plugin/doap/DoapMojo.java

2013-01-09 Thread olamy
Author: olamy
Date: Wed Jan  9 09:02:21 2013
New Revision: 1430738

URL: http://svn.apache.org/viewvc?rev=1430738view=rev
Log:
say the first what :-)

Modified:
maven/plugins/trunk/maven-doap-plugin/src/it/MDOAP-38/verify.groovy

maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java

Modified: maven/plugins/trunk/maven-doap-plugin/src/it/MDOAP-38/verify.groovy
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-doap-plugin/src/it/MDOAP-38/verify.groovy?rev=1430738r1=1430737r2=1430738view=diff
==
--- maven/plugins/trunk/maven-doap-plugin/src/it/MDOAP-38/verify.groovy 
(original)
+++ maven/plugins/trunk/maven-doap-plugin/src/it/MDOAP-38/verify.groovy Wed Jan 
 9 09:02:21 2013
@@ -21,4 +21,4 @@ def logFile = new File( basedir, 'build.
 assert logFile.exists()
 content = logFile.text
 
-assert content.contains( 'The doapOptionsshortdesc//doapOptions first is 
too long maximum words number is 10.' )
+assert content.contains( 'The doapOptionsshortdesc//doapOptions first 
sentence is too long maximum words number is 10.' )

Modified: 
maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java?rev=1430738r1=1430737r2=1430738view=diff
==
--- 
maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
 Wed Jan  9 09:02:21 2013
@@ -2641,7 +2641,7 @@ public class DoapMojo
 break;
 case SHORT_DESC_TOO_LONG:
 errorMessages.add(
-The  + toConfiguration( tags, value ) +  first is 
too long maximum words number is 10. );
+The  + toConfiguration( tags, value ) +  first 
sentence is too long maximum words number is 10. );
 break;
 default:
 throw new IllegalArgumentException( Unknown errorId= + 
errorId );




svn commit: r1430962 - in /maven/shared/trunk/maven-archiver/src/site/apt/examples: classpath.apt manifest.apt manifestEntries.apt manifestSections.apt

2013-01-09 Thread olamy
Author: olamy
Date: Wed Jan  9 17:16:08 2013
New Revision: 1430962

URL: http://svn.apache.org/viewvc?rev=1430962view=rev
Log:
[MSHARED-273] Update documentation for the Created-By manifest entry

Modified:
maven/shared/trunk/maven-archiver/src/site/apt/examples/classpath.apt
maven/shared/trunk/maven-archiver/src/site/apt/examples/manifest.apt
maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestEntries.apt
maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestSections.apt

Modified: maven/shared/trunk/maven-archiver/src/site/apt/examples/classpath.apt
URL: 
http://svn.apache.org/viewvc/maven/shared/trunk/maven-archiver/src/site/apt/examples/classpath.apt?rev=1430962r1=1430961r2=1430962view=diff
==
--- maven/shared/trunk/maven-archiver/src/site/apt/examples/classpath.apt 
(original)
+++ maven/shared/trunk/maven-archiver/src/site/apt/examples/classpath.apt Wed 
Jan  9 17:16:08 2013
@@ -90,7 +90,7 @@ Set Up The Classpath
 +-+
 Manifest-Version: 1.0
 Archiver-Version: Plexus Archiver
-Created-By: Apache Maven
+Created-By: Apache Maven ${maven.version}
 Built-By: ${user.name}
 Build-Jdk: ${java.version}
 Class-Path: plexus-utils-1.1.jar commons-lang-2.1.jar
@@ -150,7 +150,7 @@ Class-Path: plexus-utils-1.1.jar commons
 +-+
 Manifest-Version: 1.0
 Archiver-Version: Plexus Archiver
-Created-By: Apache Maven
+Created-By: Apache Maven ${maven.version}
 Built-By: ${user.name}
 Build-Jdk: ${java.version}
 Main-Class: fully.qualified.MainClass

Modified: maven/shared/trunk/maven-archiver/src/site/apt/examples/manifest.apt
URL: 
http://svn.apache.org/viewvc/maven/shared/trunk/maven-archiver/src/site/apt/examples/manifest.apt?rev=1430962r1=1430961r2=1430962view=diff
==
--- maven/shared/trunk/maven-archiver/src/site/apt/examples/manifest.apt 
(original)
+++ maven/shared/trunk/maven-archiver/src/site/apt/examples/manifest.apt Wed 
Jan  9 17:16:08 2013
@@ -33,7 +33,7 @@ Manifest
 +-+
 Manifest-Version: 1.0
 Archiver-Version: Plexus Archiver
-Created-By: Apache Maven
+Created-By: Apache Maven ${maven.version}
 Built-By: ${user.name}
 Build-Jdk: ${java.version}
 +-+
@@ -80,7 +80,7 @@ Build-Jdk: ${java.version}
 +-+
 Manifest-Version: 1.0
 Archiver-Version: Plexus Archiver
-Created-By: Apache Maven
+Created-By: Apache Maven ${maven.version}
 Built-By: ${user.name}
 Build-Jdk: ${java.version}
 Specification-Title: ${project.name}

Modified: 
maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestEntries.apt
URL: 
http://svn.apache.org/viewvc/maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestEntries.apt?rev=1430962r1=1430961r2=1430962view=diff
==
--- maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestEntries.apt 
(original)
+++ maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestEntries.apt 
Wed Jan  9 17:16:08 2013
@@ -69,7 +69,7 @@ Manifest Entries
 +-+
 Manifest-Version: 1.0
 Archiver-Version: Plexus Archiver
-Created-By: Apache Maven
+Created-By: Apache Maven ${maven.version}
 Built-By: ${user.name}
 Build-Jdk: ${java.version}
 mode: development

Modified: 
maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestSections.apt
URL: 
http://svn.apache.org/viewvc/maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestSections.apt?rev=1430962r1=1430961r2=1430962view=diff
==
--- 
maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestSections.apt 
(original)
+++ 
maven/shared/trunk/maven-archiver/src/site/apt/examples/manifestSections.apt 
Wed Jan  9 17:16:08 2013
@@ -75,7 +75,7 @@ Manifest Sections
 +-+
 Manifest-Version: 1.0
 Archiver-Version: Plexus Archiver
-Created-By: Apache Maven
+Created-By: Apache Maven ${maven.version}
 Built-By: ${user.name}
 Build-Jdk: ${java.version}
 




git commit: o Made sleep timeout configurable in forkModeIt for testing SUREFIRE-946

2013-01-09 Thread krosenvold
Updated Branches:
  refs/heads/master 4dfdb08cc - 350dd6fd7


o Made sleep timeout configurable in forkModeIt for testing SUREFIRE-946


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

Branch: refs/heads/master
Commit: 350dd6fd745f82dcdf071ff910665d8b388dfe7b
Parents: 4dfdb08
Author: Kristian Rosenvold krosenv...@apache.org
Authored: Wed Jan 9 18:20:50 2013 +0100
Committer: Kristian Rosenvold krosenv...@apache.org
Committed: Wed Jan 9 18:21:09 2013 +0100

--
 .../fork-mode/src/test/java/forkMode/Test1.java|4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/350dd6fd/surefire-integration-tests/src/test/resources/fork-mode/src/test/java/forkMode/Test1.java
--
diff --git 
a/surefire-integration-tests/src/test/resources/fork-mode/src/test/java/forkMode/Test1.java
 
b/surefire-integration-tests/src/test/resources/fork-mode/src/test/java/forkMode/Test1.java
index ca5af55..8dd3307 100644
--- 
a/surefire-integration-tests/src/test/resources/fork-mode/src/test/java/forkMode/Test1.java
+++ 
b/surefire-integration-tests/src/test/resources/fork-mode/src/test/java/forkMode/Test1.java
@@ -17,9 +17,9 @@ public class Test1
 public void test1()
 throws IOException, InterruptedException
 {
-Thread.sleep( 750 );
+int sleepLength = Integer.valueOf( System.getProperty( sleepLength, 
750 ));
+Thread.sleep(sleepLength);
 dumpPidFile( this );
-
 }
 
 public static void dumpPidFile( TestCase test )



git commit: [SUREFIRE-946] prevent hanging main process if forked process was killed (softly)

2013-01-09 Thread krosenvold
Updated Branches:
  refs/heads/master 350dd6fd7 - e54dbd810


[SUREFIRE-946] prevent hanging main process if forked process was killed 
(softly)

Fixed with extended IT


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

Branch: refs/heads/master
Commit: e54dbd810f62fe723800f11279b881bff244b707
Parents: 350dd6f
Author: agudian andreas.gud...@gmail.com
Authored: Sat Jan 5 21:13:12 2013 +0100
Committer: Kristian Rosenvold krosenv...@apache.org
Committed: Wed Jan 9 18:24:38 2013 +0100

--
 .../plugin/surefire/booterclient/ForkStarter.java  |  140 +++
 .../lazytestprovider/TestProvidingInputStream.java |   29 ++-
 .../surefire/booterclient/output/ForkClient.java   |   22 ++-
 .../maven/surefire/booter/ForkingRunListener.java  |3 +
 .../apache/maven/surefire/booter/ForkedBooter.java |   74 ++--
 .../maven/surefire/its/CrashDetectionIT.java   |7 +-
 6 files changed, 215 insertions(+), 60 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/e54dbd81/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
--
diff --git 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
index 620dfd4..892c6a9 100644
--- 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
+++ 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
@@ -21,6 +21,8 @@ package org.apache.maven.plugin.surefire.booterclient;
 
 import java.io.File;
 import java.io.IOException;
+import java.io.InputStream;
+import java.security.AccessControlException;
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
@@ -36,6 +38,7 @@ import java.util.concurrent.LinkedBlockingQueue;
 import java.util.concurrent.ThreadPoolExecutor;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicInteger;
+
 import org.apache.maven.plugin.surefire.AbstractSurefireMojo;
 import org.apache.maven.plugin.surefire.CommonReflector;
 import org.apache.maven.plugin.surefire.StartupReportConfiguration;
@@ -63,14 +66,13 @@ import org.apache.maven.surefire.report.StackTraceWriter;
 import org.apache.maven.surefire.suite.RunResult;
 import org.apache.maven.surefire.util.DefaultScanResult;
 
-
 /**
  * Starts the fork or runs in-process.
  * p/
  * Lives only on the plugin-side (not present in remote vms)
  * p/
  * Knows how to fork new vms and also how to delegate non-forking invocation 
to SurefireStarter directly
- *
+ * 
  * @author Jason van Zyl
  * @author Emmanuel Venisse
  * @author Brett Porter
@@ -80,6 +82,36 @@ import org.apache.maven.surefire.util.DefaultScanResult;
  */
 public class ForkStarter
 {
+/**
+ * Closes an InputStream
+ */
+private final class InputStreamCloser
+extends Thread
+{
+private InputStream testProvidingInputStream;
+
+public InputStreamCloser( InputStream testProvidingInputStream )
+{
+this.testProvidingInputStream = testProvidingInputStream;
+}
+
+@Override
+public void run()
+{
+if ( testProvidingInputStream != null )
+{
+try
+{
+testProvidingInputStream.close();
+}
+catch ( IOException e )
+{
+// ignore
+}
+}
+}
+}
+
 private final int forkedProcessTimeoutInSeconds;
 
 private final ProviderConfiguration providerConfiguration;
@@ -105,7 +137,6 @@ public class ForkStarter
 }
 };
 
-
 public ForkStarter( ProviderConfiguration providerConfiguration, 
StartupConfiguration startupConfiguration,
 ForkConfiguration forkConfiguration, int 
forkedProcessTimeoutInSeconds,
 StartupReportConfiguration startupReportConfiguration )
@@ -167,8 +198,9 @@ public class ForkStarter
 {
 
 ArrayListFutureRunResult results = new 
ArrayListFutureRunResult( forkCount );
-ExecutorService executorService = new ThreadPoolExecutor( forkCount, 
forkCount, 60, TimeUnit.SECONDS,
-  new 
ArrayBlockingQueueRunnable( forkCount ) );
+ExecutorService executorService =
+new ThreadPoolExecutor( forkCount, 

git commit: o Removed warning

2013-01-09 Thread krosenvold
Updated Branches:
  refs/heads/master e54dbd810 - 230c46a26


o Removed warning


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

Branch: refs/heads/master
Commit: 230c46a26eaffcb0da57fd317e6a7611ba2c8e96
Parents: e54dbd8
Author: Kristian Rosenvold krosenv...@apache.org
Authored: Wed Jan 9 19:03:30 2013 +0100
Committer: Kristian Rosenvold krosenv...@apache.org
Committed: Wed Jan 9 19:03:30 2013 +0100

--
 .../apache/maven/surefire/booter/ForkedBooter.java |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/230c46a2/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
--
diff --git 
a/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
 
b/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
index 4b04a99..5f711f4 100644
--- 
a/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
+++ 
b/surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java
@@ -72,7 +72,7 @@ public class ForkedBooter
 {
 originalIn.close();
 }
-catch ( IOException e )
+catch ( IOException ignore )
 {
 }