From 70ecce2a0ffb5216c25cdf040dc9efe8fc467314 Mon Sep 17 00:00:00 2001
From: Khalid Shakir <kshakir@broadinstitute.org>
Date: Sat, 15 Feb 2014 16:46:44 +0800
Subject: [PATCH] Fixed scope for test-jar depedencies.

---
 public/gatk-package/pom.xml    |  1 +
 public/queue-framework/pom.xml | 11 +++++++++++
 public/queue-package/pom.xml   |  2 ++
 3 files changed, 14 insertions(+)

diff --git a/public/gatk-package/pom.xml b/public/gatk-package/pom.xml
index 881058949..9272cf983 100644
--- a/public/gatk-package/pom.xml
+++ b/public/gatk-package/pom.xml
@@ -57,6 +57,7 @@
             <artifactId>gatk-framework</artifactId>
             <version>${project.version}</version>
             <type>test-jar</type>
+            <scope>test</scope>
         </dependency>
 
         <dependency>
diff --git a/public/queue-framework/pom.xml b/public/queue-framework/pom.xml
index 2f3f7b59a..670750fd0 100644
--- a/public/queue-framework/pom.xml
+++ b/public/queue-framework/pom.xml
@@ -61,6 +61,14 @@
             <scope>runtime</scope>
         </dependency>
 
+        <!--
+        NOTE: Because exec-maven-plugin is used, this aggregator dependency
+        will cause mvn compile to not work. Use mvn test-compile for now.
+
+        See:
+          http://stackoverflow.com/questions/4786881/why-is-test-jar-dependency-required-for-mvn-compile
+          http://jira.codehaus.org/browse/MEXEC-91
+        -->
         <dependency>
             <groupId>${project.groupId}</groupId>
             <artifactId>gatk-framework</artifactId>
@@ -78,6 +86,9 @@
 
     <build>
         <plugins>
+            <!--
+            See note above about exec plugin breaking mvn compile
+            -->
             <plugin>
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>exec-maven-plugin</artifactId>
diff --git a/public/queue-package/pom.xml b/public/queue-package/pom.xml
index eda2fd98f..27b6fae6d 100644
--- a/public/queue-package/pom.xml
+++ b/public/queue-package/pom.xml
@@ -88,6 +88,7 @@
             <artifactId>queue-framework</artifactId>
             <version>${project.version}</version>
             <type>test-jar</type>
+            <scope>test</scope>
         </dependency>
 
         <dependency>
@@ -95,6 +96,7 @@
             <artifactId>gatk-framework</artifactId>
             <version>${project.version}</version>
             <type>test-jar</type>
+            <scope>test</scope>
         </dependency>
 
         <dependency>