Skip to content

Add vcpkg installation instructions#13

Merged
vczh merged 1 commit intovczh-libraries:masterfrom
MonicaLiu0311:vcpkg-installition
Oct 18, 2022
Merged

Add vcpkg installation instructions#13
vczh merged 1 commit intovczh-libraries:masterfrom
MonicaLiu0311:vcpkg-installition

Conversation

@MonicaLiu0311
Copy link
Contributor

@MonicaLiu0311 MonicaLiu0311 commented Sep 14, 2022

vlpp is available as a port in vcpkg, a C++ library manager that simplifies installation for vlpp and other project dependencies. Documenting the install process here will help users get started by providing a single set of commands to build vlpp, 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.

@vczh
Copy link
Member

vczh commented Sep 16, 2022

Could you paste here the commit where you add vlpp to vcpkg? I would like to check if it is submitted as what I am expecting.

@MonicaLiu0311
Copy link
Contributor Author

MonicaLiu0311 commented Sep 16, 2022

Could you paste here the commit where you add vlpp to vcpkg? I would like to check if it is submitted as what I am expecting.

OK.
The link to add vlpp for the first time: the first commit
The link to portfile.cmake: portfile.cmake
The link to vcpkg: vcpkg

@vczh
Copy link
Member

vczh commented Sep 16, 2022

Looks like the package is not implemented correctly, let me take a look and see how could it reflect the designed way to use.

@vczh
Copy link
Member

vczh commented Sep 20, 2022

I didn't find a good solution, I decided to warn people and tell them not to use the vcpkg version. Thanks for your contribution.

@vczh vczh closed this Sep 20, 2022
@vczh
Copy link
Member

vczh commented Sep 20, 2022

4448477

@JackBoosY
Copy link
Contributor

I will fix the configure issues and already called you internal.

@vczh
Copy link
Member

vczh commented Sep 23, 2022

The latest draft PR looks promising, if that works I will merge this PR.

@MonicaLiu0311
Copy link
Contributor Author

The vlpp issue has been dealt with, please see #26887 for details. Can you please merge this PR?

@vczh vczh merged commit 9fe6111 into vczh-libraries:master Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants