Closed Bug 1371093 Opened 2 years ago Closed 2 years ago

Remove redundant tooltool manifest definitions from mozharness

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set

Tracking

(firefox55 fixed)

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(2 files)

No description provided.
Attached file Validation script
Quite similar to what I did in bug 1356952.

Note that after this bug, there still remains a bunch of "tooltool_manifest_src"s in other mozharness configs, which indicate there's more work on this front for followups.
Comment on attachment 8875522 [details]
Bug 1371093 - Remove now redundant tooltool manifest definitions from mozharness.

https://reviewboard.mozilla.org/r/146948/#review151452

Looks good, though I didn't verify that these aren't still used somehow by buildbot. I assume automation will tell us that fairly quickly :)
Attachment #8875522 - Flags: review?(mshal) → review+
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/0c8ab344a249
Remove now redundant tooltool manifest definitions from mozharness. r=mshal
https://hg.mozilla.org/mozilla-central/rev/0c8ab344a249
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.