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

cmake: add librtmp/rtmpdump option and detection #13373

Closed
wants to merge 1 commit into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Apr 15, 2024

Add CMake option USE_LIBRTMP. Disabled by default.

This library requires OpenSSL TLS-backend when linked statically.

Follow-up to #13364
Closes #13373


Requires:

@vszakats vszakats marked this pull request as draft April 15, 2024 11:45
@vszakats

This comment was marked as outdated.

@vszakats vszakats changed the title testing librtmp and AppleIDN updates cmake: add librtmp/rtmpdump option and detection Apr 15, 2024
@vszakats vszakats added the RTMP label Apr 15, 2024
@vszakats vszakats marked this pull request as ready for review April 15, 2024 18:37
Add CMake option `USE_LIBRTMP`. Disabled by default.

This library requires OpenSSL TLS-backend when linked statically.

Follow-up to 6eb9e65 curl#13364
Closes curl#13373
@vszakats vszakats closed this in edc2702 Apr 16, 2024
@vszakats vszakats deleted the tests0415 branch April 16, 2024 07:37
vszakats added a commit to vszakats/curl that referenced this pull request Apr 16, 2024
vszakats added a commit that referenced this pull request Apr 16, 2024
- `openssl_check_symbol_exists()` expects a 4th argument now.
  Follow-up to edc2702 #13373

- minor comment/script touch-ups.
  Follow-up to a362962 #11922

- fix indentation.

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

Successfully merging this pull request may close these issues.

None yet

1 participant