further testing indicates that you cannot set system properties in se…

…ttings.xml, and properties are ignored. So, jre.7 has to be set via commandline
jbosstools · May 20, 2015 · 9bb863b · 9bb863b
1 parent 7ba0ad7
commit 9bb863b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 16 deletions.
diff --git a/building/how_to_build_jbosstools_faq.adoc b/building/how_to_build_jbosstools_faq.adoc
@@ -9,21 +9,16 @@ See link:building/setup_development_environment.adoc#maven-settings[Configuring
 
 ### Java Home provided by the jbosstools.test.jre.7 system property does not exist! 
 
-Tests require the following paths to Java Home folders be defined in your `~/.m2/settings.xml` or via commandline:
+Some tests require the following paths to Java Home folders be defined via commandline:
 
-	<properties>
-	  <jbosstools.test.jre.5>/path/to/jdk1.7</jbosstools.test.jre.5>
-	  <jbosstools.test.jre.6>/path/to/jdk1.7</jbosstools.test.jre.6>
-	  <jbosstools.test.jre.7>/path/to/jdk1.7</jbosstools.test.jre.7>
-	  <jbosstools.test.jre.8>/path/to/jdk1.8</jbosstools.test.jre.8>
-	</properties>
+  mvn verify -Djbosstools.test.jre.7=/path/to/jdk1.7
 
 If not defined, you'll get this error:
 
 	junit.framework.AssertionFailedError: Java Home 
 	provided by the jbosstools.test.jre.7 system property does not exist.
 
-Note that for Jenkins jobs, this is set to a path similar to `/qa/tools/opt/jdk1.8.0_last`.
+Note that for Jenkins jobs, this is set to a path similar to `/qa/tools/opt/jdk1.7.0_last`.
 
 ##  How do I build via commandline?
 

diff --git a/building/setup_development_environment.adoc b/building/setup_development_environment.adoc
@@ -73,14 +73,6 @@ should see in your ~/.m2/settings.xml
 ----
 <settings xmlns="http://maven.apache.org/SETTINGS/1.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0 http://maven.apache.org/xsd/settings-1.0.0.xsd">
 
-  <!-- used by some JBoss Tools projects' tests, such as jbosstools-server -->
-  <properties>
-    <jbosstools.test.jre.5>/path/to/jdk1.7</jbosstools.test.jre.5>
-    <jbosstools.test.jre.6>/path/to/jdk1.7</jbosstools.test.jre.6>
-    <jbosstools.test.jre.7>/path/to/jdk1.7</jbosstools.test.jre.7>
-    <jbosstools.test.jre.8>/path/to/jdk1.8</jbosstools.test.jre.8>
-  </properties>
-
   <profiles>
     <profile>
     <id>jboss-default</id>