Inkscape fails to build on Window with GC errors

Bug #1530286 reported by jazzynico
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Critical
Alex Valavanis
Inkscape Devlibs
Fix Released
Medium
jazzynico

Bug Description

Reproduced on Windows XP (32bit), Inkscape trunk rev. 14557 and devlibs r32, compiling with btool.
Not reproduced with trunk rev. 14538.

Error messages:
-----
cc : compile with 1 threads in parallel
compile src/inkgc/gc.cpp
Make error line 307: problem compiling: src/inkgc/gc.cpp: In function 'void Inkscape::GC::{anonymous}::do_init()':
src/inkgc/gc.cpp:31:20: error: 'GC_set_no_dls' was not declared in this scope
src/inkgc/gc.cpp:32:35: error: 'GC_set_all_interior_pointers' was not declared in this scope
src/inkgc/gc.cpp:33:32: error: 'GC_set_finalize_on_demand' was not declared in this scope
-----

Very likely a regression introduced in rev. 14543 (see http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/14543).

jazzynico (jazzynico)
tags: added: build regression
tags: added: win32
Changed in inkscape:
importance: Undecided → Critical
Revision history for this message
jazzynico (jazzynico) wrote :

libgc version is 7.1.

Changed in inkscape:
status: New → In Progress
assignee: nobody → Alex Valavanis (valavanisalex)
Revision history for this message
jazzynico (jazzynico) wrote :

I guess we should upgrade our win32 devlibs with GC 7.4 too.

Changed in inkscape-devlibs:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Hopefully fixed in lp:inkscape r14558. Once we've updated to gc >= 7.2 on all supported platforms, we can remove the conditional build.

Changed in inkscape:
status: In Progress → Fix Released
Revision history for this message
jazzynico (jazzynico) wrote :

Compiled successfully with GC 7.4.2 on Windows XP (32bit devlibs). More tests in progress.

Changed in inkscape-devlibs:
assignee: nobody → jazzynico (jazzynico)
status: Triaged → In Progress
Revision history for this message
jazzynico (jazzynico) wrote :

Thanks for your patch, Alex!

My tests applied to rev. 14557, without the patch. I'm not sure if other recent operating systems still use GC 7.1 (Debian stable and Ubuntu LTS switched to 7.2). So I guess we can remove you patch as soon as the devlibs are updated.

Revision history for this message
jazzynico (jazzynico) wrote :

GC updated (with 7.4.2) in the devlibs, rev. 54.

Changed in inkscape-devlibs:
status: In Progress → Fix Committed
jazzynico (jazzynico)
Changed in inkscape-devlibs:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.