[PATCH] testbot/web: Fix the check for which log to show in full.

Francois Gouget fgouget at codeweavers.com
Fri Nov 23 09:23:14 CST 2018


The name of the current log has been decoded so it should not be
compared to a uri_escape()'d log filename.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---

This is not an issue until a locale modifier such as '@latin' ends up 
in the filename.

 testbot/web/JobDetails.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testbot/web/JobDetails.pl b/testbot/web/JobDetails.pl
index de2f9ccb40..469138f0bd 100644
--- a/testbot/web/JobDetails.pl
+++ b/testbot/web/JobDetails.pl
@@ -247,7 +247,7 @@ sub InitMoreInfo($)
     foreach my $Log (@{GetLogFileNames($TaskDir, 1)})
     {
       push @{$More->{$Key}->{Logs}}, $Log;
-      $More->{$Key}->{Full} = $Log if (uri_escape($Log) eq $Value);
+      $More->{$Key}->{Full} = $Log if ($Log eq $Value);
     }
     $More->{$Key}->{Full} ||= "";
   }
-- 
2.19.1




More information about the wine-devel mailing list