Last Comment Bug 388743 - build dbus on linux by default (--enable-dbus) to auto-detect online/offline state
: build dbus on linux by default (--enable-dbus) to auto-detect online/offline ...
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: x86 Linux
: -- normal (vote)
: mozilla1.9beta2
Assigned To: Steffen Wilberg
:
: Gregory Szorc [:gps]
Mentors:
Depends on: 312793 377992 454391
Blocks: 216490 404738
  Show dependency treegraph
 
Reported: 2007-07-19 01:43 PDT by Steffen Wilberg
Modified: 2011-10-26 01:55 PDT (History)
13 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (1.40 KB, patch)
2007-11-18 05:03 PST, Steffen Wilberg
ted: review+
Details | Diff | Splinter Review
patch v2 (1.44 KB, patch)
2007-11-18 13:31 PST, Steffen Wilberg
ted: review+
mtschrep: approval1.9+
Details | Diff | Splinter Review

Description Steffen Wilberg 2007-07-19 01:43:06 PDT
Bug 312793 introduced support for dbus online/offline events. Firefox and Thunderbird automatically toggle their state when they receive such an event.
This works fine on windows trunk, but on linux, you need to specify --enable-dbus in the mozconfig due to bug 377992.

Since the new linux reference platform has been introduced, we should build dbus by default and only disable it on the old tinderboxen specifically which don't meet its requirements.

This should be done by changing the --enable-dbus switch in configure.in.
Another option would be to add --enable-dbus to the mozconfig of the linux tinderboxen:
http://mxr.mozilla.org/seamonkey/source/tools/tinderbox-configs/firefox/linux/mozconfig
http://mxr.mozilla.org/seamonkey/source/tools/tinderbox-configs/thunderbird/linux/mozconfig
Comment 1 Ted Mielczarek [:ted.mielczarek] 2007-07-19 05:36:20 PDT
As I said in bug 377992 comment 8, we want builds to match our official nightlies unless features are explicitly disabled.  This means that to do this right, the --enable-dbus switch should become a --disable-dbus switch, and the check for dbus should error if it is not found.

I'm not sure if all the trunk Linux tinderboxen have been updated though, or just Firefox.  preed, can you comment on this?
Comment 2 J. Paul Reed [:preed] 2007-07-27 13:25:30 PDT
I know they have not... but we could turn this on for just Firefox right now.

That might be useful to get some testing, and then as we upgrade the other images, they'll know it'll "just work" (or... y'know... *should* "just work" ;-)
Comment 3 Steffen Wilberg 2007-10-25 14:33:37 PDT
All of the tinderboxen have been migrated, including Thunderbird and Seamonkey, haven't they? So we could enable dbus now for every app, I guess.
Comment 4 Steffen Wilberg 2007-11-18 05:03:55 PST
Created attachment 289210 [details] [diff] [review]
patch
Comment 5 Ted Mielczarek [:ted.mielczarek] 2007-11-18 06:05:13 PST
Comment on attachment 289210 [details] [diff] [review]
patch

r=me with a slight change.  Change MOZ_ENABLE_DBUS=force to be MOZ_ENABLE_DBUS=1 and remove the whole if block around the AC_MSG_ERROR below, so that we always error out if MOZ_ENABLE_DBUS is set but the library doesn't exist.
Comment 6 Steffen Wilberg 2007-11-18 13:31:18 PST
Created attachment 289248 [details] [diff] [review]
patch v2
Comment 7 Ted Mielczarek [:ted.mielczarek] 2007-11-18 13:45:35 PST
Comment on attachment 289248 [details] [diff] [review]
patch v2

We'll have to make sure that when this gets checked in, any tinderboxes that don't have the required version of dbus get --disable-dbus in their mozconfigs.
Comment 8 Reed Loden [:reed] (use needinfo?) 2007-11-21 00:27:07 PST
Checking in configure.in;
/cvsroot/mozilla/configure.in,v  <--  configure.in
new revision: 1.1890; previous revision: 1.1889
done

I'll fix any mozconfigs for tinderboxen that break for those that are in CVS.

Note You need to log in before you can comment on or make changes to this bug.