Closed
Bug 377843
Opened 17 years ago
Closed 17 years ago
dbus component would never build
Categories
(Toolkit Graveyard :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wolfiR, Assigned: wolfiR)
References
Details
Attachments
(2 files)
711 bytes,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
807 bytes,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
The dbus component toolkit/system/dbus will never be built because in toolkit-tiers.mk: ifdef MOZ_ENABLE_DBUS tier_toolkit_dirs += toolkit/system/dbus endif but MOZ_ENABLE_DBUS is not in autoconf.mk but only a DEFINE which doesn't apply to a Makefile.
Assignee | ||
Comment 1•17 years ago
|
||
Assignee: nobody → mozilla
Status: NEW → ASSIGNED
Assignee | ||
Updated•17 years ago
|
Attachment #261922 -
Flags: first-review?(benjamin)
Assignee | ||
Comment 2•17 years ago
|
||
short warning: the component doesn't build correctly as it seems
Assignee | ||
Comment 3•17 years ago
|
||
This fixes the build errors and it's just build config so adding it just here might make sense.
Attachment #261926 -
Flags: first-review?
Assignee | ||
Updated•17 years ago
|
Attachment #261926 -
Flags: first-review? → first-review?(benjamin)
Attachment #261922 -
Flags: first-review?(benjamin) → review?(benjamin)
Attachment #261926 -
Flags: first-review?(benjamin) → review?(benjamin)
Updated•17 years ago
|
Attachment #261926 -
Flags: review?(benjamin) → review+
Updated•17 years ago
|
Attachment #261922 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 4•17 years ago
|
||
Fixed on trunk. Checking in config/autoconf.mk.in; /cvsroot/mozilla/config/autoconf.mk.in,v <-- autoconf.mk.in new revision: 3.422; previous revision: 3.421 done Checking in toolkit/system/dbus/Makefile.in; /cvsroot/mozilla/toolkit/system/dbus/Makefile.in,v <-- Makefile.in new revision: 1.3; previous revision: 1.2 done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 5•17 years ago
|
||
This patch causes the Places linux trees to burn http://tinderbox.mozilla.org/showbuilds.cgi?tree=MozillaTest
Assignee | ||
Comment 6•17 years ago
|
||
The patch enables the component because it wasn't enabled by accident. So the initial issue is the dbus implementation. I'll open another bug for that -> bug 377992
Depends on: 377992
Depends on: 378303
Blocks: 312793
Updated•17 years ago
|
Flags: in-testsuite-
Updated•6 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•