Closed
Bug 680534
Opened 13 years ago
Closed 13 years ago
Clean up IDL Directory Rules in rules.mk
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla9
People
(Reporter: gps, Assigned: gps)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 2 obsolete files)
2.89 KB,
patch
|
Details | Diff | Splinter Review |
Currently, the $(IDL_DIR) target in rules.mk has the following issues: 1) It is defined as a double colon rule, so it gets executed every single time 2) It is defined twice The attached patch converts it to a regular rule and removes an inner definition. I believe the remaining definition is always in an outer scope and is thus always defined to the bits that need it. Although, the conditional statements in this part of rules.mk are nested pretty deep, so I could be wrong. I also believe everything writing into $(IDL_DIR) has $(IDL_DIR) as a prerequisite, so removing the double colon should be safe. I've done the following to verify the change: * Built w/ make from empty objdir * Rebuilt a subcomponent w/ make that used to always nsinstall $(IDL_DIR) and verified it didn't do this any more I'm not sure how much this improves build times by, but it should be greater than 0.
Assignee | ||
Updated•13 years ago
|
Attachment #554512 -
Flags: review?(khuey)
Assignee | ||
Comment 1•13 years ago
|
||
Added missing identical change to JS's rules.mk and added bug number to summary.
Assignee: nobody → gps
Attachment #554512 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #554512 -
Flags: review?(khuey)
Attachment #554520 -
Flags: review?(khuey)
Updated•13 years ago
|
OS: Windows 2000 → All
Version: unspecified → Trunk
Comment on attachment 554520 [details] [diff] [review] Remove double colon and redundancy of $(IDL_DIR), v2 Can we drop the comment above the $(IDL_DIR) rule we're keeping. I don't think that's been accurate in the better part of a decade ...
Attachment #554520 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 3•13 years ago
|
||
This is the same patch that was r+ by khuey except the outdated comment has been removed, per comment from khuey.
Attachment #554520 -
Attachment is obsolete: true
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 5•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/1ee70d6bb2c8
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•