conan-io / conan-center-index

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

[package] CLI11 should not be header only #25737

Open vasama opened 1 hour ago

vasama commented 1 hour ago

Description

CLI11 supports building as a static library, but the Conan recipe is always header only.

Package and Environment Details

N/A

Conan profile

N/A

Steps to reproduce

N/A

Logs

N/A

uilianries commented 1 hour ago

Correct. The CLI11 project offers an option to build static library: https://github.com/CLIUtils/CLI11/blob/v2.4.2/CMakeLists.txt#L82

I would suggest adding header-only option and enabling it by default, so we will not affect the current Package ID for users.

vasama commented 1 hour ago

I implemented this here: https://github.com/vasama/conan-center-index/tree/cli11-precompiled Waiting for access in https://github.com/conan-io/conan-center-index/issues/4.