[ 
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)

Reply via email to