Skip to content

Commit ca533d5

Browse files
Copilotjosecelano
andcommitted
fix: remove sudo from dependency installer invocation
The installer binary handles sudo internally for dependencies that require it (OpenTofu, Ansible, LXD). Running the entire binary with sudo causes cargo-machete installation to fail because cargo is not in sudo's PATH. cargo-machete is installed via 'cargo install' and doesn't need sudo. Co-authored-by: josecelano <[email protected]>
1 parent fdae848 commit ca533d5

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/copilot-setup-steps.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,8 @@ jobs:
4040
- name: Install all development dependencies
4141
run: |
4242
# Use the binary we just built to install all dependencies
43-
sudo target/release/dependency-installer install
43+
# Note: The binary handles sudo internally for dependencies that need it
44+
target/release/dependency-installer install
4445
env:
4546
# Ensure non-interactive installation
4647
DEBIAN_FRONTEND: noninteractive

0 commit comments

Comments
 (0)