From: Francois Gouget <fgouget@codeweavers.com>
Subject: [Tools] testbot: Improve the $VM::GetHost() documentation.
Message-Id: <E1eI9q8-0006Zp-D0@amboise>
Date: Fri, 24 Nov 2017 10:00:04 +0100 (CET)

Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
---
 testbot/lib/WineTestBot/VMs.pm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/testbot/lib/WineTestBot/VMs.pm b/testbot/lib/WineTestBot/VMs.pm
index 7bd6d64b..b9313429 100644
--- a/testbot/lib/WineTestBot/VMs.pm
+++ b/testbot/lib/WineTestBot/VMs.pm
@@ -179,7 +179,9 @@ sub GetHost($)
 {
   my ($self) = @_;
 
-  # The URI is of the form protocol://user@hostname/hypervisor-specific-data
+  # VirtURI is of the form proto://user@hostname:port/hypervisor-specific-data
+  # This returns the 'hostname:port' part. Note that the port can be used to
+  # tunnel connections to different VM hosts and thus must be included.
   return $1 if ($self->VirtURI =~ m%^[^:]+://(?:[^/@]*@)?([^/]+)/%);
   return "localhost";
 }

-- 
2.15.0