Closed
Bug 1434410
Opened 6 years ago
Closed 6 years ago
Include information about WebRender revision in the produced captures automatically
Categories
(Core :: Graphics: WebRender, enhancement)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: kvark, Assigned: kvark)
Details
Attachments
(1 file, 1 obsolete file)
2.58 KB,
patch
|
kats
:
review+
|
Details | Diff | Splinter Review |
As it turns out, getting the revision and passing it along every capture is a tedious task that is begging to be automated.
Assignee | ||
Comment 1•6 years ago
|
||
This patch will conflict with any WR updates until merged!
Attachment #8946794 -
Flags: review?(bugmail)
Assignee | ||
Comment 2•6 years ago
|
||
Also, a try push just in case: https://treeherder.mozilla.org/#/jobs?repo=try&revision=631f90fe4d2c6055006d14da35c4bea1d33dc2e0
Assignee: nobody → kvark
Comment 3•6 years ago
|
||
Comment on attachment 8946794 [details] [diff] [review] Include WR revision Review of attachment 8946794 [details] [diff] [review]: ----------------------------------------------------------------- Commit message needs more detail. Something like "Move the WebRender revision to a separate file and include it in captures". Also remove the try syntax. For bonus points add this to your .gitconfig: [alias] hgp = "show -M -C --binary --format=\"# HG changeset patch%n# User %an <%ae>%n%B\" -U8" and then use "git hgp <cset>" to export the patch so that it will conform to the expected format for landing. ::: gfx/doc/README.webrender @@ +178,2 @@ > > +The current WebRender revision can be found in gfx/webrender_bindings/revision.txt file. Earlier in this file there is a bit that says "update the revision number at the bottom of this file" or similar, please update that text to also point to revision.txt instead
Attachment #8946794 -
Flags: review?(bugmail)
Assignee | ||
Comment 4•6 years ago
|
||
Thanks kats! Notes are addressed now.
Attachment #8946794 -
Attachment is obsolete: true
Attachment #8946796 -
Flags: review?(bugmail)
Comment 5•6 years ago
|
||
Comment on attachment 8946796 [details] [diff] [review] Include WR revision Review of attachment 8946796 [details] [diff] [review]: ----------------------------------------------------------------- Thanks. I'll land this on inbound after the current inflight WR update merges to that branch.
Attachment #8946796 -
Flags: review?(bugmail) → review+
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/05f077d8f612 Move the WebRender revision to a separate file and include it in captures. r=kats
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/05f077d8f612
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in
before you can comment on or make changes to this bug.
Description
•