Build more files in netwerk/cache2 in unified mode; r=glandium

RESOLVED FIXED in mozilla37

Status

Firefox Build System
General
RESOLVED FIXED
4 years ago
4 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla37

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

4 years ago
Created attachment 8536036 [details] [diff] [review]
Build more files in netwerk/cache2 in unified mode
Attachment #8536036 - Flags: review?(mh+mozilla)
(Assignee)

Updated

4 years ago
Assignee: nobody → ehsan.akhgari
(Assignee)

Updated

4 years ago
Blocks: 939583
Comment on attachment 8536036 [details] [diff] [review]
Build more files in netwerk/cache2 in unified mode

Review of attachment 8536036 [details] [diff] [review]:
-----------------------------------------------------------------

::: netwerk/cache2/moz.build
@@ -27,5 @@
> -]
> -
> -# AppCacheStorage.cpp cannot be built in unified mode because it uses plarena.h.
> -# The rest of these files cannot be built in unified mode because they rely oni
> -# including prlog.h after nsCache.h.

What bug made this untrue?
Attachment #8536036 - Flags: review?(mh+mozilla) → review+
(Assignee)

Comment 3

4 years ago
(In reply to Mike Hommey [:glandium] from comment #2)
> ::: netwerk/cache2/moz.build
> @@ -27,5 @@
> > -]
> > -
> > -# AppCacheStorage.cpp cannot be built in unified mode because it uses plarena.h.
> > -# The rest of these files cannot be built in unified mode because they rely oni
> > -# including prlog.h after nsCache.h.
> 
> What bug made this untrue?

Bug 806819.
https://hg.mozilla.org/mozilla-central/rev/af78342ac3e8
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37

Updated

4 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.