Closed
Bug 1043954
Opened 10 years ago
Closed 10 years ago
Intermittent: fatal error C1083: Cannot open include file: 'prenv.h': No such file or directory
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)
2.11 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
This is due to building in config/nspr during the compile tier retriggering an export (it's nspr's build system that does that, apparently), which re-exports headers, and creates a race with other compilations using the header in dist/include.
Assignee | ||
Comment 1•10 years ago
|
||
yeah, there's a libs:: export in nsprpub/config/rules.mk :(
Assignee | ||
Comment 2•10 years ago
|
||
Ah, I had forgotten about that feature of the nspr build system:
# libs - obsolete. Now a synonym of "export".
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8463673 -
Flags: review?(gps)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → mh+mozilla
Status: NEW → ASSIGNED
Comment 4•10 years ago
|
||
Comment on attachment 8463673 [details] [diff] [review]
Export nspr headers in a different directory during the target rule
Review of attachment 8463673 [details] [diff] [review]:
-----------------------------------------------------------------
So hacky. This new directory won't be packaged, will it?
Attachment #8463673 -
Flags: review?(gps) → review+
Assignee | ||
Comment 5•10 years ago
|
||
(In reply to Gregory Szorc [:gps] from comment #4)
> So hacky. This new directory won't be packaged, will it?
No.
Assignee | ||
Comment 6•10 years ago
|
||
Comment 8•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Updated•10 years ago
|
QA Whiteboard: [qa-]
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
•