generated from devcontainers/feature-starter
-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Description
The builder feature's install script contains an unused INSTALL_DEPENDENCIES variable that appears to be a copy-paste artifact from the bosh-cli script.
Current State
File: src/builder/install.sh
- Line 4:
: ${INSTALL_DEPENDENCIES:=true}- defined but never used
Analysis
The builder tool is a simple static site generation binary that only requires:
curlandca-certificatesfor downloading- Basic file operations for installation
Unlike bosh-cli which installs additional development dependencies (build-essential, ruby, etc.), the builder feature doesn't need any optional dependencies, making the INSTALL_DEPENDENCIES variable unnecessary.
Proposed Solution
Remove the unused variable from line 4:
# Remove this line:
: ${INSTALL_DEPENDENCIES:=true}Files Affected
src/builder/install.sh(line 4)
Context
This appears to be a copy-paste oversight where the variable was copied from bosh-cli/install.sh but the corresponding functionality was never implemented (and isn't needed for this feature).