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)

defect
Not set
normal

Tracking

(firefox45 fixed)

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(2 files, 1 obsolete file)

No description provided.
Assignee: nobody → mh+mozilla
Attachment #8679788 - Flags: review?(mshal)
Attachment #8679789 - Attachment is obsolete: true
Attachment #8679789 - Flags: review?(mshal)
Attachment #8679827 - Flags: review?(mshal)
Attachment #8679788 - Flags: review?(mshal) → review+
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+
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.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: