Closed Bug 1494139 Opened 6 years ago Closed 6 years ago

Fixes to the tup backend for rust integration with spidermonkey

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox64 fixed)

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: chmanchester, Assigned: chmanchester)

References

Details

Attachments

(3 files)

I realized when debugging something for bug 1490948 that there might be multiple versions of certain crates that all want to write a build script output file, so these can't all be written to the same directory because the package name might not be enough to disambiguate on its own. We can write them to each package's out dir instead.
I have a few fixes for bug 1490948 I'd like to land here.
Assignee: nobody → cmanchester
See Also: → 1490948
Summary: Write build script output files to each package's out dir → Fixes to the tup backend for rust integration with spidermonkey
Comment on attachment 9012713 [details]
Bug 1494139 - Place build script outputs in each package's output directory in the tup backend.

Michael Shal [:mshal] has approved the revision.
Attachment #9012713 - Flags: review+
Comment on attachment 9012715 [details]
Bug 1494139 - Link rust libraries into programs in the tup backend.

Michael Shal [:mshal] has approved the revision.
Attachment #9012715 - Flags: review+
Comment on attachment 9012719 [details]
Bug 1494139 - Add rust build script outputs to the tup backend for upcoming cranelift crates.

Michael Shal [:mshal] has approved the revision.
Attachment #9012719 - Flags: review+
Pushed by cmanchester@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fd114fd7cf68
Place build script outputs in each package's output directory in the tup backend. r=mshal
https://hg.mozilla.org/integration/autoland/rev/23ebf9cc2c2d
Add rust build script outputs to the tup backend for upcoming cranelift crates. r=mshal
https://hg.mozilla.org/integration/autoland/rev/e3e005ca7ffa
Link rust libraries into programs in the tup backend. r=mshal
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: