Closed Bug 1244941 Opened 4 years ago Closed 4 years ago

Don't fill install manifest for artifact builds

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox47 fixed)

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

Since bug 1239217, artifacts builds are using a hybrid build system that uses the fastermake rules to copy files to dist/bin, which means artifact files are not removed by the processing of the dist/bin install manifest. This means we don't need to add them to the recursivemake install manifest anymore.
Comment on attachment 8714576 [details] [diff] [review]
Don't fill install manifest with artifacts

Review of attachment 8714576 [details] [diff] [review]:
-----------------------------------------------------------------

Oh, sweet!  Just gave this a quick look, I see nothing unusual.
Attachment #8714576 - Flags: review?(nalexander) → review+
https://hg.mozilla.org/mozilla-central/rev/e02003de4927
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.