Closed
Bug 1043820
Opened 10 years ago
Closed 10 years ago
Remove PARALLEL_DIRS
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla34
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(1 file)
63.06 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
Quoting my own comment from bug 1043802 comment 7: Which makes the distinction between PARALLEL_DIRS and DIRS useless since the directory traversal filters are now either putting everything in the parallel bucket, or in the sequential bucket. Will file a new bug for that.
Assignee | ||
Comment 1•10 years ago
|
||
There's a semantic difference that might matter for libs, though, that DIRS are recursed after the current directory, while PARALLEL_DIRS are recursed before.
Assignee | ||
Comment 2•10 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #1) > There's a semantic difference that might matter for libs, though, that DIRS > are recursed after the current directory, while PARALLEL_DIRS are recursed > before. but it doesn't matter anyways, since they are all already treated as DIRS anyways.
Comment 3•10 years ago
|
||
Hooray!
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8462500 -
Flags: review?(gps)
Updated•10 years ago
|
Attachment #8462500 -
Flags: review?(gps) → review+
Assignee | ||
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/22850cacf0f9
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/22850cacf0f9
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Updated•10 years ago
|
QA Whiteboard: [qa-]
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
•