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 686305 - gnome-themes-standard 3.6.1 fails to build with Quartz backend
gnome-themes-standard 3.6.1 fails to build with Quartz backend
Status: RESOLVED FIXED
Product: gnome-themes-standard
Classification: Core
Component: general
3.6.x
Other Mac OS
: Normal normal
: ---
Assigned To: gnome-themes-standard-maint
gnome-themes-standard-maint
Depends on:
Blocks:
 
 
Reported: 2012-10-17 15:06 UTC by su-v
Modified: 2012-10-17 20:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (3.03 KB, patch)
2012-10-17 18:05 UTC, Cosimo Cecchi
committed Details | Review
Minimally modified version of the patch to apply against tarball of v3.6.1 (2.99 KB, patch)
2012-10-17 20:12 UTC, su-v
committed Details | Review

Description su-v 2012-10-17 15:06:57 UTC
Latest release 3.6.1 fails to build on OS X if GTK2 and/or GTK3 are installed with the Quartz backend:

- configure.ac (line 65) checks explicitly for 'gdk-x11-2.0', but the Quartz backend of GTK2 installs 'gdk-quartz-2.0' instead
- src/adwaita_engine.c: includes header file only available with X11 backend of GTK+ (<gdk/gdkx.h>), without further testing.
- themes/Adwaita/gtk-2.0/adwaita_engine.c: includes header file only available with X11 backend of GTK2 (<gdk/gdkx.h>), without testing e.g. with '#ifdef GDK_WINDOWING_X11'.

AFAICT (I'm not a developer myself) a few additional changes are required for both theme engines to compile without X11 backend.

(gnome-themes-standard 3.6.0.2 built ok out-of-the-box, with the same setup on OS X 10.7.4)
Comment 1 Cosimo Cecchi 2012-10-17 18:04:48 UTC
I don't have an OSX machine to test this currently; could you please test if the attached patch works for you? It's against git master (but it can be easily backported on top of 3.6.1). Thanks.
Comment 2 Cosimo Cecchi 2012-10-17 18:05:22 UTC
Created attachment 226669 [details] [review]
patch
Comment 3 su-v 2012-10-17 20:12:24 UTC
Created attachment 226687 [details] [review]
Minimally modified version of the patch to apply against tarball of v3.6.1

Many thanks for your prompt response and the patch, much appreciated!

To make the patch apply to the sources from the 3.6.1 tarball, I had to change the path for the GTK3 theme engine file to 'src/adwaita_engine.c' (modified version attached). Other than that, the patch applied ok, building was successful, and the demo apps as well as Inkscape trunk (with GTK2 and with GTK3 (still experimental)) load and use the new Adwaita theme engine ok.
Comment 4 Cosimo Cecchi 2012-10-17 20:59:02 UTC
Thanks, I pushed the patch to both the master and gnome-3-6 git branches now.