Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add vcpkg installation instructions #774

Closed
wants to merge 2 commits into from
Closed

Conversation

Adela0814
Copy link
Contributor

cli11 is available as a port in vcpkg, a C++ library manager that simplifies installation for cli11 and other project dependencies. Documenting the install process here will help users get started by providing a single set of commands to build cli11, ready to be included in their projects.

We also test whether our library ports build in various configurations (dynamic, static) on various platforms (OSX, Linux, Windows: x86, x64, UWP, ARM) to keep a wide coverage for users.

I'm a maintainer for vcpkg, and here is what the port script looks like. We try to keep the library maintained as close as possible to the original library.

@codecov
Copy link

codecov bot commented Sep 14, 2022

Codecov Report

Base: 99.06% // Head: 99.06% // No change to project coverage 👍

Coverage data is based on head (132b06b) compared to base (1a26afa).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #774   +/-   ##
=======================================
  Coverage   99.06%   99.06%           
=======================================
  Files          16       16           
  Lines        3964     3964           
=======================================
  Hits         3927     3927           
  Misses         37       37           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jzakrzewski
Copy link

Sorry for the negativity, but I'm not convinced that the project should document something it does not control. Such things tend to get outdated pretty easily.

@henryiii
Copy link
Collaborator

I'd be okay to help make PRs to keep this up to date if it becomes a problem. I'm a little more worried about making the README grow too much. I think I'd rather add this to the gitbook, and then link to it from the readme. I also need to move the gitbook over to a non-dead renderer. Let's keep it open but on hold until we work on the docs a bit.

@siilky
Copy link

siilky commented Apr 3, 2023

in the meanwhile, could we update vcpkg release version? It is still 2.2 there, which is a year old

@phlptp phlptp mentioned this pull request Jun 30, 2023
@phlptp
Copy link
Collaborator

phlptp commented Jun 30, 2023

changes are included in #901

henryiii pushed a commit that referenced this pull request Jul 11, 2023
This PR includes updates from #774 and #613.  
It updates the readme and docbook with some additional installation
instructions.

---------

Co-authored-by: Owen Parkins <[email protected]>
Co-authored-by: Mengna Li <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@phlptp phlptp closed this Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants