Summary: | cmake: don't hardcode libdir | ||
---|---|---|---|
Product: | poppler | Reporter: | Emilio Pozuelo Monfort <pochu27> |
Component: | general | Assignee: | poppler-bugs <poppler-bugs> |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | ilovezfs, pochu27 |
Version: | unspecified | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: | cmake: support GNUInstallDirs |
Description
Emilio Pozuelo Monfort
2017-10-11 08:56:22 UTC
Created attachment 134892 [details] [review] cmake: support GNUInstallDirs That patch seems ok to me, does it solve your issues? It does. With it I can build with -DCMAKE_INSTALL_LIBDIR="path/to/lib/dir" and things go to the appropriate place. Pushed Unfortunately this has the side-effect of breaking the pkg-config files for everyone else who isn't explicitly setting CMAKE_INSTALL_LIBDIR and CMAKE_INSTALL_INCLUDEDIR. The reason is that rather than ending up with libdir=${exec_prefix}/lib includedir=${prefix}/include instead, by default, you get libdir=lib includedir=include I expect 0.61.0 will be a bit of a mess from a downstream packaging perspective as a result. In Homebrew, while working on the 0.61.0 upgrade pull request, I just had to puzzle around over why none of the reverse dependencies would build until I found that the pkg-config files were broken and traced it back here. Multiply that by every other downstream and you get the picture. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.