JENKINS-26848 Does not actually randomize DISPLAY

Change the wording in the help to better reflect the actual display name generation
jenkinsci · Feb 8, 2015 · 0aa3519 · 0aa3519
1 parent e7d51fb
commit 0aa3519
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/xvfb/XvfbBuildWrapper.java b/src/main/java/org/jenkinsci/plugins/xvfb/XvfbBuildWrapper.java
@@ -339,7 +339,7 @@ public Void call() throws InterruptedException {
     /** Name of the installation used in a configured job. */
     private final String  installationName;
 
-    /** X11 DISPLAY name, if NULL chosen by random. */
+    /** X11 DISPLAY name, if NULL chosen based on current executor number. */
     private final Integer displayName;
 
     /** Xvfb screen argument, in the form WxHxD (width x height x pixel depth), i.e. 800x600x8. */

diff --git a/src/main/resources/org/jenkinsci/plugins/xvfb/XvfbBuildWrapper/help-displayName.html b/src/main/resources/org/jenkinsci/plugins/xvfb/XvfbBuildWrapper/help-displayName.html
@@ -23,5 +23,6 @@
 SUCH DAMAGE.
 -->
 <div>Ordinal of the display Xvfb will be running on, if left empty
-	(default) choosen by random. Use only if you know what you're doing,
-	could leed to clashes with other builds.</div>
+	(default) choosen based on current build executor number. Use only
+	if you know what you're doing, could leed to clashes with other
+	builds.</div>