[
https://issues.apache.org/jira/browse/HDFS-7577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14285057#comment-14285057
]
Colin Patrick McCabe commented on HDFS-7577:
--------------------------------------------
This looks pretty good.
{{os/windows/cpuid.h}}: this is x86-specific, right? Can we put this in an x86
ifdef like the one from HWCrc32c.cc?
Something like this maybe:
{code}
#if ((defined(__X86__) || defined(__i386__) || defined(i386) ||
defined(_M_IX86) || defined(__386__) || defined(__x86_64__) || defined(_M_X64)))
{code}
> Add additional headers that includes need by Windows
> ----------------------------------------------------
>
> Key: HDFS-7577
> URL: https://issues.apache.org/jira/browse/HDFS-7577
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: hdfs-client
> Reporter: Thanh Do
> Assignee: Thanh Do
> Attachments: HDFS-7577-branch-HDFS-6994-0.patch
>
>
> This jira involves adding a list of (mostly dummy) headers that available in
> POSIX systems, but not in Windows. One step towards making libhdfs3 built in
> Windows.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)