Commit 23785d2
fix: remove backend-path to resolve ValueError: paths must be inside … (#231)
* fix: remove backend-path to resolve ValueError: paths must be inside source tree
python -m build fails in CI with:
ValueError: paths must be inside source tree
This is thrown by pyproject_hooks.BuildBackendHookCaller.__init__ which
validates that all backend-path entries resolve inside the package source
directory. The backend-path = ["../../versioning/helper"] in each package's
pyproject.toml points outside the package tree, triggering this check before
any build code runs. Setting PYTHONPATH has no effect since the error fires
during path validation, not import.
Fix: remove backend-path from all 14 packages. The CI pipeline already exports
PYTHONPATH="${ROOT_DIR}/versioning/helper" which makes build_backend importable
in the hook subprocess. The custom backend and its S360 version constraint
injection are unaffected. uv build from the repo root continues to work for
local development.
* Set PYTHONPATH for install step in CI workflow
Added PYTHONPATH environment variable pointing to versioning/helper during the "Install the project" step in the CI workflow. This ensures Python can locate necessary modules during installation.
* fix: set PYTHONPATH at job level so all steps can import build_backend
Step-level env only applies to a single step. uv sync passed but uv build
and tox steps still failed with ModuleNotFoundError: No module named build_backend.
Moving PYTHONPATH to job-level env ensures it is available for all steps:
uv sync, uv build, tox, and any future steps that invoke the build backend.
Also fix docstring example in build_backend.py to include packaging in
build-system.requires, matching what the packages actually declare.
---------
Co-authored-by: Grant Harris <96964444+gwharris7@users.noreply.github.com>1 parent dc936b1 commit 23785d2
16 files changed
Lines changed: 12 additions & 19 deletions
File tree
- .github/workflows
- libraries
- microsoft-agents-a365-notifications
- microsoft-agents-a365-observability-core
- microsoft-agents-a365-observability-extensions-agentframework
- microsoft-agents-a365-observability-extensions-langchain
- microsoft-agents-a365-observability-extensions-openai
- microsoft-agents-a365-observability-extensions-semantickernel
- microsoft-agents-a365-observability-hosting
- microsoft-agents-a365-runtime
- microsoft-agents-a365-tooling-extensions-agentframework
- microsoft-agents-a365-tooling-extensions-azureaifoundry
- microsoft-agents-a365-tooling-extensions-googleadk
- microsoft-agents-a365-tooling-extensions-openai
- microsoft-agents-a365-tooling-extensions-semantickernel
- microsoft-agents-a365-tooling
- versioning/helper
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
85 | 85 | | |
86 | 86 | | |
87 | 87 | | |
| 88 | + | |
| 89 | + | |
88 | 90 | | |
89 | 91 | | |
90 | 92 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
| |||
0 commit comments