Closed
Bug 1301473
Opened 9 years ago
Closed 9 years ago
Backporting manifest updates is a huge pain
Categories
(Testing :: web-platform-tests, defect)
Testing
web-platform-tests
Tracking
(firefox51 fixed)
RESOLVED
FIXED
mozilla51
| Tracking | Status | |
|---|---|---|
| firefox51 | --- | fixed |
People
(Reporter: bzbarsky, Unassigned)
Details
Attachments
(1 file)
STEPS TO REPRODUCE:
1) Write a patch that adds web platform tests
2) Try to backport it.
3) Get merge failures in the manifest, not surprising.
4) Try to use "mach web-platform-tests-create" or "mach web-platform-tests --manifest-update" to update the manifest.
EXPECTED RESULTS: Manifest gets updated.
ACTUAL RESULTS:
Exception: Binary expected at /Users/bzbarsky/mozilla/aurora/obj-firefox/dist/NightlyDebug.app/Contents/MacOS/firefox does not exist.
forcing me to build stuff I should have no reason to build.
I don't see why we need to have a Firefox binary just to update the manifest...
Flags: needinfo?(james)
| Comment hidden (mozreview-request) |
Comment 2•9 years ago
|
||
Comment on attachment 8790289 [details]
Bug 1301473 - Add a wpt-manifest-update command,
bz: Does this seem like what you want?
Flags: needinfo?(james)
| Reporter | ||
Comment 3•9 years ago
|
||
Yes, that is exactly what I want!
Comment 4•9 years ago
|
||
| mozreview-review | ||
Comment on attachment 8790289 [details]
Bug 1301473 - Add a wpt-manifest-update command,
https://reviewboard.mozilla.org/r/78184/#review76672
Assuming it works...
Attachment #8790289 -
Flags: review?(Ms2ger) → review+
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/autoland/rev/b2166c12cbbe
Add a wpt-manifest-update command, r=Ms2ger
Comment 6•9 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in
before you can comment on or make changes to this bug.
Description
•