Skip to content
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

Enable building libcurl with zstd support #3469

Merged

Conversation

Shelnutt2
Copy link
Member

@Shelnutt2 Shelnutt2 commented Aug 20, 2022

This adds support for curl to enable zstd compression in linux/macos builds. Windows uses cmake for building and the curl cmake does not allow for setting the library paths for zstd. This means the windows superbuild is not supported. For now we just enable on linux/macos.

The goal of enable zstd to is to allow this as an option for http compression.


TYPE: IMPROVEMENT
DESC: Enable superbuild libcurl to support zstd

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #20681: Support zstd for curl http compression.

This adds support for curl to enable zstd compression in linux/macos builds.
Windows uses cmake for building and the curl cmake does not allow for
setting the library paths for zstd. This means the windows superbuild is
not supported. For now we just enable on linux/macos.

The goal of enable zstd to is to allow this as an option for http compression.
@Shelnutt2 Shelnutt2 force-pushed the sethshelnutt/sc-20681/support-zstd-for-curl-http-compression branch from 363d1f1 to e47ab64 Compare August 20, 2022 14:17
cmake/Modules/FindZstd_EP.cmake Show resolved Hide resolved
@Shelnutt2 Shelnutt2 force-pushed the sethshelnutt/sc-20681/support-zstd-for-curl-http-compression branch from e86d656 to 9d3ec7b Compare August 22, 2022 12:25
@ihnorton ihnorton merged commit 51f9c8d into dev Aug 22, 2022
@ihnorton ihnorton deleted the sethshelnutt/sc-20681/support-zstd-for-curl-http-compression branch August 22, 2022 19:48
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.

None yet

3 participants