{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet/corefx/issues/4069","labels_url":"https://api.github.com/repos/dotnet/corefx/issues/4069/labels{/name}","comments_url":"https://api.github.com/repos/dotnet/corefx/issues/4069/comments","events_url":"https://api.github.com/repos/dotnet/corefx/issues/4069/events","html_url":"https://github.com/dotnet/corefx/pull/4069","id":112791802,"number":4069,"title":"native: Removes redundant check","user":{"login":"jasonwilliams200OK","id":2031765,"avatar_url":"https://avatars.githubusercontent.com/u/2031765?v=3","gravatar_id":"","url":"https://api.github.com/users/jasonwilliams200OK","html_url":"https://github.com/jasonwilliams200OK","followers_url":"https://api.github.com/users/jasonwilliams200OK/followers","following_url":"https://api.github.com/users/jasonwilliams200OK/following{/other_user}","gists_url":"https://api.github.com/users/jasonwilliams200OK/gists{/gist_id}","starred_url":"https://api.github.com/users/jasonwilliams200OK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasonwilliams200OK/subscriptions","organizations_url":"https://api.github.com/users/jasonwilliams200OK/orgs","repos_url":"https://api.github.com/users/jasonwilliams200OK/repos","events_url":"https://api.github.com/users/jasonwilliams200OK/events{/privacy}","received_events_url":"https://api.github.com/users/jasonwilliams200OK/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dotnet/corefx/labels/cla-already-signed","name":"cla-already-signed","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-10-22T12:25:25Z","updated_at":"2015-10-22T17:40:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dotnet/corefx/pulls/4069","html_url":"https://github.com/dotnet/corefx/pull/4069","diff_url":"https://github.com/dotnet/corefx/pull/4069.diff","patch_url":"https://github.com/dotnet/corefx/pull/4069.patch"},"body":"`sys/socket.h` has the definition of `AP_PACKET` on Linux and `AP_LINK`\r\non OSX and BSD. We were testing the capability by compiling the source\r\nin cmake, which was failing on FreeBSD because of the missing\r\n`sys/types.h`, but as it turned out this check was not required."},"comment":{"url":"https://api.github.com/repos/dotnet/corefx/issues/comments/150301335","html_url":"https://github.com/dotnet/corefx/pull/4069#issuecomment-150301335","issue_url":"https://api.github.com/repos/dotnet/corefx/issues/4069","id":150301335,"user":{"login":"mellinoe","id":8918977,"avatar_url":"https://avatars.githubusercontent.com/u/8918977?v=3","gravatar_id":"","url":"https://api.github.com/users/mellinoe","html_url":"https://github.com/mellinoe","followers_url":"https://api.github.com/users/mellinoe/followers","following_url":"https://api.github.com/users/mellinoe/following{/other_user}","gists_url":"https://api.github.com/users/mellinoe/gists{/gist_id}","starred_url":"https://api.github.com/users/mellinoe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mellinoe/subscriptions","organizations_url":"https://api.github.com/users/mellinoe/orgs","repos_url":"https://api.github.com/users/mellinoe/repos","events_url":"https://api.github.com/users/mellinoe/events{/privacy}","received_events_url":"https://api.github.com/users/mellinoe/received_events","type":"User","site_admin":false},"created_at":"2015-10-22T17:40:50Z","updated_at":"2015-10-22T17:40:50Z","body":"I'm fine with this approach, but like pat said, use #if defined() everywhere, I guess that's the style we settled on."}}