Inkscape fails to build on Window with GC errors

Bug #1530286 reported by jazzynico on 2015-12-31
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Critical
Alex Valavanis
Inkscape Devlibs
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) on 2015-12-31
tags: added: build regression
tags: added: win32
Changed in inkscape:
importance: Undecided → Critical
jazzynico (jazzynico) wrote :

libgc version is 7.1.

Changed in inkscape:
status: New → In Progress
assignee: nobody → Alex Valavanis (valavanisalex)
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
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
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
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.

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) on 2016-01-07
Changed in inkscape-devlibs:
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