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

Installation instructions are not POSIX compliant #187

Open
jarkkojs opened this issue Jul 12, 2024 · 1 comment
Open

Installation instructions are not POSIX compliant #187

jarkkojs opened this issue Jul 12, 2024 · 1 comment

Comments

@jarkkojs
Copy link

Describe the bug

Installation instructions are not POSIX compliant, which could be useful e.g. when installing zsh starting from busybox.

Steps to reproduce

$ bash --posix -c "zsh <(curl -s https://raw.githubusercontent.com/zap-zsh/zap/master/install.zsh) --branch release-v1 --keep"
bash: -c: line 0: syntax error near unexpected token `('
bash: -c: line 0: `zsh <(curl -s https://raw.githubusercontent.com/zap-zsh/zap/master/install.zsh) --branch release-v1 --keep'

Expected behavior

POSIX compliant alternative:

bash --posix -c "curl -sSf https://raw.githubusercontent.com/zap-zsh/zap/master/install.zsh | zsh -s -- --branch release-v1 --keep"
⚡ Zap - Installer

Screenshots and recordings

No response

OS / Linux distribution

Fedora Linux, macOS

Zsh version

5.9

Zap version

1.2.3

Terminal emulator

Alacritty

If using WSL on Windows, which version of WSL

None

Additional context

No response

@jarkkojs
Copy link
Author

It is also somewhat idiomatic pattern among open source projects (e.g. https://www.rust-lang.org/tools/install).

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

No branches or pull requests

1 participant