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: FindNGHTTP2 add static lib name to find_library call #13495

Closed
wants to merge 1 commit into from

Conversation

fuzzard
Copy link

@fuzzard fuzzard commented Apr 29, 2024

Adds the static library name, nghttp2_static as a name to search.

This provides cmake parity with the winbuild Makefile.vc allowing the cmake build to find and allow the link to static nghttp2 library

curl/winbuild/Makefile.vc

Lines 147 to 158 in dfdd978

!IF "$(ENABLE_NGHTTP2)"=="yes"
# compatibility bit, WITH_NGHTTP2 is the correct flag
WITH_NGHTTP2 = dll
USE_NGHTTP2 = true
NGHTTP2 = dll
!ELSEIF "$(WITH_NGHTTP2)"=="dll"
USE_NGHTTP2 = true
NGHTTP2 = dll
!ELSEIF "$(WITH_NGHTTP2)"=="static"
USE_NGHTTP2 = true
NGHTTP2 = static
!ENDIF

Adds the static library name, nghttp2_static as a name to search.
@vszakats vszakats changed the title [cmake] FindNGHTTP2 add static lib name to find_library call cmake: FindNGHTTP2 add static lib name to find_library call Apr 29, 2024
@vszakats
Copy link
Member

Thank you @fuzzard, merged in 86805bf.

@vszakats vszakats closed this Apr 29, 2024
@bagder
Copy link
Member

bagder commented Apr 30, 2024

@vszakats remember to add Closes #[num] next time!

@fuzzard fuzzard deleted the cmake_nghttp2-static branch April 30, 2024 07:11
@vszakats
Copy link
Member

@bagder I will do, sorry for missing this.

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

3 participants