Closed Bug 675500 Opened 13 years ago Closed 13 years ago

Remove --disable-storage

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla8

People

(Reporter: matjk7, Assigned: matjk7)

References

Details

(Keywords: dev-doc-complete, Whiteboard: [inbound])

Attachments

(1 file, 2 obsolete files)

      No description provided.
Depends on: 669040
Attached patch patch (obsolete) — Splinter Review
Attachment #549656 - Flags: review?(khuey)
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+
Attached patch patch (obsolete) — Splinter Review
Review comments addressed.
Attachment #549656 - Attachment is obsolete: true
Attachment #549995 - Flags: review+
Attached patch patchSplinter Review
Updated to tip.
Attachment #549995 - Attachment is obsolete: true
Attachment #551228 - Flags: review+
Keywords: checkin-needed
Keywords: checkin-needed
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/9754cac0194c
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Keywords: dev-doc-needed
Blocks: 678794
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: