From 37931306d54066f222def676e9f797afa8f40260 Mon Sep 17 00:00:00 2001
From: Trustin Lee <t@motd.kr>
Date: Thu, 22 May 2014 18:53:09 +0900
Subject: [PATCH] Use maven-antrun-plugin instead of exec-maven-plugin

Motivation:

exec-maven-plugin does not flush stdout and stderr, making the console
output from the examples invisible to users

Modification:

Use maven-antrun-plugin instead

Result:

A user sees the output from the examples immediately.
---
 all/pom.xml     |  2 +-
 example/pom.xml | 23 ++++++++++-------------
 pom.xml         |  7 +------
 run-example.sh  |  2 +-
 4 files changed, 13 insertions(+), 21 deletions(-)

diff --git a/all/pom.xml b/all/pom.xml
index e5630a1e44..8b1ab45131 100644
--- a/all/pom.xml
+++ b/all/pom.xml
@@ -292,7 +292,7 @@
             <goals>
               <goal>run</goal>
             </goals>
-            <configuration>
+            <configuration combine.self="override">
               <target>
                 <taskdef resource="net/sf/antcontrib/antlib.xml" />
                 <propertyselector property="versions" match="^(${project.groupId}:(?!netty-example)[^:]+:jar)$" select="\1" />
diff --git a/example/pom.xml b/example/pom.xml
index 233277f444..36bd3530b1 100644
--- a/example/pom.xml
+++ b/example/pom.xml
@@ -104,20 +104,17 @@
   <build>
     <plugins>
       <plugin>
-        <groupId>org.codehaus.mojo</groupId>
-        <artifactId>exec-maven-plugin</artifactId>
+        <artifactId>maven-antrun-plugin</artifactId>
         <configuration>
-          <executable>${java.home}/bin/java</executable>
-          <commandlineArgs>
-            ${argLine.common}
-            ${argLine.bootcp}
-            ${argLine.leak}
-            ${argLine.coverage}
-            -classpath %classpath
-            ${argLine.example}
-            ${exampleClass}
-          </commandlineArgs>
-          <classpathScope>runtime</classpathScope>
+          <target>
+            <taskdef resource="net/sf/antcontrib/antlib.xml" />
+            <propertyregex
+              property="mergedArgs" regexp="[\r\n\s]+" replace=" " global="true"
+              input="${argLine.common} ${argLine.bootcp} ${argLine.leak} ${argLine.coverage} ${argLine.example}" />
+            <java classname="${exampleClass}" classpathref="maven.runtime.classpath" fork="true">
+              <jvmarg line="${mergedArgs}"/>
+            </java>
+          </target>
         </configuration>
       </plugin>
     </plugins>
diff --git a/pom.xml b/pom.xml
index 678e064513..eef3afedd5 100644
--- a/pom.xml
+++ b/pom.xml
@@ -857,7 +857,7 @@
             <goals>
               <goal>run</goal>
             </goals>
-            <configuration>
+            <configuration combine.self="override">
               <target>
                 <taskdef resource="net/sf/antcontrib/antlib.xml" />
 
@@ -1065,11 +1065,6 @@
           <artifactId>build-helper-maven-plugin</artifactId>
           <version>1.8</version>
         </plugin>               
-        <plugin>
-          <groupId>org.codehaus.mojo</groupId>
-          <artifactId>exec-maven-plugin</artifactId>
-          <version>1.3</version>
-        </plugin>
         <plugin>
           <groupId>org.fusesource.hawtjni</groupId>
           <artifactId>maven-hawtjni-plugin</artifactId>
diff --git a/run-example.sh b/run-example.sh
index 4344865350..6b63221cd7 100755
--- a/run-example.sh
+++ b/run-example.sh
@@ -58,5 +58,5 @@ fi
 
 cd "`dirname "$0"`"/example
 echo "[INFO] Running: $EXAMPLE ($EXAMPLE_CLASS $EXAMPLE_ARGS)"
-exec mvn -nsu compile exec:exec -DargLine.example="$EXAMPLE_ARGS" -DexampleClass="$EXAMPLE_CLASS"
+exec mvn -nsu compile antrun:run -Dcheckstyle.skip=true -Dantfile=src/antrun/run-example.xml -DargLine.example="$EXAMPLE_ARGS" -DexampleClass="$EXAMPLE_CLASS"