winetest: Add a link to the test summary page that specifies the test configuration details, the dll versions, etc.

Francois Gouget fgouget at codeweavers.com
Mon Feb 25 08:53:42 CST 2013


Clarify two other link names to avoid confusion.
---

I did not test this patch so it should be verified carefully before 
committing. I recently found myself annoyed that I could not go from a 
page such as this one:

http://test.winehq.org/data/209b58c64e6181c9bf448cb4f3130d3ba705c5d2/win7_newtb-w7pro64-32/user32:listbox.html

to what I call that test's summary page:

http://test.winehq.org/data/209b58c64e6181c9bf448cb4f3130d3ba705c5d2/win7_newtb-w7pro64-32/version.html

Of course 'summary' referred instead to the page with all the 'build 
results' which I found very confusing. So I renamed it to just that 
('build results', not 'confusing' <g>). Then I renamed the 'index' link 
to 'home' since it points to the winetest site's landing page (aka its 
'home' page).


 winetest/dissect |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/winetest/dissect b/winetest/dissect
index f2e18a2..f3fb6bb 100755
--- a/winetest/dissect
+++ b/winetest/dissect
@@ -407,8 +407,9 @@ for (my $i = 0; $i <= $#boxes; $i++)
     print FILE "<div class=\"navbar\">";
     print FILE $boxes[$i]->{prev} ? "<a href=\"./$boxes[$i]->{prev}.html\">prev</a> | " : "prev | ";
     print FILE $boxes[$i]->{next} ? "<a href=\"./$boxes[$i]->{next}.html\">next</a> | " : "next | ";
+    printf FILE "<a href=\"version.html\">summary</a> | ";
     printf FILE "<a href=\"report.html%s\">full report</a> | ", $i ? "#$boxes[$i]->{id}" : "";
-    print FILE "<a href=\"..\">summary</a> | <a href=\"../..\">index</a></div>\n";
+    print FILE "<a href=\"..\">build results</a> | <a href=\"../..\">home</a></div>\n";
 
     printf FILE "<div id=\"%s\" class=\"%s\">\n", $boxes[$i]->{id}, $boxes[$i]->{class};
     printf FILE "<div class=\"updownbar\">%s</div>\n", $boxes[$i]->{title};
-- 
1.7.10.4




More information about the wine-patches mailing list