The default bug view has changed. See this FAQ.

Remove --disable-storage

RESOLVED FIXED in mozilla8

Status

()

Core
Build Config
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Matheus Kerschbaum, Assigned: Matheus Kerschbaum)

Tracking

({dev-doc-complete})

Trunk
mozilla8
dev-doc-complete
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(1 attachment, 2 obsolete attachments)

27.25 KB, patch
Matheus Kerschbaum
: review+
Details | Diff | Splinter Review
Comment hidden (empty)
(Assignee)

Updated

6 years ago
Depends on: 669040
(Assignee)

Comment 1

6 years ago
Created attachment 549656 [details] [diff] [review]
patch
Attachment #549656 - Flags: review?(khuey)
(Assignee)

Updated

6 years ago
Blocks: 675501
Assignee: nobody → matjk7
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment on attachment 549656 [details] [diff] [review]
patch

>diff --git a/netwerk/build.mk b/netwerk/build.mk

Don't bother touching this, netwerk/build.mk is dead code.

>diff --git a/toolkit/toolkit-tiers.mk b/toolkit/toolkit-tiers.mk
>--- a/toolkit/toolkit-tiers.mk
>+++ b/toolkit/toolkit-tiers.mk
>@@ -47,23 +47,22 @@ TIERS += platform
> ifdef NS_TRACE_MALLOC
> tier_platform_dirs = tools/trace-malloc/lib
> endif
> 
> ifdef MOZ_TREE_FREETYPE
> tier_platform_staticdirs += modules/freetype2
> endif
> 
>-tier_platform_dirs += xpcom
>-
> ifndef MOZ_NATIVE_ZLIB
> tier_platform_dirs += modules/zlib
> endif
> 
> tier_platform_dirs += \
>+		xpcom \
> 		modules/libreg \
> 		modules/libpref \
> 		intl \
> 		netwerk \
> 		$(NULL)
> 
> ifdef MOZ_AUTH_EXTENSION
> tier_platform_dirs += extensions/auth

Drop this hunk, it's irrelevant and scary.

r=me
Attachment #549656 - Flags: review?(khuey) → review+
(Assignee)

Comment 3

6 years ago
Created attachment 549995 [details] [diff] [review]
patch

Review comments addressed.
Attachment #549656 - Attachment is obsolete: true
Attachment #549995 - Flags: review+
(Assignee)

Comment 4

6 years ago
Created attachment 551228 [details] [diff] [review]
patch

Updated to tip.
Attachment #549995 - Attachment is obsolete: true
Attachment #551228 - Flags: review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
Keywords: checkin-needed
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/9754cac0194c
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
(Assignee)

Updated

6 years ago
Keywords: dev-doc-needed
(Assignee)

Updated

6 years ago
Blocks: 678794
Docs updated:

https://developer.mozilla.org/en/Configuring_Build_Options#Other_Not.c2.a0So.c2.a0Commonly_Used_Options

Listed on Firefox 8 for developers.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.