[
https://issues.apache.org/jira/browse/THRIFT-2798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16742777#comment-16742777
]
yuanyuan chen commented on THRIFT-2798:
---------------------------------------
[~jking3] I don't know which versions of VS can be installed on XP and support
C++ 11 at the same time.But it is time to drop XP support.
> Thrift does not compile on Windows with TARGET_WIN_XP=0
> -------------------------------------------------------
>
> Key: THRIFT-2798
> URL: https://issues.apache.org/jira/browse/THRIFT-2798
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Library
> Affects Versions: 0.9.1, 0.9.2
> Reporter: Alex Merry
> Priority: Major
>
> To compile with Visual Studio 2013 after defining TARGET_WIN_XP=0 (and
> HAVE_STDINT_H=1), I have to do the following:
> # Remove BoostThreadFactory.cpp and BoostThreadFactory.h from concurrency
> # Add
> \#ifndef WINVER
> \# define WINVER 0x0601
> \#endif
> after
> \#ifndef _WIN32_WINNT
> \# define _WIN32_WINNT 0x0601
> \#endif
> in windows/config.h
> # Remove #include <thrift/transport/PlatformSocket.h> from Thrift.h (or move
> it to after #include <thrift/thrift-config.h>), since it depends on the
> WINVER definition from windows/config.h.
> I'm not sure what the best fix for the boost thread factory thing is (maybe
> #ifdef the file contents based on USE_BOOST_THREAD?), but I think the other
> code changes should go in.
> It may also be worth having
> \#if _MSC_VER >= 1800
> \# define HAVE_STDINT_H
> \#endif
> in windows/config.h.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
