Closed Bug 1419167 Opened 7 years ago Closed 7 years ago

Remove now unnecessary ssh/plink installation step for Vagrant

Categories

(Tree Management :: Treeherder, enhancement, P1)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

21:05 <Sam_Kelly> Hey guys, I'm running windows 10 and setting up vagrant for the first time. Following this: https://treeherder.readthedocs.io/installation.html and got up to "Windows only: Ensure MSYS ssh (ships with Git for Windows) is on the PATH, so Vagrant can find it". Not sure how to go about that? 21:14 <•emorley> I've been looking at the official Vagrant docs and I think perhaps it now bundles an SSH client itself so that step might not be necessary any more. Could you try without it? If that works, I'll update the docs 21:17 <•emorley> I just tried using Windows cmd.exe (I normally use MSYS2 inside ConEmu) and it seemed to work fine 21:17 <Sam_Kelly> Seems to be working, I've got a % complete thing running now Looks like recent Vagrant now bundles its own SSH client: https://github.com/hashicorp/vagrant-installers/pull/107 So we can remove this step: https://treeherder.readthedocs.io/installation.html#prerequisites
Commit pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/626edb59ad6f6e8f745a2a375214f2bd24f0b55b Bug 1419167 - Vagrant: Remove unnecessary MSYS SSH pre-requisite (#2966) Since recent Vagrant now bundles its own SSH client.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: Treeherder: Docs & Development → TreeHerder
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: