Skip to content

Commit

Permalink
unix: move net/if.h include
Browse files Browse the repository at this point in the history
This commit moves the net/if.h include into src/getaddrinfo.c to
prevent AIX compilation errors. With these symbols exposed
publicly, Node.js compilation failed on AIX by exposing Free(),
which conflicts with another API.

Refs: nodejs/node#16835
Refs: libuv#1445
PR-URL: libuv#1622
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
  • Loading branch information
cjihrig committed Nov 9, 2017
1 parent 921b030 commit c83f8b9
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 0 additions & 1 deletion include/uv-unix.h
Expand Up @@ -32,7 +32,6 @@
#include <netinet/tcp.h>
#include <arpa/inet.h>
#include <netdb.h>
#include <net/if.h>

#include <termios.h>
#include <pwd.h>
Expand Down
1 change: 1 addition & 0 deletions src/unix/getaddrinfo.c
Expand Up @@ -32,6 +32,7 @@
#include <stddef.h> /* NULL */
#include <stdlib.h>
#include <string.h>
#include <net/if.h> /* if_indextoname() */

/* EAI_* constants. */
#include <netdb.h>
Expand Down

0 comments on commit c83f8b9

Please sign in to comment.