Skip to content

docs: clarify dependency on OpenSSH#329

Open
federicobozzini wants to merge 1 commit into
mainfrom
open-ssh-dep
Open

docs: clarify dependency on OpenSSH#329
federicobozzini wants to merge 1 commit into
mainfrom
open-ssh-dep

Conversation

@federicobozzini

Copy link
Copy Markdown
Contributor

Changes

  • topo actually depends on OpenSSH, not any generic ssh implementation. Let's make this explicit.

Signed-off-by: Federico Bozzini <federico.bozzini@arm.com>
@federicobozzini federicobozzini requested a review from a team as a code owner July 10, 2026 11:32
@federicobozzini federicobozzini changed the title docs: clarify dependnecy on SSH docs: clarify dependency on OpenSSH Jul 10, 2026
@th3james

Copy link
Copy Markdown
Contributor

This is a bit misleading in its current form. It suggests that we've validated that OpenSSH is installed, but we've actually only validated that ssh is on the PATH. If we're doing this, we should probably test for OpenSSH, not just SSH

@federicobozzini

Copy link
Copy Markdown
Contributor Author

This is a bit misleading in its current form. It suggests that we've validated that OpenSSH is installed, but we've actually only validated that ssh is on the PATH. If we're doing this, we should probably test for OpenSSH, not just SSH

I agree but this would misleading in the same way saying docker is OK is misleading when its version is not supported. At the moment we only check for binaries and ssh is the right binary for OpenSSH. We can replace the check with something better but I would treat it as a separate issue.

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.

2 participants