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

garden: add support for custom shell commands #26

Merged
merged 4 commits into from
Feb 14, 2024
Merged

garden: add support for custom shell commands #26

merged 4 commits into from
Feb 14, 2024

Conversation

davvid
Copy link
Member

@davvid davvid commented Feb 14, 2024

garden.shell can not be configured to arbitrary interpreter commands.
We now run the value through the "shlex" parser to separate the
string into individual command-line arguments.

Shorthand shell values are still supported and special-cased.
Only when the command string expands to a command with multiple
arguments we treat the command like a custom shell and
disable the internal builtin arguments.

garden.shell can not be configured to arbitrary interpreter commands.
We now run the value through the "shlex" parser to separate the
string into individual command-line arguments.

Shorthand shell values are still supported and special-cased.
Only when the command string expands to a command with multiple
arguments we treat the command like a custom shell and disable the
internal builtin arguments.
Allow the interactive shell to be configured separately via the
"garden.interactive-shell" configuration value. This allows custom commands
to be configured in "garden.shell" that take command strings while
specifying an interactive shell command via "garden.interactive-shell".
use the current tree for "garden shell" when nothing
is specified. This makes it easier to start a shell
in the current directory.
@davvid davvid merged commit 9122a82 into main Feb 14, 2024
8 checks passed
@davvid davvid deleted the custom-shell branch February 14, 2024 04:33
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.

1 participant