Last Comment Bug 526451 - allow to build gconf and gnomevfs/gio support independently
: allow to build gconf and gnomevfs/gio support independently
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: All Linux
-- normal (vote)
: mozilla1.9.3a1
Assigned To: Wolfgang Rosenauer [:wolfiR]
: Gregory Szorc [:gps] (away until 2017-03-20)
Depends on: 536745
Blocks: 458211 545343
  Show dependency treegraph
Reported: 2009-11-04 03:25 PST by Wolfgang Rosenauer [:wolfiR]
Modified: 2010-02-09 23:30 PST (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (8.61 KB, patch)
2009-11-04 03:30 PST, Wolfgang Rosenauer [:wolfiR]
ted: review+
Details | Diff | Splinter Review

Description User image Wolfgang Rosenauer [:wolfiR] 2009-11-04 03:25:44 PST
Currently gconf support is only compiled in mozgnome component if gnome-vfs is also available.
Because gconf could be used outside of Gnome (for example hildon) it should be possible to build the component without gnome-vfs and also in an optional way outside of the Gtk backend.
Comment 1 User image Wolfgang Rosenauer [:wolfiR] 2009-11-04 03:30:21 PST
Created attachment 410203 [details] [diff] [review]

This patch seperates the gnome-vfs/gio stuff from gconf. So if any of the possible mozgnome components is selected to compile it creates mozgnome.
gconf and gnome-vfs are still built by default in the gtk platform while gconf can be selected by --enable-gconf independent from the used toolkit now.
I also rearranged the sections in a bit to make more sense by grouping the mozgnome components.
Comment 2 User image Ted Mielczarek [:ted.mielczarek] 2009-12-21 12:25:40 PST
Comment on attachment 410203 [details] [diff] [review]

>diff --git a/ b/
>--- a/
>+++ b/
>+    dnl ========================================================
>+    dnl = GConf support module
>+    dnl ========================================================
>+    # if not set already in GTK2 environment we want to have it
>+    # available as an option nevertheless
>+    MOZ_ARG_ENABLE_BOOL(gconf,
>+    [  --enable-gconf          Enable gconf support (default for Gtk2 builds)],
>+       MOZ_ENABLE_GCONF=force,
>+    )

Do you expect that anyone would really want to --enable-gconf in a build where it doesn't already happen automatically? If you only want to be able to flip this per-app or per-platform, then you don't need the actual configure argument, just the variable and the logic that controls the setting based on the variable. I'd like to reduce the number of configure arguments, not increase them.

>diff --git a/toolkit/system/gnome/ b/toolkit/system/gnome/
>--- a/toolkit/system/gnome/
>+++ b/toolkit/system/gnome/
>@@ -44,20 +44,24 @@ include $(DEPTH)/config/
> MODULE		= mozgnome
> LIBRARY_NAME	= mozgnome
>-	nsGConfService.cpp \
> 	nsGnomeModule.cpp \
> 	$(NULL)

You could just move this assignment to be a single line now.

>+CPPSRCS += \
>+	nsGConfService.cpp \
>+	$(NULL)

Two-space indent on the continued lines. Don't use tabs for anything but rule commands in Makefiles, please.

r=me with those changes.
Comment 3 User image Ted Mielczarek [:ted.mielczarek] 2009-12-21 12:25:59 PST
Also, sorry for the delayed review!
Comment 4 User image Wolfgang Rosenauer [:wolfiR] 2009-12-22 23:40:14 PST
Checked in w/o a configure option as discussed in IRC and above style fixes.
(for indentation I followed the rest of the file before though)
Comment 5 User image Karsten Düsterloh 2009-12-25 09:22:05 PST
This caused bug 536745.

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