From 6d857053555a70381dbb21edf246de33b621eab2 Mon Sep 17 00:00:00 2001 From: James Nord Date: Thu, 12 Oct 2017 18:10:56 +0100 Subject: [PATCH] Don't set stupid defaults. I doubt any plugin cares about multicast dns discovery so just disable it rather than giving it a non standard port (which could lead to announcments on a public wifi network) ALso reuseForks is true by default so do not set it and allow users to use the standard -DreuseForks=false without having to change the plugin cofnig. On the same note concurrency is only ever used in surefires forkCount - yet another non standard property. So stop that nonsense and use forkCount (which is initialized to concurrency for backwards compatability) --- pom.xml | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/pom.xml b/pom.xml index fbdff5a428..c2a49b204c 100644 --- a/pom.xml +++ b/pom.xml @@ -59,7 +59,8 @@ ${java.level} - 1 + 1 + ${concurrency} false @@ -689,11 +690,9 @@ hudson.udp - 33849 + -1 - true - ${concurrency}