facebook / fbthrift

Facebook's branch of Apache Thrift, including a new C++ server.
Apache License 2.0
2.55k stars 608 forks source link

Install Thrift annotations #550

Closed vitaut closed 1 year ago

vitaut commented 1 year ago

Summary: CMake install rules for Thrift annotations introduced by D35953194 are completely wrong. We should be installing .thrift files, not just generated C++ files.

Fix this, apply minor cleanup and remove most of the copy-pasta.

Differential Revision: D45360104

facebook-github-bot commented 1 year ago

This pull request was exported from Phabricator. Differential Revision: D45360104

facebook-github-bot commented 1 year ago

This pull request was exported from Phabricator. Differential Revision: D45360104

facebook-github-bot commented 1 year ago

This pull request was exported from Phabricator. Differential Revision: D45360104