[THRIFT-1846] Restore socket.h header to support builds with Android NDK

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.9
• Fix Version/s: 0.9.1
• Component/s: C++ - Library

Environment

Building with Android Linux NDK R8D (and, it looks like, all previous NDKs)

• Patch Info: Patch Available
• Assignee: Nicolas Trésegnie
• Reporter: Nate Rosenblum
• Created: 29/Jan/13 16:34
• Updated: 20/Feb/21 15:26
• Resolved: 08/Jun/13 03:15

Description

Unlike every other rational un.h that I've seen, the sys/un.h (actually linux/un.h) distributed with the Android NDK doesn't define sa_family_t or include another header that does. Removal of socket.h in THRIFT-1740 broke the Android build.

Issue Links

• is duplicated by: THRIFT-1865 Thrift don't build on Android