Include xpfe/components/directory in the list of things that Thunderbird builds

RESOLVED FIXED in mozilla2.0b3

Status

()

Core
Build Config
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
mozilla2.0b3
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
Created attachment 453708 [details] [diff] [review]
The fix

In xpfe/components there is a small module called "directory", this currently provides some code that is useful when listing directories on an ftp server.

Whilst Thunderbird doesn't necessarily need this, I'm wanting to include it in the build for the following reasons:

- It gets rid of some nasty product-specific ifdefs in core code. Hence moving Thunderbird closer to core.
- The activity on content tabs in Thunderbird mean that someone may want it.
- It is a relatively small bit of code, that isn't going to hurt Thunderbird much to have it.

I'm also proposing that we remove the ifdefs for Sunbird and the standalone composer - neither of those are currently being maintained on trunk, and really they should move towards being closer to core anyway.

The only patch necessary is to the makefile, the build wizardry does the rest of including it in the right places (and as Thunderbird is a static build there's no components to add).
Attachment #453708 - Flags: superreview?(bienvenu)
Attachment #453708 - Flags: review?(ted.mielczarek)

Updated

8 years ago
Attachment #453708 - Flags: superreview?(bienvenu) → superreview+
Comment on attachment 453708 [details] [diff] [review]
The fix

Wanna fix the indentation in DIRS to be two-space while you're there?
Attachment #453708 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 2

8 years ago
Comment on attachment 453708 [details] [diff] [review]
The fix

Requesting approval for 2.0. This is a small build config change that doesn't affect Firefox, but reduces some ifdefs in xpfe and brings Thunderbird a little closer to core.
Attachment #453708 - Flags: approval2.0?

Updated

8 years ago
Attachment #453708 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 3

8 years ago
Checked in: http://hg.mozilla.org/mozilla-central/rev/7e4b21f34d2a
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
(Assignee)

Updated

8 years ago
Target Milestone: --- → mozilla2.0b3
You need to log in before you can comment on or make changes to this bug.