Github user lizhenghn123 closed the pull request at:
https://github.com/apache/thrift/pull/739
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user lizhenghn123 commented on the pull request:
https://github.com/apache/thrift/pull/739#issuecomment-166468711
Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user kcris commented on the pull request:
https://github.com/apache/thrift/pull/739#issuecomment-165898653
I did. I have both removed Thrift.cpp from the project file, and also added
TProtocol.cpp and TOutput.cpp (see pull request #745, was merged already).
So this pr
Github user lizhenghn123 commented on the pull request:
https://github.com/apache/thrift/pull/739#issuecomment-165325703
add TProtocol.cpp and TOutput.cpp in libthrift.vcxproj
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user kcris commented on the pull request:
https://github.com/apache/thrift/pull/739#issuecomment-165162254
seems incomplete, until more files are added to the vcxproj, see pull
request #745
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user lizhenghn123 opened a pull request:
https://github.com/apache/thrift/pull/739
fix compile error(Windows7 x64) : there is no lib\cpp\src\thrift\Thriâ¦
* fix compile error(Windows7 x64) : there is no
lib\cpp\src\thrift\Thrift.cpp file.
You can merge this pull request