Closed
Bug 1216901
Opened 9 years ago
Closed 9 years ago
mach build faster doesn't refresh when jar.mn files are being changed
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox45 fixed)
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee: nobody → mh+mozilla
Attachment #8679788 -
Flags: review?(mshal)
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8679789 -
Flags: review?(mshal)
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8679789 -
Attachment is obsolete: true
Attachment #8679789 -
Flags: review?(mshal)
Attachment #8679827 -
Flags: review?(mshal)
Updated•9 years ago
|
Attachment #8679788 -
Flags: review?(mshal) → review+
Comment 4•9 years ago
|
||
Comment on attachment 8679827 [details] [diff] [review]
Make the FasterMake backend reticulate splines when moz.build or jar.mn files are modified
>+ # Add backend dependencies:
>+ mk.create_rule([self._backend_output_list_file]).add_dependencies(
>+ self.backend_input_files)
What is this used for? As far as I can tell, it seems to behave the same without it. Namely, the backend.RecursiveMakeBackend rule triggers when backend_input_files changes, which regenerates the faster-make backend as well. Looks good otherwise, if I'm understanding it correctly.
Attachment #8679827 -
Flags: review?(mshal) → review+
Assignee | ||
Comment 5•9 years ago
|
||
self.backend_iunput_files contains more files in the fastermake backend, so there are effectively more files that can trigger splines reticulations for that backend compared to the recursive make backend. And those are not actual dependencies for the recursive make backend either.
Comment 7•9 years ago
|
||
backout bugherder landing |
Backed out to hopefully fix the Win PGO bustage so I can merge inbound.
https://hg.mozilla.org/integration/mozilla-inbound/rev/9bd43c8ec7be
https://hg.mozilla.org/integration/mozilla-inbound/rev/d0da0df3ed52
Comment 9•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ec7ac102bcb9
https://hg.mozilla.org/mozilla-central/rev/28116363f21a
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Comment 10•9 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/ec7ac102bcb9
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/28116363f21a
status-b2g-v2.5:
--- → fixed
Comment 11•9 years ago
|
||
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
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
•