674942 - Mailnews part of cookie code still gets built as MOZ_MAIL_NEWS is not configured correctly

Status: RESOLVED WORKSFORME

(Firefox Build System :: General, defect)

Description

[Ian Neal]

Attachment: Fix configure.in

Currently MOZ_MAIL_NEWS ends up defined as @MOZ_MAIL_NEWS@ which equates to true so, even though mailnews code is ifdef'd out in the cookie code, it still gets built.
This patch, hopefully, fixes that by:
* Adding two lines to configure.in to make sure that MOZ_MAIL_NEWS is not defined.

Comment 1

[Mark Banner (:standard8)]

In bug 557047, I'm proposing on changing the way it works and removing the MOZ_MAIL_NEWS option completely.

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 549140 [details] [diff] [review]
Fix configure.in

I was going to say this should get review from a cookies peer, but the changes on the cookies side are trivial enough that I don't think that's necessary.

r=me

Comment 3

[Mark Banner (:standard8)]

Bug 557047 removed the need for this by reworking the code so that MOZ_MAIL_NEWS is no longer required and the mailnews options are flags set by protocols in mailnews.