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

Use bourne shell for rustup.sh #360

Closed
wants to merge 1 commit into from
Closed

Conversation

zzmp
Copy link

@zzmp zzmp commented Aug 9, 2014

Nothing's really broken, but rustup.sh does start with #!/bin/sh.

Nothing's really broken, but rustup.sh does start with `#!/bin/sh`.
@alexcrichton
Copy link
Member

PRs for the website should be made against https://github.com/wycats/cargo-website instead of this repo (these are just generated docs). I also believe that we explicitly use sh as it's likely to be practically everywhere and it's easier to type. Regardless, I would bring the issue up on cargo-website and likely on the official rust website as well as we'd like to keep the two in sync.

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