Skip to content

Commit

Permalink
Build curl with ZStandard support on vcpkg when serialization is enab…
Browse files Browse the repository at this point in the history
…led. (#4166) (#4192)

[SC-31595](https://app.shortcut.com/tiledb-inc/story/31595/enable-zstd-in-vcpkg-curl-build)

The vcpkg port for curl does not have a feature to enable zstd. I created one and opened microsoft/vcpkg#32533 to add it upstream.

---
TYPE: BUILD
DESC: Build curl with ZStandard support on vcpkg when serialization is enabled.

Co-authored-by: Theodore Tsirpanis <theodore.tsirpanis@tiledb.com>
  • Loading branch information
Shelnutt2 and teo-tsirpanis authored Jul 22, 2023
1 parent cced1b5 commit 69676e6
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 1 deletion.
1 change: 1 addition & 0 deletions ports/curl/portfile.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
winidn USE_WIN32_IDN
winldap USE_WIN32_LDAP
websockets ENABLE_WEBSOCKETS
zstd CURL_ZSTD
INVERTED_FEATURES
non-http HTTP_ONLY
winldap CURL_DISABLE_LDAP # Only WinLDAP support ATM
Expand Down
6 changes: 6 additions & 0 deletions ports/curl/vcpkg.json
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,12 @@
"dependencies": [
"wolfssl"
]
},
"zstd": {
"description": "ZStandard support (zstd)",
"dependencies": [
"zstd"
]
}
}
}
5 changes: 4 additions & 1 deletion vcpkg.json
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,10 @@
"serialization": {
"description": "Enable TileDB Cloud Support",
"dependencies": [
"curl",
{
"name": "curl",
"features": [ "zstd" ]
},
{
"name": "capnproto",
"version>=": "0.8.0"
Expand Down

0 comments on commit 69676e6

Please sign in to comment.