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

Update template.sh - bugfix for cmd #761

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tschettervictor
Copy link
Contributor

This fixes a bug where templated "cmd" would not working when doing some commands like fetch.

@bmac2
Copy link
Collaborator

bmac2 commented Dec 16, 2024

@tschettervictor what was the command syntax that was failing before that now works? What can I run to see it broken before your change?

@tschettervictor
Copy link
Contributor Author

tschettervictor commented Dec 16, 2024

It was a fetch command.

CMD fetch -o /tmp/dasherr-install.sh https://raw.githubusercontent.com/tschettervictor/bsd-apps/main/dasherr/dasherr-install.sh

This PR simply puts the args inside literal quotes instead of single quotes.

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