Skip to content

Fix participant installer command path - #5

Merged
bestquark merged 2 commits into
mainfrom
agent/fix-installer-sbin-path
Aug 12, 2026
Merged

Fix participant installer command path#5
bestquark merged 2 commits into
mainfrom
agent/fix-installer-sbin-path

Conversation

@bestquark

@bestquark bestquark commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • include Debian system-administration directories in the participant installer PATH
  • support both generic spi and Radxa vendor spidev device groups
  • add regression tests for terminal PATH and service permissions

Root causes

  • the workshop desktop terminal PATH omits /usr/sbin, so modinfo was unavailable despite kmod being installed
  • Radxa assigns /dev/spidev3.0 to group spidev, while the service originally received only group spi

Verification

  • PYTHONPATH=src .venv/bin/python -m unittest discover -s tests -v (20 tests)
  • reproduced and diagnosed on the admin ZERO 3W

@bestquark
bestquark marked this pull request as ready for review August 12, 2026 14:42
@bestquark
bestquark merged commit 4eb4a4f into main Aug 12, 2026
4 checks passed
@bestquark
bestquark deleted the agent/fix-installer-sbin-path branch August 12, 2026 14:43
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