469392 - MOZILLA_1_9_1_BRANCH define doesn't work in C++ code

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

As I almost had to fix a mozilla-central + comm-central bustage today, I realised the MOZILLA_1_9_1_BRANCH define doesn't work in c++ code because its not included in mozilla-config.h

The fix is a simple change in configure.in.

Comment 1

[Robert Kaiser]

Comment on attachment 352786 [details] [diff] [review]
The fix

I even thought about this case when I did the other review but guessed you might want to defer to doing explicit DEFINES in Makefiles, which I guess would probably also work. This is surely the less intrusive way where we can't avoid using this flag, though.

Comment 2

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/c5884855549c