Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 620136 - sys-libs/zlib-1.2.11: mingw-w64: zlib.pc.in processed results in references to builddir
Summary: sys-libs/zlib-1.2.11: mingw-w64: zlib.pc.in processed results in references t...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2017-05-29 14:44 UTC by hanetzer
Modified: 2017-06-19 02:08 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
zlib.pc with incorrect values (zlib.pc,364 bytes, text/plain)
2017-05-29 14:44 UTC, hanetzer
Details
x86_64-w64-mingw32-emerge --info sys-libs/zlib (zlib.info,2.84 KB, application/x-info)
2017-05-29 14:46 UTC, hanetzer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description hanetzer 2017-05-29 14:44:32 UTC
Created attachment 474652 [details]
zlib.pc with incorrect values

as it currently stands mingw-w64 builds resolve the wrong values in
multilib_src_compile when processing the zlib.pc.in file with sed,
and end up with references to the build dir in the final file.

Unsure how to resolve this atm, will be looking into it.
Comment 1 hanetzer 2017-05-29 14:46:19 UTC
Created attachment 474654 [details]
x86_64-w64-mingw32-emerge --info sys-libs/zlib
Comment 2 hanetzer 2017-05-29 18:52:32 UTC
On a second look I am incorrect; the manual processing of zlib.pc.in
with sed does work, but gets installed to /usr/share/pkgconfig/zlib.pc,
along with /usr/lib/pkgconfig/zlib.pc created from win32/Makefile.gcc
which contains the incorrect references to the build dir.

I have made a pull request to https://github.com/gentoo/gentoo which
makes overwrites the win32/Makefile.gcc zlib.pc with the manually
processed, correct values zlib.pc.
Comment 3 Jonas Stein gentoo-dev 2017-05-29 21:32:07 UTC
Thank you for your contribution.
Comment 4 hanetzer 2017-06-19 02:08:16 UTC
Fixed with https://github.com/gentoo/gentoo/pull/4803