Skip to content

Commit

Permalink
cmnghttp2: check for HAVE_SIZEOF_SSIZE_T and not HAVE_SSIZE_T
Browse files Browse the repository at this point in the history
The `check_type_size(ssize_t SIZEOF_SSIZE_T` call in cmcurl (referenced
by the comment above) defines `HAVE_SIZEOF_SSIZE_T` and not
`HAVE_SSIZE_T`.  The `HAVE_SSIZE_T` variable *might* get defined, but
via the `CHECK_TYPE_SIZE(ssize_t SSIZE_T)` call in cmlibarchive, which
would be configured *after* cmnghttp2, and so the first configure would
lead to an invalid `cmnghttp2/config.h` file.
  • Loading branch information
alimpfard committed Jan 6, 2022
1 parent 13e71b7 commit 99acfe4
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions Utilities/cmnghttp2/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@ elseif(CMAKE_C_COMPILER_ID STREQUAL "PathScale")
endif()

# Re-use some check result cache entries from cmcurl:
# * HAVE_ARPA_INET_H
# * HAVE_NETINET_IN_H
# * HAVE_SSIZE_T
if(NOT HAVE_SSIZE_T)
# * HAVE_ARPA_INET_H (referenced in cmakeconfig.h.in)
# * HAVE_NETINET_IN_H (referenced in cmakeconfig.h.in)
# * HAVE_SIZEOF_SSIZE_T (referenced here)
if(NOT HAVE_SIZEOF_SSIZE_T)
set(ssize_t KWIML_INT_intptr_t)
endif()
configure_file(cmakeconfig.h.in config.h)
Expand Down

0 comments on commit 99acfe4

Please sign in to comment.