[2/2] testbot/WineRunTask: Set the system time on the Windows VM before running the tests.

Francois Gouget fgouget at codeweavers.com
Thu Mar 6 08:26:34 CST 2014


---

And this should fix bug 33788.
http://bugs.winehq.org/show_bug.cgi?id=33788

Note that we only set the system time on the Windows VMs. For the build 
VM it should not matter and we would not have permission anyway as the 
build is done in a non-root account. One day we'll get this permission 
issue on Windows VMs too once we add non-administrator/non-power test 
configurations.

 testbot/bin/WineRunTask.pl | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/testbot/bin/WineRunTask.pl b/testbot/bin/WineRunTask.pl
index 3caa673..3b5c17e 100755
--- a/testbot/bin/WineRunTask.pl
+++ b/testbot/bin/WineRunTask.pl
@@ -228,6 +228,11 @@ if (defined $ErrMessage)
   FatalError "Can't set VM status to running: $ErrMessage\n",
              $FullErrFileName, $Job, $Step, $Task;
 }
+if (!$TA->SetTime())
+{
+    # Not a fatal error
+    LogTaskError("Unable to set the VM system time (". $TA->GetVersion() ."): ". $TA->GetLastError() ."\n", $FullErrFileName);
+}
 
 my $FileType = $Step->FileType;
 if ($FileType ne "exe32" && $FileType ne "exe64")
-- 
1.8.5.3




More information about the wine-patches mailing list