mirror of
https://github.com/curl/curl.git
synced 2024-11-27 05:50:21 +08:00
build: remove redundant CURL_PULL_*
settings
These macros were not propagated to the source code from CMake. autotools set only one of them (`CURL_PULL_SYS_POLL_H`), initially to address an AIX issue [1]. This later broke when introducing `system.h` [2] without the logic it enabled. A subsequent fix [3] re-added the logic, and also enabled it for AIX before its use, directly in `system.h`. [1] 2012-11-23:665adcd4b7
[2] 2017-03-29:9506d01ee5
#1373 [3] 2017-08-25:8a84fcc4b5
#1828 #1833 Reviewed-by: Daniel Stenberg Closes #12502
This commit is contained in:
parent
6265718c77
commit
ee6992c66a
@ -1386,13 +1386,6 @@ if(CMAKE_COMPILER_IS_GNUCC AND APPLE)
|
||||
endif()
|
||||
endif()
|
||||
|
||||
# TODO test which of these headers are required
|
||||
if(NOT WIN32)
|
||||
set(CURL_PULL_SYS_TYPES_H ${HAVE_SYS_TYPES_H})
|
||||
set(CURL_PULL_SYS_SOCKET_H ${HAVE_SYS_SOCKET_H})
|
||||
set(CURL_PULL_SYS_POLL_H ${HAVE_SYS_POLL_H})
|
||||
endif()
|
||||
|
||||
include(CMake/OtherTests.cmake)
|
||||
|
||||
add_definitions(-DHAVE_CONFIG_H)
|
||||
|
35
acinclude.m4
35
acinclude.m4
@ -156,7 +156,6 @@ AC_DEFUN([CURL_CHECK_AIX_ALL_SOURCE], [
|
||||
#endif])
|
||||
AC_BEFORE([$0], [AC_SYS_LARGEFILE])dnl
|
||||
AC_BEFORE([$0], [CURL_CONFIGURE_REENTRANT])dnl
|
||||
AC_BEFORE([$0], [CURL_CONFIGURE_PULL_SYS_POLL])dnl
|
||||
AC_MSG_CHECKING([if OS is AIX (to define _ALL_SOURCE)])
|
||||
AC_EGREP_CPP([yes_this_is_aix],[
|
||||
#ifdef _AIX
|
||||
@ -1116,40 +1115,6 @@ cat >>confdefs.h <<_EOF
|
||||
_EOF
|
||||
])
|
||||
|
||||
dnl CURL_CONFIGURE_PULL_SYS_POLL
|
||||
dnl -------------------------------------------------
|
||||
dnl The need for the sys/poll.h inclusion arises mainly to properly
|
||||
dnl interface AIX systems which define macros 'events' and 'revents'.
|
||||
|
||||
AC_DEFUN([CURL_CONFIGURE_PULL_SYS_POLL], [
|
||||
AC_REQUIRE([CURL_INCLUDES_POLL])dnl
|
||||
#
|
||||
tst_poll_events_macro_defined="unknown"
|
||||
#
|
||||
AC_COMPILE_IFELSE([
|
||||
AC_LANG_PROGRAM([[
|
||||
$curl_includes_poll
|
||||
]],[[
|
||||
#if defined(events) || defined(revents)
|
||||
return 0;
|
||||
#else
|
||||
force compilation error
|
||||
#endif
|
||||
]])
|
||||
],[
|
||||
tst_poll_events_macro_defined="yes"
|
||||
],[
|
||||
tst_poll_events_macro_defined="no"
|
||||
])
|
||||
#
|
||||
if test "$tst_poll_events_macro_defined" = "yes"; then
|
||||
if test "x$ac_cv_header_sys_poll_h" = "xyes"; then
|
||||
CURL_DEFINE_UNQUOTED([CURL_PULL_SYS_POLL_H])
|
||||
fi
|
||||
fi
|
||||
#
|
||||
])
|
||||
|
||||
|
||||
dnl CURL_CHECK_FUNC_SELECT
|
||||
dnl -------------------------------------------------
|
||||
|
@ -3500,8 +3500,6 @@ CURL_RUN_IFELSE(
|
||||
AC_MSG_RESULT([no])
|
||||
])
|
||||
|
||||
CURL_CONFIGURE_PULL_SYS_POLL
|
||||
|
||||
TYPE_IN_ADDR_T
|
||||
|
||||
TYPE_SOCKADDR_STORAGE
|
||||
|
Loading…
Reference in New Issue
Block a user