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)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
Details |
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
Comment 1•7 years ago
|
||
Comment 2•7 years ago
|
||
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.
Assignee | ||
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Component: Treeherder: Docs & Development → TreeHerder
You need to log in
before you can comment on or make changes to this bug.
Description
•