Skip to content

fix(install): support normalized dev deb assets#1217

Open
jtoelke2 wants to merge 1 commit intomainfrom
jt/os49-install-dev-deb-lookup
Open

fix(install): support normalized dev deb assets#1217
jtoelke2 wants to merge 1 commit intomainfrom
jt/os49-install-dev-deb-lookup

Conversation

@jtoelke2
Copy link
Copy Markdown
Collaborator

@jtoelke2 jtoelke2 commented May 6, 2026

Summary

Fix install-dev.sh so the dev Debian installer can find the normalized dev release assets published by Release Dev.

Related Issue

Related to OS-49.

Changes

  • Prefer openshell-dev-<arch>.deb entries from openshell-checksums-sha256.txt.
  • Keep the previous openshell_..._<arch>.deb lookup as a fallback for compatibility.
  • Add a focused POSIX shell lookup test for normalized names, binary checksum markers, legacy fallback, preference order, and architecture mismatch.
  • Include install-dev.sh in the install-script workflow path filters and run the new lookup test there.

Testing

  • sh -n install-dev.sh
  • sh -n e2e/install/install_dev_lookup_test.sh
  • sh e2e/install/install_dev_lookup_test.sh
  • git diff --check
  • mise run pre-commit passes

mise run pre-commit was attempted locally. It did not pass due to unrelated existing workspace issues: markdownlint picked up untracked .codex-learning/*.md files, and test:rust failed in ssh::tests::launch_editor_returns_friendly_error_when_binary_missing. The targeted installer checks passed.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not applicable)

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