Skip to content

chore: update python buildpack (enables uv projects)#226

Open
rokroskar wants to merge 3 commits into
mainfrom
000-update-python-buildpack
Open

chore: update python buildpack (enables uv projects)#226
rokroskar wants to merge 3 commits into
mainfrom
000-update-python-buildpack

Conversation

@rokroskar
Copy link
Copy Markdown
Member

Updates the python buildpack which adds support for uv python projects.

Copy link
Copy Markdown
Collaborator

@SalimKayal SalimKayal left a comment

Choose a reason for hiding this comment

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

builders/cuda-selector should be updated to mirror those changes

@rokroskar
Copy link
Copy Markdown
Member Author

@SalimKayal agreed! However, the builds aren't working yet...

@rokroskar
Copy link
Copy Markdown
Member Author

@SalimKayal I think I fixed it - could you please test?

@SalimKayal
Copy link
Copy Markdown
Collaborator

sure let me check

@SalimKayal
Copy link
Copy Markdown
Collaborator

python-dependency-manager is not participating currently

@SalimKayal
Copy link
Copy Markdown
Collaborator

So it's not fixed yet.

Other question. do we want UV to be usable within a session ? if yes cp -ras "$original_cache_dir" "$RENKU_WORKING_DIR/.uv_cache" and export UV_CACHE_DIR="$RENKU_WORKING_DIR/.uv_cache can be a nice way of enabling this. We would need uv in the image though.

Comment thread builders/selector/builder.toml
@rokroskar rokroskar marked this pull request as ready for review May 19, 2026 12:39
@SalimKayal SalimKayal self-requested a review May 19, 2026 17:06
Copy link
Copy Markdown
Collaborator

@SalimKayal SalimKayal left a comment

Choose a reason for hiding this comment

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

LGTM

@rokroskar
Copy link
Copy Markdown
Member Author

I can't merge because of the failing test for brew. Should I see if there is an easy-ish fix/reason?

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.

2 participants