Skip to content

Update install.sh for release 2026.4.2#38

Open
hecspc wants to merge 1 commit into
mainfrom
update-install-scripts-2026.4.2
Open

Update install.sh for release 2026.4.2#38
hecspc wants to merge 1 commit into
mainfrom
update-install-scripts-2026.4.2

Conversation

@hecspc
Copy link
Copy Markdown
Contributor

@hecspc hecspc commented Apr 2, 2026

Automated update of install.sh for release 2026.4.2

This PR updates the installer script to sync with the latest version from the main repository.

Changes include:

  • Sync installer script with latest version
  • Automated update from release workflow

  - Sync installer script with latest version
  - Automated update from release workflow
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1e673536b4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread install.sh
Comment on lines +121 to +124
if [ -w "$INSTALL_DIR" ]; then
cp "${TEMP_DIR}/vvctl" "${INSTALL_DIR}/vvctl"
else
sudo cp "${TEMP_DIR}/vvctl" "${INSTALL_DIR}/vvctl"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Create INSTALL_DIR before copying binary

The install step now copies directly into INSTALL_DIR without ensuring the directory exists, so installs fail with No such file or directory when users set INSTALL_DIR to a new path (or on minimal systems where /usr/local/bin is absent). In those cases both branches fail (cp and sudo cp), which is a regression from the previous behavior that created the directory first.

Useful? React with 👍 / 👎.

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.

1 participant