Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
---
testbot/bin/build/WineReconfig.pl | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/testbot/bin/build/WineReconfig.pl b/testbot/bin/build/WineReconfig.pl
index 2416eebfa4..63d72e89d3 100755
--- a/testbot/bin/build/WineReconfig.pl
+++ b/testbot/bin/build/WineReconfig.pl
@@ -37,6 +37,9 @@ sub BEGIN
$::BuildEnv = 1;
}
+use File::Basename;
+use File::Path;
+
use Build::Utils;
use WineTestBot::Config;
@@ …
[View More]-95,6 +98,16 @@ sub UpdateWinePrefixes($)
{
my ($Targets) = @_;
+ # Make sure no obsolete wineprefix is left behind in case WineReconfig
+ # is called with a different set of targets
+ foreach my $Dir (glob("'$DataDir/wineprefix-*'"))
+ {
+ if (basename($Dir) =~ /^(wineprefix-[a-zA-Z0-9\(a)_.-]+)$/) # untaint
+ {
+ rmtree("$DataDir/$1");
+ }
+ }
+
# Set up brand new WinePrefixes ready for use for testing.
# This way we do it once instead of doing it for every test, thus saving
# time. Note that this requires using a different wineprefix for each build.
--
2.19.1
[View Less]
Hi,
Any comment on this set of patches? (patches 2 - 6)
Let me know if you have a different vision on how they should be
implemented, or any comments
El lun., 15 oct. 2018 a las 22:46, Marvin (<testbot(a)winehq.org>) escribió:
> Thank you for your contribution to Wine!
>
> This is an automated notification to let you know that your patch has
> been reviewed and its status set to "Assigned".
>
> This means that the patch has been assigned to a reviewer. If the
> …
[View More]reviewer signs off on the patch, it will be committed, otherwise
> you'll receive advice on how to improve it. If you don't hear anything
> after a couple of weeks, please resend the patch.
>
[View Less]