Fix zstd CMake config name#21538

Closed

dg0yt wants to merge 1 commit into

curl:masterfrom

Contributor

dg0yt commented May 8, 2026

They install zstdConfig.cmake, https://github.com/facebook/zstd/blob/885c79ba4ae8345e006f61bc97b270d4cf7ff076/build/cmake/CMakeModules/ZstdPackage.cmake#L33-L38.
With the Config.cmake pattern, this is a case-sensitive package name, zstd.

Congrats. This Find module now has zstd, Zstd and ZSTD vars.


          Fix zstd CMake config name

18175d4

github-actions Bot added build cmake labels

bagder requested a review from vszakats

May 8, 2026 16:14

dg0yt mentioned this pull request

[curl] Fix dependencies, cleanup microsoft/vcpkg#51597

Merged

vszakats approved these changes

View reviewed changes

vszakats closed this in

10d4b34

Member

vszakats commented May 11, 2026

Thanks!

dg0yt deleted the zstd branch

May 12, 2026 03:42

outcast36 pushed a commit to greearb/curl that referenced this pull request


          cmake: fix zstd CMake config name

4f13a50

They install `zstdConfig.cmake`,
https://github.com/facebook/zstd/blob/885c79ba4ae8345e006f61bc97b270d4cf7ff076/build/cmake/CMakeModules/ZstdPackage.cmake#L33-L38.
With the `Config.cmake` pattern, this is a case-sensitive package name,
`zstd`.

Follow-up to 8fce3e1 curl#20814
Closes curl#21538

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels