754432 - Let Thunderbird have access to ContactService.jsm and ContactDB.jsm.

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

Attachment: Patch v1

ContactService.jsm and ContactDB.jsm are the fallback storage mechanism for the Contacts API, and are currently only available in b2g builds.

The Thunderbird team has started to make moves to use the Contacts API for storage / retrieval of contacts for a new address book.  One of those moves is to start working on desktop support for the Contacts API.  As part of this effort, we'll want access to the fallback storage mechanism.

The attached patch alters the Makefile for dom/contacts to package ContactService.jsm and ContactDB.jsm for both b2g and mail.

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 623278 [details] [diff] [review]
Patch v1

r?'ing fabrice because he originally r+'d the patch in bug 731855.

Comment 2

[Mike Conley (:mconley) (:⚙️)]

So that filter is my way of doing logical OR for Makefiles... let me know if there's another preferred method.

Comment 3

[[:fabrice] Fabrice Desré]

Comment on attachment 623278 [details] [diff] [review]
Patch v1

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

I don't know of any other method either.

Comment 4

[Mike Conley (:mconley) (:⚙️)]

Thanks for the fast review, Fabrice.

mozilla-inbound:  https://hg.mozilla.org/integration/mozilla-inbound/rev/de6d19899b89

Comment 5

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/de6d19899b89

Comment 6

[Philip Chee]

Does this also include SeaMonkey? Or do we need an extra patch in /suite/?

Comment 7

[Mike Conley (:mconley) (:⚙️)]

(In reply to Philip Chee from comment #6)
> Does this also include SeaMonkey? Or do we need an extra patch in /suite/?

This patch only took care of Thunderbird, I'm afraid. If suite needs access to ContactService.jsm and ContactDB.jsm, an extra patch will be required here.