Skip to content

Commit

Permalink
wip: showcase working uv plugin fix
Browse files Browse the repository at this point in the history
  • Loading branch information
bepri committed Feb 4, 2025
1 parent dc39bba commit 8ae8b19
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 14 deletions.
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ dependencies = [
"craft-application",
"craft-archives",
"craft-cli",
"craft-parts",
"craft-parts @ git+https://github.com/canonical/craft-parts@work/uv-plugin-system-interpreter/CRAFT-4010",
"craft-platforms",
"craft-providers",
"overrides",
Expand Down
8 changes: 2 additions & 6 deletions rockcraft/plugins/uv_plugin.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,8 @@ def _should_remove_symlinks(self) -> bool:

@override
def _get_system_python_interpreter(self) -> str | None:
"""Overridden because Python must always be provided by the parts.
The uv plugin requires a name to reference Python, so we must depend
on a relative python3 being installed. Should return None once
https://github.com/canonical/craft-parts/issues/991 is closed."""
return "python3"
"""Overridden because Python must always be provided by the parts."""
return None

@override
def _get_script_interpreter(self) -> str:
Expand Down
14 changes: 7 additions & 7 deletions uv.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 8ae8b19

Please sign in to comment.