Skip to content

Fix failing GitHub workflows#38

Closed
kvelakur wants to merge 1 commit into
mainfrom
export-D110124796
Closed

Fix failing GitHub workflows#38
kvelakur wants to merge 1 commit into
mainfrom
export-D110124796

Conversation

@kvelakur

@kvelakur kvelakur commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary:
Pull Request: #38

Fix linux-sdk packaging and macOS wheel CI failures

Two CMake fixes:

  • deb/rpm packaging: add a PACKAGING_TEST option and test_solve target so build_linux_sdk.sh (passing -DPACKAGING_TEST=ON) builds and installs the smoke-test binary to bin/.
  • macOS wheels: set INSTALL_RPATH_USE_LINK_PATH plus loader_path/_lib on the SKBUILD targets so delocate can resolve and bundle librebalancer.dylib and the getdeps deps (libfolly, ...). Without baked-in rpaths it failed with DelocationError: Could not find all dependencies.

Differential Revision: D110124796

@meta-cla meta-cla Bot added the cla signed label Jun 30, 2026
@meta-codesync

meta-codesync Bot commented Jun 30, 2026

Copy link
Copy Markdown

@kvelakur has exported this pull request. If you are a Meta employee, you can view the originating Diff in D110124796.

Summary:
Pull Request: #38

# Fix linux-sdk packaging and macOS wheel CI failures

Two CMake fixes:

- **deb/rpm packaging**: add a `PACKAGING_TEST` option and `test_solve` target so `build_linux_sdk.sh` (passing `-DPACKAGING_TEST=ON`) builds and installs the smoke-test binary to `bin/`. 
- **macOS wheels**: set `INSTALL_RPATH_USE_LINK_PATH` plus `loader_path/_lib` on the SKBUILD targets so `delocate` can resolve and bundle `librebalancer.dylib` and the getdeps deps (libfolly, ...). Without baked-in rpaths it failed with `DelocationError: Could not find all dependencies`.

Differential Revision: D110124796
@meta-codesync meta-codesync Bot changed the title Fix failing linux-sdk / package-amd64 GitHub workflow Fix failing GitHub workflows Jun 30, 2026
@meta-codesync meta-codesync Bot force-pushed the export-D110124796 branch from 0e4b62c to c7b43ef Compare June 30, 2026 05:09
@meta-codesync meta-codesync Bot closed this in 645eee3 Jun 30, 2026
@meta-codesync meta-codesync Bot added the Merged label Jun 30, 2026
@meta-codesync

meta-codesync Bot commented Jun 30, 2026

Copy link
Copy Markdown

This pull request has been merged in 645eee3.

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.

1 participant