After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 661871 - Mutter does not build without introspection
Mutter does not build without introspection
Status: RESOLVED FIXED
Product: mutter
Classification: Core
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: mutter-maint
mutter-maint
: 666014 669962 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-10-15 22:32 UTC by Sjoerd Job Postmus
Modified: 2012-02-13 03:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (689 bytes, patch)
2011-12-12 17:53 UTC, Lionel Landwerlin
committed Details | Review

Description Sjoerd Job Postmus 2011-10-15 22:32:36 UTC
As the title. Mutter does not build without introspection.

Reason:
function main() in "mutter.c" uses MetaPluginManager and related from "meta-plugin-manager.h". This header is conditionally loaded if HAVE_INTROSPECTION is defined.

Seeing as I am not very familiar with mutter, I'm not sure if #ifdef ... #endif-ing the part in main, or removing the #ifdef ... #endif around the header is the way to go. The first option at least allows it to compile on my computer.
Comment 1 Owen Taylor 2011-10-17 13:49:37 UTC
Why do you need mutter without introspection? While this doesn't sound hard to fix, my first inclination is to just remove the support for building mutter without introspection - if we never test it that way, it's just going to rot.
Comment 2 Sjoerd Job Postmus 2011-10-17 14:24:27 UTC
I was building mutter without introspection, because GTK failed to build with, and I just added '--enable-intropsection=no' to autogenargs in JHBuild, only to find it bites me.

Anyhow, if you supply a '--enable/--disable', '--with/--without', make sure it works, and otherwise drop it. If you don't plan to test it, drop it.

(Probably, it would still not allow me to build it, so you should probably check if the other packages have been build with introspection enabled as well, if you need that.)

So, drop the flag.
Comment 3 Florian Müllner 2011-12-12 17:47:28 UTC
*** Bug 666014 has been marked as a duplicate of this bug. ***
Comment 4 Lionel Landwerlin 2011-12-12 17:53:15 UTC
Hi there,

As a developer on the ex-moblin/ex-meego/etc... mutter plugin, I'm still compiling mutter without introspection (even though I'd like to change that at some point).

I'm attaching a patch, it's really a small patch. I would to see it integrated.

Thanks
Comment 5 Lionel Landwerlin 2011-12-12 17:53:41 UTC
Created attachment 203268 [details] [review]
patch
Comment 6 Jasper St. Pierre (not reading bugmail) 2012-02-13 03:57:47 UTC
*** Bug 669962 has been marked as a duplicate of this bug. ***