Closed
Bug 1220000
Opened 10 years ago
Closed 10 years ago
GENERATED_FILES dep files should be added to EXTRA_EXPORT_MDDEPEND_DIRS
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox45 fixed, b2g-v2.5 fixed)
RESOLVED
FIXED
mozilla45
People
(Reporter: chmanchester, Unassigned)
Details
Attachments
(1 file)
They aren't included right now. I have:
$ touch browser/themes/preprocess-tab-svgs.py
~/m-c
$ REBUILD_CHECK=1 ./mach build browser/themes/
0:00.10 /usr/bin/make -C /home/chris/m-c/obj-dbg -j8 -s backend.RecursiveMakeBackend
0:00.22 /usr/bin/make -C browser/themes -j8 -s
0:00.25 Rebuilding tab-selected-end.svg because /home/chris/m-c/browser/themes/preprocess-tab-svgs.py changed
0:00.26 Rebuilding tab-selected-start.svg because /home/chris/m-c/browser/themes/preprocess-tab-svgs.py changed
0:00.64 /usr/bin/make -C browser/app -j8 -s
0:00.98 Your build was successful!
but
$ touch browser/themes/linux/linuxShared.inc
~/m-c
$ REBUILD_CHECK=1 ./mach build browser/themes/
0:00.09 /usr/bin/make -C /home/chris/m-c/obj-dbg -j8 -s backend.RecursiveMakeBackend
0:00.22 /usr/bin/make -C browser/themes -j8 -s
0:00.59 /usr/bin/make -C browser/app -j8 -s
0:00.98 Your build was successful!
Or maybe we should get rid of EXTRA_EXPORT_MDDEPEND_DIRS and do this a simpler way, because I don't see it used anywhere.
Reporter | ||
Comment 1•10 years ago
|
||
Bug 1220000 - Use EXTRA_EXPORT_MDDEPEND_DIRS so GENERATED_FILES dep files are included properly. r=glandium
Attachment #8680979 -
Flags: review?(mh+mozilla)
Reporter | ||
Comment 2•10 years ago
|
||
Comment on attachment 8680979 [details]
MozReview Request: Bug 1220000 - Unconditionally include EXTRA_MDDEPEND_FILES so callers get what they expect. r=glandium
Bug 1220000 - Use EXTRA_EXPORT_MDDEPEND_DIRS so GENERATED_FILES dep files are included properly. r=glandium
Comment 3•10 years ago
|
||
Comment on attachment 8680979 [details]
MozReview Request: Bug 1220000 - Unconditionally include EXTRA_MDDEPEND_FILES so callers get what they expect. r=glandium
https://reviewboard.mozilla.org/r/23713/#review21429
Looking around, it turns out:
- Nothing uses EXTRA_EXPORT_MDDEPEND_FILES anymore (it was added and used for dom bindings, until bug 928195)
- EXTRA_MDDEPEND_FILES is only used by GeneratedFiles and manually used by accessible/xpcom/Makefile.in
- The use in accessible/xpcom/Makefile.in is actually wrong, since EXTRA_MDDEPEND_FILES are currently *not* included for the export tier.
- The uses for GeneratedFiles are wrong for the same reason, and this is what you're trying to fix.
I'd say we should just kill EXTRA_EXPORT_MDDEPEND_FILES and include EXTRA_MDDEPEND_FILES unconditionally.
Attachment #8680979 -
Flags: review?(mh+mozilla)
Reporter | ||
Comment 4•10 years ago
|
||
Comment on attachment 8680979 [details]
MozReview Request: Bug 1220000 - Unconditionally include EXTRA_MDDEPEND_FILES so callers get what they expect. r=glandium
Bug 1220000 - Unconditionally include EXTRA_MDDEPEND_FILES so callers get what they expect. r=glandium
GENERATED_FILES and accessible/xpcom/Makefile.in add to EXTRA_MDDEPEND_FILES, but for
targets that run during export. Export doesn't include EXTRA_MDDEPEND_FILES, so none
of them is ending up with correct dependencies. The EXTRA_EXPORT_MDDEPEND_FILES variable
could be used for this purpose, but given the circumstances this variable is removed,
and EXTRA_MDDEPEND_FILES is included unconditionally to this mechanism.
Attachment #8680979 -
Attachment description: MozReview Request: Bug 1220000 - Use EXTRA_EXPORT_MDDEPEND_DIRS so GENERATED_FILES dep files are included properly. r=glandium → MozReview Request: Bug 1220000 - Unconditionally include EXTRA_MDDEPEND_FILES so callers get what they expect. r=glandium
Attachment #8680979 -
Flags: review?(mh+mozilla)
Comment 5•10 years ago
|
||
Comment on attachment 8680979 [details]
MozReview Request: Bug 1220000 - Unconditionally include EXTRA_MDDEPEND_FILES so callers get what they expect. r=glandium
https://reviewboard.mozilla.org/r/23713/#review21455
Attachment #8680979 -
Flags: review?(mh+mozilla) → review+
Reporter | ||
Comment 6•10 years ago
|
||
Comment 7•10 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Comment 8•10 years ago
|
||
bugherder uplift |
status-b2g-v2.5:
--- → fixed
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•