[PATCH] testbot: There is no point unsetting $ENV.

Francois Gouget fgouget at codeweavers.com
Wed Jan 9 03:03:52 CST 2019


It is not a special environment variable or perl value and typically
does not even exist.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/bin/Engine.pl               | 1 -
 testbot/bin/Janitor.pl              | 1 -
 testbot/bin/LibvirtTool.pl          | 2 --
 testbot/bin/WineRunBuild.pl         | 1 -
 testbot/bin/WineRunReconfig.pl      | 1 -
 testbot/bin/WineRunTask.pl          | 1 -
 testbot/bin/WineRunWineTest.pl      | 1 -
 testbot/bin/WineSendLog.pl          | 1 -
 testbot/bin/build/Build.pl          | 1 -
 testbot/bin/build/Reconfig.pl       | 1 -
 testbot/bin/build/WineReconfig.pl   | 1 -
 testbot/bin/build/WineTest.pl       | 1 -
 testbot/scripts/CheckWineTestBot.pl | 1 -
 13 files changed, 14 deletions(-)

diff --git a/testbot/bin/Engine.pl b/testbot/bin/Engine.pl
index 79b25fac..9ffdf968 100755
--- a/testbot/bin/Engine.pl
+++ b/testbot/bin/Engine.pl
@@ -742,7 +742,6 @@ sub main()
   }
 
   $ENV{PATH} = "/usr/bin:/bin";
-  delete $ENV{ENV};
   $SIG{CHLD} = \&REAPER;
 
   $WineTestBot::Engine::Notify::RunningInEngine = 1;
diff --git a/testbot/bin/Janitor.pl b/testbot/bin/Janitor.pl
index 12084512..e7662852 100755
--- a/testbot/bin/Janitor.pl
+++ b/testbot/bin/Janitor.pl
@@ -77,7 +77,6 @@ sub Error(@)
 #
 
 $ENV{PATH} = "/usr/bin:/bin";
-delete $ENV{ENV};
 
 # Grab the command line options
 my ($Usage, $DryRun);
diff --git a/testbot/bin/LibvirtTool.pl b/testbot/bin/LibvirtTool.pl
index e6742f4c..a078fcbd 100755
--- a/testbot/bin/LibvirtTool.pl
+++ b/testbot/bin/LibvirtTool.pl
@@ -88,8 +88,6 @@ EOF
 #
 
 $ENV{PATH} = "/usr/bin:/bin";
-delete $ENV{ENV};
-
 
 # Grab the command line options
 my ($Usage, $Action, $VMKey);
diff --git a/testbot/bin/WineRunBuild.pl b/testbot/bin/WineRunBuild.pl
index 68e6be4c..b1349f0d 100755
--- a/testbot/bin/WineRunBuild.pl
+++ b/testbot/bin/WineRunBuild.pl
@@ -73,7 +73,6 @@ sub Error(@)
 #
 
 $ENV{PATH} = "/usr/bin:/bin";
-delete $ENV{ENV};
 
 my $Usage;
 sub ValidateNumber($$)
diff --git a/testbot/bin/WineRunReconfig.pl b/testbot/bin/WineRunReconfig.pl
index 9bb402a6..2c8acdd6 100755
--- a/testbot/bin/WineRunReconfig.pl
+++ b/testbot/bin/WineRunReconfig.pl
@@ -74,7 +74,6 @@ sub Error(@)
 #
 
 $ENV{PATH} = "/usr/bin:/bin";
-delete $ENV{ENV};
 
 my $Usage;
 sub ValidateNumber($$)
diff --git a/testbot/bin/WineRunTask.pl b/testbot/bin/WineRunTask.pl
index fea8c03f..e6112fd8 100755
--- a/testbot/bin/WineRunTask.pl
+++ b/testbot/bin/WineRunTask.pl
@@ -104,7 +104,6 @@ sub TakeScreenshot($$)
 #
 
 $ENV{PATH} = "/usr/bin:/bin";
-delete $ENV{ENV};
 
 my $Usage;
 sub ValidateNumber($$)
diff --git a/testbot/bin/WineRunWineTest.pl b/testbot/bin/WineRunWineTest.pl
index 5ce1448e..8792be39 100755
--- a/testbot/bin/WineRunWineTest.pl
+++ b/testbot/bin/WineRunWineTest.pl
@@ -103,7 +103,6 @@ sub TakeScreenshot($$)
 #
 
 $ENV{PATH} = "/usr/bin:/bin";
-delete $ENV{ENV};
 
 my $Usage;
 sub ValidateNumber($$)
diff --git a/testbot/bin/WineSendLog.pl b/testbot/bin/WineSendLog.pl
index a651c097..1d13454f 100755
--- a/testbot/bin/WineSendLog.pl
+++ b/testbot/bin/WineSendLog.pl
@@ -407,7 +407,6 @@ EOF
 #
 
 $ENV{PATH} = "/usr/bin:/bin";
-delete $ENV{ENV};
 
 my $Usage;
 sub ValidateNumber($$)
diff --git a/testbot/bin/build/Build.pl b/testbot/bin/build/Build.pl
index b50a25ca..894686da 100755
--- a/testbot/bin/build/Build.pl
+++ b/testbot/bin/build/Build.pl
@@ -115,7 +115,6 @@ sub BuildTestExecutables($$$)
 #
 
 $ENV{PATH} = "/usr/lib/ccache:/usr/bin:/bin";
-delete $ENV{ENV};
 
 my ($Usage, $PatchFile, $MissionStatement);
 while (@ARGV)
diff --git a/testbot/bin/build/Reconfig.pl b/testbot/bin/build/Reconfig.pl
index b720788e..c43fbda3 100755
--- a/testbot/bin/build/Reconfig.pl
+++ b/testbot/bin/build/Reconfig.pl
@@ -111,7 +111,6 @@ sub UpdateWineBuilds($$)
 #
 
 $ENV{PATH} = "/usr/lib/ccache:/usr/bin:/bin";
-delete $ENV{ENV};
 
 my ($Usage, $OptUpdate, $OptBuild, $OptNoRm, $MissionStatement);
 while (@ARGV)
diff --git a/testbot/bin/build/WineReconfig.pl b/testbot/bin/build/WineReconfig.pl
index 77810958..4e03c314 100755
--- a/testbot/bin/build/WineReconfig.pl
+++ b/testbot/bin/build/WineReconfig.pl
@@ -143,7 +143,6 @@ sub UpdateWinePrefixes($)
 #
 
 $ENV{PATH} = "/usr/lib/ccache:/usr/bin:/bin";
-delete $ENV{ENV};
 
 my ($Usage, $OptUpdate, $OptBuild, $OptNoRm, $OptAddOns, $OptWinePrefix, $MissionStatement);
 while (@ARGV)
diff --git a/testbot/bin/build/WineTest.pl b/testbot/bin/build/WineTest.pl
index b588b8d3..eb0012db 100755
--- a/testbot/bin/build/WineTest.pl
+++ b/testbot/bin/build/WineTest.pl
@@ -210,7 +210,6 @@ sub TestExe($$$)
 #
 
 $ENV{PATH} = "/usr/lib/ccache:/usr/bin:/bin";
-delete $ENV{ENV};
 
 my $Action = "";
 my ($Usage, $OptNoSubmit, $MissionStatement, $FileName, $BaseTag);
diff --git a/testbot/scripts/CheckWineTestBot.pl b/testbot/scripts/CheckWineTestBot.pl
index a0c0bda5..9c2e2911 100755
--- a/testbot/scripts/CheckWineTestBot.pl
+++ b/testbot/scripts/CheckWineTestBot.pl
@@ -39,7 +39,6 @@ use WineTestBot::Config;
 use WineTestBot::Engine::Notify;
 
 $ENV{PATH} = "/sbin:/usr/sbin:/usr/bin:/bin";
-delete $ENV{ENV};
 
 my $rc = 0;
 if (! PingEngine())
-- 
2.20.1



More information about the wine-devel mailing list