Fix build for Poppler 22.12 transition

Bug #1999764 reported by Nathan Teodosio
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
inkscape (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Inkscape won't build with Poppler >=22.09 because of calls such as

    state->getLineDash(&dash_pattern, &dash_length, &dash_start);

Poppler's getLineDash API changed.

The attached patch fixes the compilation, nonetheless the package to build (https://launchpad.net/~nteodosio/+archive/ubuntu/poppler/+build/24938248) because of a failed Pango test:

--->
304/307 Test #304: render_text-glyphs-combining ........................................***Failed 1.14 sec
Failed to get connection
** (inkscape:19505): CRITICAL **: 18:49:52.468: dbus_g_proxy_new_for_name: assertion 'connection != NULL' failed

** (inkscape:19505): CRITICAL **: 18:49:52.468: dbus_g_proxy_call: assertion 'DBUS_IS_G_PROXY (proxy)' failed

** (inkscape:19505): CRITICAL **: 18:49:52.468: dbus_g_connection_register_g_object: assertion 'connection != NULL' failed
end_font_face_cb: font face rule limited support.
  font-family : 'Noto Sans';
  src : url(fonts/NotoSans-Regular.ttf)
end_font_face_cb: Added font: /<<PKGBUILDDIR>>/testfiles/rendering_tests/fonts/NotoSans-Regular.ttf
Background RRGGBBAA: ffffff00
Area 0:0:600:600 exported to 600 x 600 pixels (96 dpi)
2
text-glyphs-combining FAILED
Failed to get connection
** (inkscape:19519): CRITICAL **: 18:49:52.688: dbus_g_proxy_new_for_name: assertion 'connection != NULL' failed

** (inkscape:19519): CRITICAL **: 18:49:52.688: dbus_g_proxy_call: assertion 'DBUS_IS_G_PROXY (proxy)' failed

** (inkscape:19519): CRITICAL **: 18:49:52.688: dbus_g_connection_register_g_object: assertion 'connection != NULL' failed
end_font_face_cb: font face rule limited support.
  font-family : 'Noto Sans';
  src : url(fonts/NotoSans-Regular.ttf)
end_font_face_cb: Added font: /<<PKGBUILDDIR>>/testfiles/rendering_tests/fonts/NotoSans-Regular.ttf
Background RRGGBBAA: ffffff00
Area 0:0:600:600 exported to 2400 x 2400 pixels (384 dpi)
text-glyphs-combining-large FAILED
<---

Tags: patch
Revision history for this message
Nathan Teodosio (nteodosio) wrote :
description: updated
Revision history for this message
Nathan Teodosio (nteodosio) wrote :

When I executed the tests locally, they succeeded.

        Start 304: render_text-glyphs-combining
304/307 Test #304: render_text-glyphs-combining ........................................ Passed 1.26 sec

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "fix-build-for-poppler-22-09.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
Jeremy Bícha (jbicha) wrote :

That test fails for me in my lunar sbuild also.

If your local environment is Kinetic, one big change is that Lunar includes a much newer harfbuzz than Kinetic.

Revision history for this message
Nathan Teodosio (nteodosio) wrote : Re: [Bug 1999764] Re: Fix build for Poppler 22.12 transition

Yes, that's right, I'm on Kinetic.

Revision history for this message
Jeremy Bícha (jbicha) wrote :

I've uploaded your patch along with some patches from Debian to skip the failing tests.

I've unsubscribed ubuntu-sponsors; please feel free to resubscribe if you have something else that needs to be sponsored.

https://launchpad.net/ubuntu/+source/inkscape/1.1.2-3ubuntu6

Changed in inkscape (Ubuntu):
status: New → Fix Committed
Revision history for this message
Jeremy Bícha (jbicha) wrote :

Closing issue that was fixed a long time ago

Changed in inkscape (Ubuntu):
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.