Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Find ssh commands via locate #1983

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

uenz
Copy link
Contributor

@uenz uenz commented Oct 31, 2021

SparkleShare uses several comandline tools to provide functionality. While working on the windows version i saw that ssh, ssh-keygen and ssh-keyscan was hard coded and sparkleshare expects that these are in the path. This is not always given.
I extended the SSHCommand.cs to da the location of all ssh related commands. The additional calls in BaseController.cs checks for existance of the shs commands. As ssh-keyscan and ssh-keygen does not provide a version info they are called to check if they are there.
The enquating is changed from ' to ", because this is the platform independent way.

@hbons hbons self-requested a review October 31, 2021 16:26
@uenz uenz force-pushed the feature/LocateSSHCommands branch from 2004997 to 210ac0f Compare November 3, 2022 18:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant