694464 - c-c configure.in: port bug 464088 and bug 690369. (NSS_DISABLE_DBM)

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Serge Gautherie (:sgautherie)]

Noticed while working on bug 690369.

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) configure.in: add --disable-dbm (NSS_DISABLE_DBM) option [Checked in: Comment 5]

Copy bug 464088 and
"
http://hg.mozilla.org/mozilla-central/rev/f0b46212962e
Fix indentation in configure --help. Whitespace changes only.
"

NB: c-c ignores "WebM".

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 566996 [details] [diff] [review]
(Av1) configure.in: add --disable-dbm (NSS_DISABLE_DBM) option
[Checked in: Comment 5]

I don't care for this, it doesn't seem high value for us, we should spend the time on bug 648979.

Comment 3

[Robert Kaiser]

Comment on attachment 566996 [details] [diff] [review]
(Av1) configure.in: add --disable-dbm (NSS_DISABLE_DBM) option
[Checked in: Comment 5]

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

Man, I so much want to get rid of even needing to care about this, it's so useless. We're not using this in c-c anywhere and never will, it's just clutter.

The patch is correct for adding the --disable-dbm flag, but I'm not sure if we even should bother. Callek, what's your opinion?

Comment 4

[Justin Wood (:Callek)]

Comment on attachment 566996 [details] [diff] [review]
(Av1) configure.in: add --disable-dbm (NSS_DISABLE_DBM) option
[Checked in: Comment 5]

Wouldn't have bothered with this, but r+=me

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 566996 [details] [diff] [review]
(Av1) configure.in: add --disable-dbm (NSS_DISABLE_DBM) option
[Checked in: Comment 5]

http://hg.mozilla.org/comm-central/rev/4d583d242668