Skip to content

ci: Run both tests pybind11 and nanobind in CI#5176

Merged
lgritz merged 1 commit intoAcademySoftwareFoundation:mainfrom
soswow:nanobind-ci-integration
May 5, 2026
Merged

ci: Run both tests pybind11 and nanobind in CI#5176
lgritz merged 1 commit intoAcademySoftwareFoundation:mainfrom
soswow:nanobind-ci-integration

Conversation

@soswow
Copy link
Copy Markdown
Contributor

@soswow soswow commented Apr 30, 2026

Follow up to #5084

Here we make at last one OS to run both tests

Assisted-by: Cursor/Composer-2

Comment thread src/build-scripts/gh-installdeps.bash Fixed
Comment thread src/build-scripts/install_homebrew_deps.bash Fixed
@soswow
Copy link
Copy Markdown
Contributor Author

soswow commented Apr 30, 2026

works

image

@soswow soswow force-pushed the nanobind-ci-integration branch 4 times, most recently from 197862e to 7733874 Compare May 1, 2026 13:12
- Add workflow input and env for OIIO_PYTHON_BINDINGS_BACKEND; run with
  backend=both on selected Linux, macOS, and Windows matrix rows.
- Pass the flag through ci-build.bash into CMake.
- Install pinned nanobind for CMake discovery (pip --require-hashes on Linux
  and Windows; Homebrew on macOS).
- Point nanobind Python tests at lib/python/nanobind (not per-$<CONFIG>).
- Extend the nanobind package __path__ on Windows for MSVC config output dirs.

Assisted-by: Cursor/Composer-2

Signed-off-by: Aleksandr Motsjonov <soswow@gmail.com>
@soswow soswow force-pushed the nanobind-ci-integration branch from 7733874 to dbc55f5 Compare May 1, 2026 13:13
@soswow soswow marked this pull request as ready for review May 2, 2026 06:26
Copy link
Copy Markdown
Collaborator

@lgritz lgritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lgritz lgritz changed the title Run both tests pybind11 and nanobind in CI ci: Run both tests pybind11 and nanobind in CI May 5, 2026
@lgritz lgritz merged commit 6ea45b1 into AcademySoftwareFoundation:main May 5, 2026
65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants