-
Notifications
You must be signed in to change notification settings - Fork 131
[EXPERIMENTAL] Add support for vcpkg
#293
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
Conversation
@tomerd This one is not WIP👀 The only thing we should do is to add such a manifest. And it just automatically works. |
Anyone to merge? |
What exactly does this do and how is it used? |
@neonichu This adds a manifest file of
flag passed on CMake, to let
|
The official builds and documentation don't install them on the system, they build them as part of the toolchain. |
@compnerd confused on how this relates to the toolchain build machinery already in place. should discuss this some more with @shahmishal and team? |
I don’t think that’s the case for SQLite3 here. We’re never building SQLite along with the toolchain. |
We actually do build SQLite3 from source as well. |
|
Sure, but then does it make sense to perhaps have this be residing elsewhere? |
@compnerd I believe we’d better place it within our repo to keep it tracked if the dependencies change. Alternatively, we can pull in another repo containing only |
Windows developers have a relatively difficult experience for pulling in dependencies. This patch adds support for
vcpkg
's manifest mode, which can seamlessly integrate with CMake.