-
Notifications
You must be signed in to change notification settings - Fork 185
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
Allow use of vcpkg for handling local dependencies #3950
Conversation
This pull request has been linked to Shortcut Story #26199: Allow using vcpkg locally for all dependencies. |
8d3e80e
to
7de0fd1
Compare
@@ -34,6 +34,7 @@ Configuration: | |||
--help print this message | |||
--prefix=PREFIX install files in tree rooted at PREFIX | |||
['"${default_prefix}"'] | |||
--enable-vcpkg use vcpkg for downloading and building dependnecies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bootstrap.ps1
also needs to be updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have any way to test changes here so I didn't bother attempting to write something that might or might not work. I was hoping one of our Windows dev folks would be able to add the option after this PR is merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll preface this with a heavy "I have no idea what I'm talking about" disclaimer, but another thing that caught my eye in the vcpkg docs was that they seemed to focus on IDEs when talking about configuring Windows projects. I still think we should add a bootstrap.ps1 option for symmetry, but I wasn't sure if it'd actually be used or if there'd be some other set of instructions for configuring IDEs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dependnecies
-> dependeNcies
next time someone touches this
c686c06
to
c50965a
Compare
c50965a
to
3582c17
Compare
This adds a `--enable-vcpkg` configuration flag to CMake that allows developers to build all dependencies with vcpkg.
3582c17
to
c562d9f
Compare
@@ -34,6 +34,7 @@ Configuration: | |||
--help print this message | |||
--prefix=PREFIX install files in tree rooted at PREFIX | |||
['"${default_prefix}"'] | |||
--enable-vcpkg use vcpkg for downloading and building dependnecies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dependnecies
-> dependeNcies
next time someone touches this
They were introduced in TileDB-Inc#3950 (which says it just copied them from upstream vcpkg) and have not been modified since.
They were introduced in TileDB-Inc#3950 (which says it just copied them from upstream vcpkg) and have not been modified since.
They were introduced in TileDB-Inc#3950 (which says it just copied them from upstream vcpkg) and have not been modified since.
They were introduced in TileDB-Inc#3950 (which says it just copied them from upstream vcpkg) and have not been modified since.
This adds a
--enable-vcpkg
flag to bootstrap which allows using vcpkg to build all dependencies.I highly recommend reviewing this PR one commit at a time as the commit that adds all the port definitions is rather large and just copied from upstream vcpkg's registry.
TYPE: IMPROVEMENT
DESC: Allow use of vcpkg for installing dependencies locally