configure should use libpng.pc pkgconfig file

Bug #261410 reported by hawkes
2
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Alex Valavanis

Bug Description

libpng12 provides a png.pc as well, so configure.ac should use this as a first choice for determining cflags as well

Tags: build
su_v (suv-lp)
tags: added: build
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Confirmed, except it seems that the pc module name is libpng

Changed in inkscape:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
su_v (suv-lp) wrote :

Related (has patch):
Bug #520521 “Detect properly the libpng so that inkscape can build against 1.4.x and further too”
<https://bugs.launchpad.net/inkscape/+bug/520521>

Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Just had another look at this... We *do* actually check for libpng.pc in configure.ac: PKG_CHECK_MODULES(INKSCAPE, ... libpng >= 1.2 ...)

However, there is a manual check for libpng too. I guess this is redundant. Running a test-build without it.

Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Fix committed in lp:inkscape r12152. We now just use pkg-config rather than unreliable manual check.

Changed in inkscape:
assignee: nobody → Alex Valavanis (valavanisalex)
milestone: none → 0.49
status: Triaged → Fix Committed
Bryce Harrington (bryce)
Changed in inkscape:
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.