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

Add support for double-quoted strings in wrapper.sh #20

Open
RainbowRivey opened this issue Mar 3, 2025 · 0 comments
Open

Add support for double-quoted strings in wrapper.sh #20

RainbowRivey opened this issue Mar 3, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@RainbowRivey
Copy link

Currently all double-quotes disappear when using wrapper script.
This is important for passing complex arguments (e.g. lists to Hydra configs)

For example:

pegasus-bridle/wrapper.sh echo "test"

Outputs

[...]
execute: echo test
test

Output with new feature should be:

[...]
execute: echo "test"
test

Note
There IS actually a workaround right now, putting escaped quotes will do the thing, but its quite inconvenient:

pegasus-bridle/wrapper.sh echo "\"test\""
@RainbowRivey RainbowRivey added the enhancement New feature or request label Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant