Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: pass command line arguments verbatim #1582

Merged
merged 2 commits into from
Jul 8, 2024

Conversation

baszalmstra
Copy link
Contributor

Fixes #1492

Passes additional command line arguments verbatim via pixi run to deno task shell.

> pixi run python -c 'import sys; print(sys.argv)' '*foo*'
['-c', '*foo*']

But only if there is more than one argument, so if a user quotes the entire command it is still interpreted in its entirety by deno task shell:

> pixi run "python -c 'import sys; print(sys.argv)' '*foo*'"
glob: no matches found '.../*foo*'

Copy link
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

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

Nice!

@ruben-arts ruben-arts merged commit 8e07f3e into prefix-dev:main Jul 8, 2024
24 checks passed
renan-r-santos added a commit to renan-r-santos/pixi-kernel that referenced this pull request Jul 25, 2024
renan-r-santos added a commit to renan-r-santos/pixi-kernel that referenced this pull request Jul 25, 2024
renan-r-santos added a commit to renan-r-santos/pixi-kernel that referenced this pull request Jul 25, 2024
* fix: update R kernel logic for Pixi >= 0.26.0
prefix-dev/pixi#1582

* chore: bump minimum Pixi version

* chore: bump package version
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.

Quoting not passed to subcommand
2 participants