Skip to content

Fix: prefix install command with sudo#17

Open
mdocevski-infinum wants to merge 1 commit into
masterfrom
fix/sudo-install-command
Open

Fix: prefix install command with sudo#17
mdocevski-infinum wants to merge 1 commit into
masterfrom
fix/sudo-install-command

Conversation

@mdocevski-infinum
Copy link
Copy Markdown

Summary

The installation command in the README is missing sudo, which causes it to fail with permission errors when trying to write files to /usr/local/bin/.

Problem

Without sudo, the script fails because it lacks the necessary permissions to create files in /usr/local/bin/:

~  /bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/infinum/app-deploy-script/master/install.sh)"
  ==> This script will install:
  /usr/local/bin/app-deploy

  Do you want to proceed? [y/n] y

  Fetching script data...
  Installing...
  /bin/bash: line 44: /usr/local/bin/app-deploy: Permission denied
  mkdir: /usr/local/bin/.app-deploy-sources/: Permission denied
  cp: /usr/local/bin/.app-deploy-sources: Permission denied
  chmod: /usr/local/bin/app-deploy: No such file or directory
  chmod: /usr/local/bin/.app-deploy-sources/: No such file or directory
  Done!

Fix

Prefix the install command with sudo so it runs with the required elevated permissions:

sudo /bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/infinum/app-deploy-script/master/install.sh)"

Note: The README already mentions that sudo is needed ("Script commands such as install, and --update will need sudo to execute successfully."), but the example command itself did not reflect this.

🤖 Generated with Claude Code

The installation command requires sudo to write files to /usr/local/bin/.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant