search

conan-io / conan-center-index

Recipes for the ConanCenter repository
https://conan.io/center
MIT License
918 stars 1.65k forks source link

[package] minizip-ng/*: libbsd isn't used #24413

Open Nekto89 opened 1 week ago

Nekto89 commented 1 week ago

Description

Hi,

After updating libbsd I've noticed that minizip-ng wasn't rebuilt. Recipe has option "with_libbsd" but doesn't use it for adding libbsd to requirements.

Package and Environment Details

Conan profile

[settings] arch=x86_64 build_type=Release compiler=gcc compiler.cppstd=17 compiler.libcxx=libstdc++11 compiler.version=10 os=Linux [conf] tools.cmake.cmaketoolchain:generator=Ninja

Steps to reproduce

conan graph info -r conancenter -o minizip-ng/*:with_libbsd=True --requires=minizip-ng/4.0.6 | grep "libbsd"

Logs

Click to expand log ``` conan graph info -r conancenter -o minizip-ng/*:with_libbsd=True --requires=minizip-ng/4.0.6 | grep "libbsd" ======== Computing dependency graph ======== Graph root cli Requirements bzip2/1.0.8#457c272f7da34cb9c67456dd217d36c4 - Cache libiconv/1.17#73fefc1b696e069df90fd1d18aa63edd - Cache minizip-ng/4.0.6#43d3e8e87793c04b61ada77e5e444cd7 - Cache openssl/3.2.2#899583c694f9deccec74dbe0bbc65a15 - Cache xz_utils/5.4.5#51e5a6e6564f4ea3afd79def01f035ad - Cache zlib/1.3.1#f52e03ae3d251dec704634230cd806a2 - Cache zstd/1.5.5#1f239731dc45147c7fc2f54bfbde73df - Cache Build requirements cmake/3.29.6#1a639e3e7401ea46e16c83ec9c5a7922 - Cache meson/1.2.2#04bdfb85d665c82b08a3510aee3ffd19 - Cache ninja/1.11.1#77587f8c8318662ac8e5a7867eb4be21 - Cache pkgconf/2.1.0#27f44583701117b571307cf5b5fe5605 - Cache Resolved version ranges cmake/[>=3.19 <4]: cmake/3.29.6 openssl/[>=1.1 <4]: openssl/3.2.2 zlib/[>=1.2.11 <2]: zlib/1.3.1 ======== Computing necessary packages ======== Requirements bzip2/1.0.8#457c272f7da34cb9c67456dd217d36c4:6056ff2fc700a644a5d2175eaaf80c21f2b10988 - Missing libiconv/1.17#73fefc1b696e069df90fd1d18aa63edd:3f0cbc1394cb3fd9f3473d67001da1a496fb5b26 - Missing minizip-ng/4.0.6#43d3e8e87793c04b61ada77e5e444cd7:271153345c9657f211c3b59df1b9cf907fa43466 - Missing openssl/3.2.2#899583c694f9deccec74dbe0bbc65a15:0f23c34c5ddf520533f038576fe7131a88782711 - Missing xz_utils/5.4.5#51e5a6e6564f4ea3afd79def01f035ad:3f0cbc1394cb3fd9f3473d67001da1a496fb5b26 - Missing zlib/1.3.1#f52e03ae3d251dec704634230cd806a2:3f0cbc1394cb3fd9f3473d67001da1a496fb5b26 - Missing zstd/1.5.5#1f239731dc45147c7fc2f54bfbde73df:7f788f70c9e77a96235b3978aa101db97eef891b - Missing Build requirements Skipped binaries cmake/3.29.6, meson/1.2.2, ninja/1.11.1, pkgconf/2.1.0 -------- Basic graph information -------- with_libbsd: True with_libbsd: True with_libbsd: ['True', 'False'] with_libbsd: True ```
RubenRBS commented 6 days ago

As far as I can see form upstream's CMakeLists, this is because libbsd comes from pkgconf, so there would not be a need to have a proper library.

Nekto89 commented 6 days ago

As far as I can see form upstream's CMakeLists, this is because libbsd comes from pkgconf, so there would not be a need to have a proper library.

what do you mean by this? openssl, liblzma and zstd also come through pkgconf

RubenRBS commented 6 days ago

Oh you mean that Conan is not requiring it! Sorry about that, I missunderstood most of your message :)