[ 
https://issues.apache.org/jira/browse/THRIFT-591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12918925#action_12918925
 ] 

Richard Swift commented on THRIFT-591:
--------------------------------------

Thanks to everyone for the rapid responses and thanks to Steve for the leg up. 
As I mentioned, Thrift is still under evaluation for us and, although things 
are looking very good so far (Thanks Thrift team), I won't be able to put any 
concerted effort into this until we make a decision. In the interim, time 
permitting I will take Steve's work and use it as my starting point.

I agree with Roger's comments and will take them into consideration going 
forwards. The breakout he suggests makes sense to me and I'll watch this thread 
for additional comments on that topic. I assume that the initial direction 
taken by Rush for the first two bullets is satisfactory. 

Finally, If anyone else has the time and wants to claim some part that would be 
great. I'm new to Thrift and only recently returned to Windows development so I 
won't be as quick as someone with a smaller learning curve :-) From our 
perspective, removing the Cygwin/MinGW dependencies is the critical part. the 
MSVC stuff is just nice-to-have.

Also new to the process. If there are specific steps to follow please point me 
to them. I'll go and look over the docs/wiki when I have a chance.

Thanks,

/richard



> Make the C++ runtime library be compatible with Windows and Visual Studio
> -------------------------------------------------------------------------
>
>                 Key: THRIFT-591
>                 URL: https://issues.apache.org/jira/browse/THRIFT-591
>             Project: Thrift
>          Issue Type: New Feature
>          Components: C++ - Library
>         Environment: Windows XP and above, Visual Studio 2005 and above
>            Reporter: Rush Manbert
>         Attachments: MsvcPatchSupportScripts.zip, 
> MSVCSupportUpdatedButUntestedForThrift-0.4.0.patch, 
> THRIFT-591_MSVCSupportUpdatedButUntestedForThrift-0.4.0.patch_error.log, 
> thrift-818530-patched.zip, ThriftMsvcPatchForSvnRev818530.txt.zip, 
> thriftWindowsRev818530BugFix.zip
>
>
> Modify the C++ runtime library to be compatible with Windows and able to be 
> built by Visual Studio.
> The work has been done and a patch is available. I will attach it soon.
> Note that this issue and the attached patch supercedes the patches that I 
> wrongly attached to JIRA 311. That issue is about making the C++ library 
> support async client/server interaction.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to