Links to Trac from TestFailures page don't always show all relevant revisions
Created attachment 104168 [details] Patch
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 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.
(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.
Committed r93224: <http://trac.webkit.org/changeset/93224>