build failure introduced by r14955 with autotools-based build

Bug #1589300 reported by su_v on 2016-06-05
This bug affects 1 person
Affects Status Importance Assigned to Milestone

Bug Description

Latest trunk r14955 fails to compile with autotools-based build:

  CXX extension/param/parameter.o
In file included from /Volumes/magenta/mp-trunk/x11/include/glibmm-2.4/glibmm.h:89,
                 from /Volumes/magenta/mp-trunk/x11/include/gtkmm-2.4/gtkmm/table.h:7,
                 from ./ui/widget/color-notebook.h:25,
                 from extension/param/parameter.cpp:25:
/Volumes/magenta/mp-trunk/x11/include/glibmm-2.4/glibmm/threads.h:209: error: field ‘gobject_’ has incomplete type
/Volumes/magenta/mp-trunk/x11/include/glibmm-2.4/glibmm/threads.h: In member function ‘GThread* Glib::Threads::Thread::gobj()’:
/Volumes/magenta/mp-trunk/x11/include/glibmm-2.4/glibmm/threads.h:205: error: ‘gobject_’ was not declared in this scope
/Volumes/magenta/mp-trunk/x11/include/glibmm-2.4/glibmm/threads.h: In member function ‘const GThread* Glib::Threads::Thread::gobj() const’:
/Volumes/magenta/mp-trunk/x11/include/glibmm-2.4/glibmm/threads.h:206: error: ‘gobject_’ was not declared in this scope
make[3]: *** [extension/param/parameter.o] Error 1

A cmake-based build on the other hand succeeds (same build environment otherwise, all tests done on OS X 10.7.5), as does one with btool on Windows (as confirmed on irc by the author of r14955).

Changing the order of the includes (see attached diff) allows to compile r14955 successfully with autotools-based build. Please review the proposed diff and apply if ok.

su_v (suv-lp) wrote :
jazzynico (jazzynico) on 2016-06-05
Changed in inkscape:
assignee: nobody → su_v (suv-lp)
importance: Undecided → High
milestone: none → 0.92
status: New → In Progress
Patrick Storz (ede123) wrote :
Changed in inkscape:
status: In Progress → Fix Committed
su_v (suv-lp) on 2016-06-11
Changed in inkscape:
assignee: su_v (suv-lp) → nobody
Bryce Harrington (bryce) on 2017-01-10
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers