[THRIFT-3414] #include <pwd.h> in THeaderTransport.h breaks windows build

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

Environment

Visual Studio 2013

• Patch Info: Patch Available
• Flags: Patch
• Assignee: Pascal Bach
• Reporter: Pascal Bach
• Created: 10/Nov/15 11:50
• Updated: 18/Mar/16 18:02
• Resolved: 18/Mar/16 18:02

Description

The #include <pwd.h> in THeaderTransport.h breaks windows build, because it doesn't have a pwd.h.

When removing the header I don't see any errors neither on Windows nor on Linux (GCC).

I think the file is not needed at all. The attached patch removes it.