[tools] testbot/build: Simplify the --no-rm handling.
Francois Gouget
fgouget at codeweavers.com
Mon Mar 16 17:55:03 CDT 2020
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
testbot/bin/build/Reconfig.pl | 6 ++++--
testbot/bin/build/WineReconfig.pl | 2 +-
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/testbot/bin/build/Reconfig.pl b/testbot/bin/build/Reconfig.pl
index f0ed128d4e..62644e11a9 100755
--- a/testbot/bin/build/Reconfig.pl
+++ b/testbot/bin/build/Reconfig.pl
@@ -40,6 +40,8 @@ sub BEGIN
$::BuildEnv = 1;
}
+use File::Path;
+
use Build::Utils;
use WineTestBot::Config;
use WineTestBot::Missions;
@@ -53,13 +55,13 @@ sub BuildNative($)
{
my ($NoRm) = @_;
+ rmtree "$DataDir/wine-native" if (!$NoRm);
mkdir "$DataDir/wine-native" if (!-d "$DataDir/wine-native");
# Rebuild from scratch to make sure cruft will not accumulate
InfoMsg "\nRebuilding native tools\n";
my $CPUCount = GetCPUCount();
system("cd '$DataDir/wine-native' && set -x && ".
- ($NoRm ? "" : "rm -rf * && ") .
"time ../wine/configure --enable-win64 --without-x --without-freetype --disable-winetest && ".
"time make -j$CPUCount __tooldeps__");
@@ -77,6 +79,7 @@ sub BuildCross($$$)
my ($TaskMissions, $NoRm, $Build) = @_;
return 1 if (!$TaskMissions->{Builds}->{$Build});
+ rmtree "$DataDir/wine-$Build" if (!$NoRm);
mkdir "$DataDir/wine-$Build" if (!-d "$DataDir/wine-$Build");
# Rebuild from scratch to make sure cruft will not accumulate
@@ -84,7 +87,6 @@ sub BuildCross($$$)
my $CPUCount = GetCPUCount();
my $Host = ($Build eq "exe64" ? "x86_64-w64-mingw32" : "i686-w64-mingw32");
system("cd '$DataDir/wine-$Build' && set -x && ".
- ($NoRm ? "" : "rm -rf * && ") .
"time ../wine/configure --host=$Host --with-wine-tools=../wine-native --without-x --without-freetype --disable-winetest && ".
"time make -j$CPUCount buildtests");
if ($? != 0)
diff --git a/testbot/bin/build/WineReconfig.pl b/testbot/bin/build/WineReconfig.pl
index 835be4609b..ad0d429e1f 100755
--- a/testbot/bin/build/WineReconfig.pl
+++ b/testbot/bin/build/WineReconfig.pl
@@ -54,6 +54,7 @@ sub BuildWine($$$$;$)
my ($TaskMissions, $NoRm, $Build, $Extras, $WithWine) = @_;
return 1 if (!$TaskMissions->{Builds}->{$Build});
+ rmtree "$DataDir/wine-$Build" if (!$NoRm);
mkdir "$DataDir/wine-$Build" if (!-d "$DataDir/wine-$Build");
# If $NoRm is not set, rebuild from scratch to make sure cruft will not
@@ -62,7 +63,6 @@ sub BuildWine($$$$;$)
my $CPUCount = GetCPUCount();
$Extras .= " --with-wine64='$WithWine'" if (defined $WithWine);
system("cd '$DataDir/wine-$Build' && set -x && ".
- ($NoRm ? "" : "rm -rf * && ") .
"time ../wine/configure $Extras && ".
"time make -j$CPUCount");
if ($? != 0)
--
2.20.1
More information about the wine-devel
mailing list