Add uv run command
#3074
Add uv run command
#3074
Conversation
zanieb
added
internal
| #[derive(Args)] | ||
| #[allow(clippy::struct_excessive_bools)] | ||
| pub(crate) struct RunArgs { | ||
| /// Command |
There was a problem hiding this comment.
This needs some explanation what command means and some examples, esp. compared to
usage: python [option] ... [-c cmd | -m mod | file | -] [arg] ...
There was a problem hiding this comment.
I'm going to reserve the real documentation for when it's a little more nailed down what the interface is. I think I extend these a bit in the next pull request though.
| // Spawn and wait for completion | ||
| // Standard input, output, and error streams are all inherited | ||
| // TODO(zanieb): Throw a nicer error message if the command is not found | ||
| let mut handle = process.spawn()?; |
There was a problem hiding this comment.
I could switch yeah, I thought I might need to do something while it's running but perhaps not?
| let status = handle.wait().await?; | ||
|
|
||
| // Exit based on the result of the command | ||
| // TODO(zanieb): Do we want to exit with the code of the child process? Probably. |
There was a problem hiding this comment.
This is important for scripts that check the output status of the python code (e.g. exit status 1 vs exit status 2)
There was a problem hiding this comment.
Yeah agree that's just not built into our exit system so I want to do it separately
|
Some things i learned i'm gonna dump here: You can load dlopen libpython and use the c api to load libpython, and run Again on unix, you use |
|
Thanks for the notes! Do you see a strong benefit to doing |
|
Not really, |
Adds
uv runwhich executes a command in your current virtual environment.This is a simple first milestone, lots of remaining work and behavior. The command is hidden.