Closed
Bug 758782
Opened 12 years ago
Closed 12 years ago
makefiles: declare mkdir_deps targets .PRECIOUS
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla15
People
(Reporter: joey, Assigned: joey)
References
Details
Attachments
(1 file, 1 obsolete file)
2.37 KB,
patch
|
Details | Diff | Splinter Review |
mkdir_deps generated dependencies will be considered an intermediate file and removed when used as a pre-requisite for a wildcard target. Add ".PRECIOUS: %/.mkdir.done" within config/makefiles/autotargets.mk to inhibit the behavior. Problem surfaced while compiling idl files { toolkit/components/alerts }. libalerts_s.a may also suffer from a similar problem: nsAlertsService.cpp /usr/bin/ccache c++ -o nsAlertsService.o [...] rm -f libalerts_s.a
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → joey
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 627370 [details] [diff] [review] mkdir_deps dependencies should be .PRECIOUS Flag directory creation deps as .PRECIOUS to inhibit removal when they are considered an "intermediate file".
Attachment #627370 -
Flags: review?(ted.mielczarek)
Comment 4•12 years ago
|
||
Comment on attachment 627370 [details] [diff] [review] mkdir_deps dependencies should be .PRECIOUS Review of attachment 627370 [details] [diff] [review]: ----------------------------------------------------------------- ::: config/makefiles/autotargets.mk @@ +18,5 @@ > MKDIR ?= mkdir -p > TOUCH ?= touch > > +# Deps will be considered intermediate when used as a pre-requisite for > +# wildcard targets. Inhibit their removal, mkdir -p is a standlaone op. typo in standalone
Attachment #627370 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Attachment #627370 -
Attachment is obsolete: true
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 628388 [details] [diff] [review] mkdir_deps dependencies should be .PRECIOUS r=glandium carried forward. Fixed typo in 'standalone' spelling.
Attachment #628388 -
Flags: review?(ted.mielczarek)
Assignee | ||
Comment 7•12 years ago
|
||
Try job: https://tbpl.mozilla.org/?tree=Try&rev=f47e7fde1a62
Assignee | ||
Updated•12 years ago
|
Attachment #628388 -
Flags: review?(ted.mielczarek)
Assignee | ||
Comment 8•12 years ago
|
||
Inbound push: https://hg.mozilla.org/integration/mozilla-inbound/rev/2e755d7847ac
Comment 9•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2e755d7847ac
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
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
•