FS#52703 - [libpng12] libpng12.pc deleted along with libpng.pc

Attached to Project: Community Packages
Opened by tdy (tdy) - Wednesday, 25 January 2017, 03:44 GMT
Last edited by Doug Newgard (Scimmia) - Wednesday, 25 January 2017, 08:24 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To No-one
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

The whole /usr/lib/pkgconfig folder is deleted, which removes both libpng.pc and libpng12.pc. Without libpng12.pc, configure scripts can't check/use libpng12 with pkg-config (e.g. I have a package that tries `pkg-config --check libpng12`).

If we delete libpng.pc and keep libpng12.pc, it should still be compatible with extra/libpng as far as I can tell.
   libpng12-pc.diff (0.5 KiB)
This task depends upon

Closed by  Doug Newgard (Scimmia)
Wednesday, 25 January 2017, 08:24 GMT
Reason for closing:  Not a bug
Comment by tdy (tdy) - Wednesday, 25 January 2017, 03:46 GMT
Had a leftover comment in the original patch
   libpng12-pc-fixed.diff (0.4 KiB)
Comment by tdy (tdy) - Wednesday, 25 January 2017, 03:51 GMT
This can be worked around (e.g. manually adding -lpng12), so maybe just a feature request
Comment by Doug Newgard (Scimmia) - Wednesday, 25 January 2017, 05:45 GMT
What package are we talking about? The .pc file isn't all that useful without the headers, and that request has already been rejected.
Comment by tdy (tdy) - Wednesday, 25 January 2017, 06:15 GMT
Package = community/libpng12. Sorry I wasn't aware of any previous discussion on this. I searched for libpng12 on here and didn't see any hits.

I don't really know what goes on under the hood, so I'm not sure if there's any other use for the pc file. In my case I came across a configure script tries to check for libpng12 with $(pkg-config --exists libpng12) and then tries to add -lpng12 with $(pkg-config --libs libpng12). For now I just manually added `-lpng12` to the compile args and it works, but I just didn't get why libpng12.pc had been removed.
Comment by Doug Newgard (Scimmia) - Wednesday, 25 January 2017, 06:31 GMT
No, I mean the package that's causing you problem.
Comment by tdy (tdy) - Wednesday, 25 January 2017, 06:39 GMT
Ohh it's this one: https://github.com/tim-janik/rapicorn
Comment by Doug Newgard (Scimmia) - Wednesday, 25 January 2017, 07:38 GMT
That's a bad configure script, and it appears it was fixed in October
Comment by tdy (tdy) - Wednesday, 25 January 2017, 07:46 GMT
Ah I see it now, master branch checks for libpng since 9cd1fcc. Sorry for the noise.

Loading...