Module: tools
Branch: master
Commit: ade2cc55a64580bc20ca60ff3ec841c2b0cd1945
URL: https://source.winehq.org/git/tools.git/?a=commit;h=ade2cc55a64580bc20ca60f…
Author: Francois Gouget <fgouget(a)codeweavers.com>
Date: Wed Oct 2 07:02:22 2019 +0200
testbot/LibvirtTool: Only allow dashes in the locale snapshot names.
Matching the Windows locale syntax makes it easier to interface with
SetWinLocale.
Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>
---
testbot/bin/LibvirtTool.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/testbot/bin/LibvirtTool.pl b/testbot/bin/LibvirtTool.pl
index 415cdc5..d590191 100755
--- a/testbot/bin/LibvirtTool.pl
+++ b/testbot/bin/LibvirtTool.pl
@@ -493,7 +493,7 @@ sub Revert()
my $Domain = $VM->GetDomain();
if (($VM->Type eq "win32" or $VM->Type eq "win64") and
!$Domain->HasSnapshot($DomainSnapshot) and
- $DomainSnapshot =~ s/-([a-z]{2})[_-]([A-Z]{2})$//)
+ $DomainSnapshot =~ s/-([a-z]{2})-([A-Z]{2})$//)
{
# Add some extra time to set up the VM locale and reboot it
$ExtraTimeout += $VMToolTimeout;
Module: tools
Branch: master
Commit: 980fc8a758b729f1181c0a22b270b1e67ff31935
URL: https://source.winehq.org/git/tools.git/?a=commit;h=980fc8a758b729f1181c0a2…
Author: Francois Gouget <fgouget(a)codeweavers.com>
Date: Wed Oct 2 07:02:15 2019 +0200
testbot/LibvirtTool: Only create locale snapshots for Windows VMs.
This is the only supported and needed case.
Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>
---
testbot/bin/LibvirtTool.pl | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/testbot/bin/LibvirtTool.pl b/testbot/bin/LibvirtTool.pl
index 5438f29..415cdc5 100755
--- a/testbot/bin/LibvirtTool.pl
+++ b/testbot/bin/LibvirtTool.pl
@@ -491,7 +491,8 @@ sub Revert()
my ($SetLocale, $CreateSnapshot);
my $Domain = $VM->GetDomain();
- if (!$Domain->HasSnapshot($DomainSnapshot) and
+ if (($VM->Type eq "win32" or $VM->Type eq "win64") and
+ !$Domain->HasSnapshot($DomainSnapshot) and
$DomainSnapshot =~ s/-([a-z]{2})[_-]([A-Z]{2})$//)
{
# Add some extra time to set up the VM locale and reboot it