Closed
Bug 1301473
Opened 7 years ago
Closed 7 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•7 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•7 years ago
|
||
Yes, that is exactly what I want!
Comment 4•7 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•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b2166c12cbbe
Status: NEW → RESOLVED
Closed: 7 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
•