Skip to content

Migrate OSS project to handle protocol split #158

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Mizuchi
Copy link
Contributor

@Mizuchi Mizuchi commented May 14, 2025

Summary:
X-link: facebook/CacheLib#386

X-link: facebook/fb303#67

fbthrift has moved compact/binary protocol instantiation outside _types.cpp (to _types_compact.cpp and _types_binary.cpp). We need to add these two files to the build system.

Differential Revision: D74687663

Summary:
X-link: facebook/CacheLib#386

X-link: facebook/fb303#67

fbthrift has moved compact/binary protocol instantiation outside _types.cpp (to _types_compact.cpp and _types_binary.cpp). We need to add these two files to the build system.

Differential Revision: D74687663
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@Mizuchi Mizuchi force-pushed the export-D74687663 branch from 182e2d1 to b3c7037 Compare May 14, 2025 18:48
Mizuchi added a commit to Mizuchi/mcrouter that referenced this pull request May 14, 2025
Summary:
X-link: facebook/openr#158

X-link: facebook/fboss#434

X-link: facebookincubator/fizz#162

X-link: facebookexperimental/moxygen#58

X-link: facebook/CacheLib#386

X-link: facebook/fb303#67

fbthrift has moved compact/binary protocol instantiation outside _types.cpp (to _types_compact.cpp and _types_binary.cpp). We need to add these two files to the build system.

Differential Revision: D74687663
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants