[
https://issues.apache.org/jira/browse/THRIFT-1387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13125304#comment-13125304
]
Hudson commented on THRIFT-1387:
--------------------------------
Integrated in Thrift #294 (See [https://builds.apache.org/job/Thrift/294/])
THRIFT-1387 Build MSVC libraries with Boost Threads instead of Pthreads
Patch: Peace C & Alexandre Parenteau
roger : http://svn.apache.org/viewvc/?view=rev&rev=1182024
Files :
* /thrift/trunk/lib/cpp/README_WINDOWS
* /thrift/trunk/lib/cpp/libthrift.vcxproj
* /thrift/trunk/lib/cpp/libthrift.vcxproj.filters
* /thrift/trunk/lib/cpp/libthriftnb.vcxproj
* /thrift/trunk/lib/cpp/libthriftnb.vcxproj.filters
* /thrift/trunk/lib/cpp/src/transport/TFDTransport.cpp
* /thrift/trunk/lib/cpp/src/transport/TSimpleFileTransport.cpp
* /thrift/trunk/lib/cpp/src/windows/Fcntl.cpp
* /thrift/trunk/lib/cpp/src/windows/Fcntl.h
* /thrift/trunk/lib/cpp/src/windows/WinFcntl.cpp
* /thrift/trunk/lib/cpp/src/windows/WinFcntl.h
* /thrift/trunk/lib/cpp/src/windows/config.h
* /thrift/trunk/lib/cpp/src/windows/force_inc.h
* /thrift/trunk/lib/cpp/thrift.sln
> Build MSVC libraries with Boost Threads instead of Pthreads
> -----------------------------------------------------------
>
> Key: THRIFT-1387
> URL: https://issues.apache.org/jira/browse/THRIFT-1387
> Project: Thrift
> Issue Type: Improvement
> Components: C++ - Library
> Affects Versions: 0.8
> Environment: Windows - Visual Studio
> Reporter: Peace C
> Assignee: alexandre parenteau
> Priority: Minor
> Labels: boost, msvc, pthreads, studio, threads, visual, windows
> Fix For: 0.8
>
> Attachments: THRIFT-1387_BoostThreadsWin.patch,
> THRIFT-1387_BoostThreadsWin_v2.patch
>
>
> This patch is based on Alex's Boost threads patch (THRIFT-1361) and adds
> support in the MSVC project to build with Boost threads instead of Pthreads.
> The option is configurable using "#define HAVE_PTHREAD_H" in
> thrift/lib/cpp/src/windows/config.h (default is to build using Boost
> threads). In order to support building either way, I wrapped #if(n)def
> USE_BOOST_THREAD around Monitor.cpp, Mutex.cpp, PosixThreadFactory.cpp and
> their analogous Boost versions (BoostMonitor.cpp, BoostMutex.cpp,
> BoostThreadFactory.cpp).
> Remember that any code calling errno should add force_inc.h to the project's
> Force Include list. This tripped me up in TNonblockingServer.cpp when errno
> would always return 0. Using the project properties is more opaque but we
> can't add force_inc.h to config.h because any project using the thrift
> library will get the forced redefinitions (not good if the app expects the MS
> SDK definitions).
> I also adjusted the thrift library projects to generate appropriately named
> pdb files. They were defaulting to vc100.pdb, causing one project's build to
> stomp over the other's pdb.
> The patch was created using TortoiseHg workbench. Please notify me if the
> format isn't compatible with the Thrift team's patch tool and I will modify
> the patch accordingly.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
