Skip to content
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

Fix OSS build #518

Closed
wants to merge 1 commit into from
Closed

Fix OSS build #518

wants to merge 1 commit into from

Conversation

kvtsoy
Copy link
Contributor

@kvtsoy kvtsoy commented Sep 20, 2024

Summary:
#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Differential Revision: D63148510

@facebook-github-bot
Copy link
Contributor

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

Summary:
X-link: facebookincubator/zstrong#1007


facebook#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

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

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

facebook-github-bot pushed a commit to facebook/fb303 that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebookexperimental/edencommon that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebookexperimental/rust-shed that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in eaa20ff.

facebook-github-bot pushed a commit to facebook/openr that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/wangle that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/mvfst that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebookexperimental/moxygen that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/fboss that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/folly that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/watchman that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebookincubator/fizz that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebookincubator/katran that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebookincubator/hsthrift that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/sapling that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/fbthrift that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
facebook-github-bot pushed a commit to facebook/hhvm that referenced this pull request Oct 8, 2024
Summary:
X-link: facebookincubator/zstrong#1007

X-link: facebook/proxygen#518

facebook/proxygen#510

git patches on Windows OSS proxygen builds been failing for a while because of new lines Windows vs Unix.
I re-generated the patches on Windows and that seems to fix it, and it also works for linux builds so seems fine.

Reviewed By: afrind, bigfootjon

Differential Revision: D63148510

fbshipit-source-id: 0ee605896820784335c1acd2a3e22b3bdf8394f1
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