From 5dbefff441c32cf6d6199fd8ab279e98d8063af9 Mon Sep 17 00:00:00 2001
From: Alexandre Montplaisir <alexmonthy@efficios.com>
Date: Mon, 24 Oct 2016 19:14:37 -0400
Subject: [PATCH] releng: Do not build the testing feature when some tests are
 skipped

The testing feature, as well as the "alltests" plugin, cannot be built
when -Dskip-automated-ui-tests is on, as they require all SWTBot tests.
This means the the releng-site cannot be built either. All of them
should only be run when the tests are active.

Change-Id: I9d89444818c2373239dbcd04e8749ab6e033038c
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/83830
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
---
 releng/pom.xml | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/releng/pom.xml b/releng/pom.xml
index eb12663f5c..45758e0060 100644
--- a/releng/pom.xml
+++ b/releng/pom.xml
@@ -25,10 +25,7 @@
   <name>Trace Compass Release Engineering Parent</name>
 
   <modules>
-    <module>org.eclipse.tracecompass.alltests</module>
-    <module>org.eclipse.tracecompass.releng-site</module>
     <module>org.eclipse.tracecompass.target</module>
-    <module>org.eclipse.tracecompass.testing</module>
   </modules>
 
   <profiles>
@@ -40,7 +37,14 @@
         </property>
       </activation>
       <modules>
+        <!--
+          The releng-site requires the testing feature, and the testing feature
+          requires all SWTBot tests, so only build those when we build those tests.
+        -->
+        <module>org.eclipse.tracecompass.alltests</module>
         <module>org.eclipse.tracecompass.integration.swtbot.tests</module>
+        <module>org.eclipse.tracecompass.releng-site</module>
+        <module>org.eclipse.tracecompass.testing</module>
       </modules>
     </profile>
   </profiles>
-- 
2.34.1