Skip to content

fix(fbcode_builder): Fixes to fbcode_builder needed by sapling/edenfs build#1126

Closed
ben-- wants to merge 3 commits intofacebook:mainfrom
ben--:builder-fixes
Closed

fix(fbcode_builder): Fixes to fbcode_builder needed by sapling/edenfs build#1126
ben-- wants to merge 3 commits intofacebook:mainfrom
ben--:builder-fixes

Conversation

@ben--
Copy link
Contributor

@ben-- ben-- commented Sep 22, 2025

Three independent changes:

  1. Fix merge damage in FBPythonBinary.cmake
  2. Define fbcode_build=False in rust builds to eliminate errors/warnings
  3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):

act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml

@meta-cla meta-cla bot added the CLA Signed label Sep 22, 2025
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D83007559. (Because this pull request was imported automatically, there will not be any future comments.)

@ben-- ben-- changed the title Builder fixes fix(fbcode_builder): Fixes to fbcode_builder needed by sapling/edenfs build Sep 22, 2025
meta-codesync bot pushed a commit to facebook/fboss that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/proxygen that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/mvfst that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/fbthrift that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebookexperimental/rust-shed that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebookexperimental/moxygen that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebookincubator/hsthrift that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/watchman that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebookincubator/fizz that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
@meta-codesync meta-codesync bot closed this in e0e866c Oct 13, 2025
@meta-codesync
Copy link

meta-codesync bot commented Oct 13, 2025

This pull request has been merged in e0e866c.

meta-codesync bot pushed a commit to facebook/hhvm that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/folly that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/fb303 that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebookexperimental/edencommon that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/openr that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/CacheLib that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebook/wangle that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebookincubator/cinderx that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

fbshipit-source-id: e44a3f8eed4cf032c3ad33b8dc306eb589901842
meta-codesync bot pushed a commit to facebookincubator/katran that referenced this pull request Oct 13, 2025
Summary:
Three independent changes:

1. Fix merge damage in FBPythonBinary.cmake
2. Define `fbcode_build=False` in rust builds to eliminate errors/warnings
3. Add a missing dependency to zstd

Tested by building sapling using the GitHub action (won't run fully until other PRs land):
```
act --container-architecture linux/x86_64 -W .github/workflows/edenfs_linux.yml
```

X-link: facebook/sapling#1126

Differential Revision: D83007559

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