Bug 66373 - Links to Trac from TestFailures page don't always show all relevant revisions
Summary: Links to Trac from TestFailures page don't always show all relevant revisions
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adam Roben (:aroben)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-17 06:38 PDT by Adam Roben (:aroben)
Modified: 2011-08-17 11:18 PDT (History)
3 users (show)

See Also:


Attachments
Patch (8.07 KB, patch)
2011-08-17 06:38 PDT, Adam Roben (:aroben)
dbates: review+
aroben: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-08-17 06:38:13 PDT
Links to Trac from TestFailures page don't always show all relevant revisions
Comment 1 Adam Roben (:aroben) 2011-08-17 06:38:41 PDT
Created attachment 104168 [details]
Patch
Comment 2 Daniel Bates 2011-08-17 09:19:01 PDT
Comment on attachment 104168 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=104168&action=review

r=me

> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js:140
> +        limit: endRevision - startRevision + 2, // For some reason, Trac requires limit to be 5 in order to display 4 revisions (e.g.).

What's the example? (since you mentioned "(e.g.)"). I would suggest mentioning the Trac version we are using (0.11.7) in this comment so as to provide a bit more context on this Trac bug. Even better, if we know the Trac ticket associated with this issue then we should list it in this comment (*). Do we know if this bug has been resolved in a newer version of Trac, say Trac 0.12? We could also file a WebKit bug to track this Trac issue.

(*) <http://trac.edgewall.org/report/>
Comment 3 Adam Roben (:aroben) 2011-08-17 09:40:32 PDT
Comment on attachment 104168 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=104168&action=review

>> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js:140
>> +        limit: endRevision - startRevision + 2, // For some reason, Trac requires limit to be 5 in order to display 4 revisions (e.g.).
> 
> What's the example? (since you mentioned "(e.g.)"). I would suggest mentioning the Trac version we are using (0.11.7) in this comment so as to provide a bit more context on this Trac bug. Even better, if we know the Trac ticket associated with this issue then we should list it in this comment (*). Do we know if this bug has been resolved in a newer version of Trac, say Trac 0.12? We could also file a WebKit bug to track this Trac issue.
> 
> (*) <http://trac.edgewall.org/report/>

The example is what I mentioned in the comment: limit must be 5 in order to display 4 revisions. I could have also mentioned that limit must be 13 in order to display 11 revisions, etc. Do you have a suggestion for how to make this clearer?

I filed http://trac.edgewall.org/ticket/10317 about this issue. It does not seem to be fixed in newer versions of Trac.
Comment 4 Daniel Bates 2011-08-17 09:47:20 PDT
(In reply to comment #3)
> (From update of attachment 104168 [details])
> The example is what I mentioned in the comment: limit must be 5 in order to display 4 revisions. I could have also mentioned that limit must be 13 in order to display 11 revisions, etc. Do you have a suggestion for how to make this clearer?

I just suggest referencing the Trac ticket in the comment now that you filed one. I realized your comment was the example. I'm not familiar with your usage of "e.g" . The form I'm familiar with is e.g. [example], where [example] is the example test.

> 
> I filed http://trac.edgewall.org/ticket/10317 about this issue. It does not seem to be fixed in newer versions of Trac.

Thanks for filing this. I'm sure Trac is happy as well.
Comment 5 Adam Roben (:aroben) 2011-08-17 11:18:14 PDT
Committed r93224: <http://trac.webkit.org/changeset/93224>