[libspng] Install pkg-config on Windows - #53510
[libspng] Install pkg-config on Windows#53510Billy O'Neal (BillyONeal) wants to merge 3 commits into
Conversation
Related: microsoft#52891 Related: randy408/libspng#286 This changes libspng to install pkg-config on all platforms matching that installed by Debian. Also fix the declared licenses to match the SPDX expression. Their README says: >## License > >Code is licensed under the BSD 2-clause "Simplified" License. > >The project contains optimizations and test images from libpng, these are licensed under the [PNG Reference Library License version 2](http://www.libpng.org/pub/png/src/libpng-LICENSE.txt). but they don't have a copy of the license. The content under that license is incorporated into the .c so we have to install the whole thing.
| - foreach(libname ${spng_TARGETS}) | ||
| - configure_file(${CMAKE_CURRENT_SOURCE_DIR}/cmake/libspng.pc.in ${CMAKE_CURRENT_BINARY_DIR}/cmake/lib${libname}.pc @ONLY) | ||
| +foreach(libname ${spng_TARGETS}) | ||
| + configure_file(${CMAKE_CURRENT_SOURCE_DIR}/cmake/spng.pc.in ${CMAKE_CURRENT_BINARY_DIR}/cmake/${libname}.pc @ONLY) |
There was a problem hiding this comment.
Which names did this install, and which names does it install now?
There was a problem hiding this comment.
Kai Pastor (@dg0yt) libspng has two build systems. A meson one and a CMake one. The meson one installs spng.pc, which is also the name used in the Debian package and expected by libjpeg-turbo. This CMake one installs libspng.pc or libspng_static.pc. This PR resolves the conflict in favor of their meson build system on the grounds that that is what I see downstream customers expecting.
|
GPT 5.6 Sol reports (lest you think I would exempt myself):
I agree this is suboptimal but appears to be an explicit decision upstream has made so I think my hands are tied. The meson build system we are trying to match does not have the contrivance to build both static and dynamic in the same build like the CMake one has.
Stand by... |
On the other hand Debian is only ever installing the dynamic one. Maybe we should fix it up to always not have |
Related: #52891
Related: randy408/libspng#286
This changes libspng to install pkg-config on all platforms matching that installed by Debian.
Also fix the declared licenses to match the SPDX expression. Their README says:
but they don't have a copy of the license. The content under that license is incorporated into the .c so we have to install the whole thing.