It is useful to know the exact locale the tests were run with, particularly since the TestBot may have modified it to add the codeset. In particular this helps diagnosis in case Wine does not recognize the locale (i.e. if WineTest still shows 0409).
Signed-off-by: Francois Gouget fgouget@codeweavers.com --- testbot/bin/build/WineTest.pl | 1 + 1 file changed, 1 insertion(+)
diff --git a/testbot/bin/build/WineTest.pl b/testbot/bin/build/WineTest.pl index 4947a12a62..de9f0eda3f 100755 --- a/testbot/bin/build/WineTest.pl +++ b/testbot/bin/build/WineTest.pl @@ -86,6 +86,7 @@ sub SetupTest($$)
my $BaseName = SetupWineEnvironment($Mission); InfoMsg "\nRunning $Test in the $BaseName Wine\n"; + InfoMsg "LANG=$ENV{LANG}\n"; return $BaseName; }
Signed-off-by: Francois Gouget fgouget@codeweavers.com ---
This one does not technically depend on the previous patch but if we don't log $LANG there's not much point trying to highlight the LANG= lines.
testbot/lib/WineTestBot/LogUtils.pm | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm index fee4278105..b75607af89 100644 --- a/testbot/lib/WineTestBot/LogUtils.pm +++ b/testbot/lib/WineTestBot/LogUtils.pm @@ -130,6 +130,8 @@ sub GetLogLineCategory($)
if (# Build messages $Line =~ /^+ \S/ or + $Line =~ /^LANG=/ or + $Line =~ /^Running (?:the tests|WineTest) / or $Line =~ /^Task: (?:ok|tests)/) { return "info";