Skip to content

Use python.dependency() instead of INCLUDEPY to fix mesonpy buid in PEP517#110

Open
woodtp wants to merge 1 commit intomainfrom
fix/pep517-mesonbuild
Open

Use python.dependency() instead of INCLUDEPY to fix mesonpy buid in PEP517#110
woodtp wants to merge 1 commit intomainfrom
fix/pep517-mesonbuild

Conversation

@woodtp
Copy link
Contributor

@woodtp woodtp commented Mar 7, 2026

The PyORBIT3 build can fail while using a PEP 517-compliant build front end (e.g., pixi) if the temporary build environment is placed within the source tree.

pixi add --pypi 'pyorbit @ file:///path/to/PyORBIT3'

results in

../src/meson.build:252:6: ERROR: Tried to form an absolute path to a dir in the source tree.
      You should not do that but use relative paths instead, for
      directories that are part of your project.

This PR makes a small modification to meson.build declaring Python as an explicit dependency so that the appropriate paths to Python headers/libs are resolved automatically by meson.

@woodtp woodtp requested a review from azukov March 7, 2026 16:22
@woodtp woodtp self-assigned this Mar 7, 2026
@woodtp woodtp added bug Something isn't working build labels Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant