978778 - MOZ_PURPLE check needed in the about menu

Status: RESOLVED DUPLICATE of bug 954801

(Instantbird Graveyard :: Other, defect)

Description

[Patrick Cloke [:clokep]]

10:46:53 - flo-retina: would be interesting to check if libpurple is loaded correctly
10:47:05 - Mic: Ah!
10:47:25 - Mic: We show libpurple information on the About dialog, maybe that's why it crashes when trying to open it.
10:47:30 - aleth: iamjayakumars: Which protocols are you using? gtalk and fb only?
10:47:38 - flo-retina: Mic: do we?
10:47:45 - Mic: Yes, the libpurple version.
10:47:51 - flo-retina: oh yes
10:47:59 - flo-retina: doesn't that need #ifdef MOZ_PURPLE added?

Comment 1

[Florian Quèze [:florian]]

The proxy UI will need similar ifdefs.

Comment 2

[Patrick Cloke [:clokep]]

There's already checks for this:
 - http://mxr.mozilla.org/comm-central/source/im/content/aboutDialog.xul#53

Mic added this in bug 954801: http://hg.mozilla.org/comm-central/rev/d0b952d42fc4

Comment 3

[Florian Quèze [:florian]]

Hmm, I didn't remember we made these changes, but I was thinking that now that we have a build time define that's easy to use, we may want to use it instead of a runtime check. Especially I thought we may want to avoid packaging the proxy UI files that are irrelevant without libpurple. But I guess they aren't really heavy, so it probably doesn't really matter.

Comment 4

[Patrick Cloke [:clokep]]

I kind of dislike making them build time dependent, it would mean you can't get an IB without libpurple (somehow) and then add libpurple (once it's built as an extension). Maybe just a pipe dream, but it's where my mind went.

I think the "right" solution would be to actually make these overlays that are part of the extension. Thoughts?

Comment 5

[Florian Quèze [:florian]]

(In reply to Patrick Cloke [:clokep] from comment #4)

> I think the "right" solution would be to actually make these overlays that
> are part of the extension. Thoughts?

That would be better.

The real right solution would be to finally fix the proxy handling, so that libpurple prpls use the proxies defined by Mozilla preferences.