Establish a shared file for storing static build related data



Firefox Build System
2 years ago
2 months ago


(Reporter: gps, Assigned: gps)


Firefox Tracking Flags

(Not tracked)


MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Error loading review requests:


(1 attachment)



2 years ago
Patch forthcoming.

Comment 1

2 years ago
Created attachment 8723783 [details]
MozReview Request: Bug 1251411 - Support defining static build data; r?glandium

There are various static data structures that are shared between
multiple consumers. The build system, mach, etc. Currently, there is
a lot of duplication of this code because there isn't an obvious
location for this shared data.

This commit attempts to change that.

We introduce build/ as a standalone Python file that will hold
static data. We import this module in the ConfigEnvironment class and
expose the DATA module attribute (which is a dict) as .data in the
ConfigEnvironment. Consumers will be added in the near future.

We should arguably be putting this data in files or configure.
At this junction, I'm not ready to commit to either. I'm worried that
we'll end up stuffing lots of data in the root file. And
given the volatility of configure at the moment, I'd rather wait until is running before we consider that. We're talking about
static data, so porting it in the future shouldn't be too difficult.

Review commit:
See other reviews:
Attachment #8723783 - Flags: review?(mh+mozilla)
Comment on attachment 8723783 [details]
MozReview Request: Bug 1251411 - Support defining static build data; r?glandium

I cannot tell whether I want this landed or not without having a clear example of what you expect to end up in this file.
Attachment #8723783 - Flags: review?(mh+mozilla)

Comment 3

a year ago
I'm not actively working on this. And I don't see a major reason to keep the bug open.
Last Resolved: a year ago
Resolution: --- → INCOMPLETE


2 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.