deps: use nghttp2's config.h on all platforms

Fix warnings about use of htonl(), etc. by including config.h for all
platforms, defining HAVE_ARPA_INET_H on non-Windows, and therefore
including <arpa/inet.h>, which defines the host to network byte order
conversion functions.

PR-URL: https://github.com/nodejs/node/pull/27283
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
This commit is contained in:
Sam Roberts committed 2019-04-22 12:08:17 -07:00
1 parent 72308a5deb
commit 90cf2d5f00
2 files changed
+4 -2

No files matched your search

+3 -1
View File
@@ -54,7 +54,9 @@ typedef intptr_t ssize_t;
/* #undef NOTHREADS */
/* Define to 1 if you have the <arpa/inet.h> header file. */
/* #undef HAVE_ARPA_INET_H */
#ifndef _WIN32
# define HAVE_ARPA_INET_H 1
#endif
/* Define to 1 if you have the <fcntl.h> header file. */
#define HAVE_FCNTL_H 1
+1 -1
View File
@@ -12,13 +12,13 @@
'defines': [
'BUILDING_NGHTTP2',
'NGHTTP2_STATICLIB',
'HAVE_CONFIG_H',
],
'conditions': [
['OS=="win"', {
'defines': [
'WIN32',
'_WINDOWS',
'HAVE_CONFIG_H',
],
'msvs_settings': {
'VCCLCompilerTool': {