Closed
Bug 1498031
Opened 6 years ago
Closed 6 years ago
Merge code paths for deciding when to run configure between tup and make
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(firefox64 fixed)
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: chmanchester, Assigned: chmanchester)
References
Details
Attachments
(1 file)
These should already be equivalent because they get their dependencies from the same place. Let's clean up the calling code so we don't have to rely on client.mk for this anymore.
Assignee | ||
Comment 1•6 years ago
|
||
While writing this patch it stops making sense for the make build system to know about config.status at all, including various checks we have that will cause configure to run if config.status is out of date. This could be an issue for people relying on "make" as an entry point to the build system, but I'm going to assume in this patch that while we probably want the entry points "make" and "configure", reducing the lengths a bare "make" invocation will go to ensure configure is up to date is ok.
Assignee | ||
Comment 2•6 years ago
|
||
This addresses a related issue along the way: a build that results in running configure would not update the value of self.config_environment (and therefore self.substs) as seen from the build driver, so out of date values would have been used. The changes to Makefile.in and client.mk made exploit the assumption that by he time anything in the Make build is running, config.status is up to date. Users running "make" without the benefit of "mach" will need to manually run configure when necessary in order to take this into account.
Pushed by cmanchester@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6dad1580bc3e Merge code paths for running configure between Tup and Make based backends. r=firefox-build-system-reviewers,mshal
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6dad1580bc3e
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•