-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for Include
#11
Comments
Looks like https://github.com/danieljimenez/ssh_config should have supported that, but it doesn't 🤔 Seems he misunderstood how |
Yeah, it doesn’t work like it’s supposed to. Now you bring it up, i should probably revert to the old library with no
|
@deanishe I know this is old issue. but I will take a look and see if can figure out recursive ssh configs. doesn't look like anyone else has attempted to figure it out. |
Oops. I have a working parser here. I'm afraid I just haven't bothered to tidy it up and integrate it yet. |
Looks like I never got around to that either 🙈 @deanishe would be cool if you could include that! |
Any updates on this? I saw a mention elsewhere on your Alfred-Workflows issues (awesome lib btw, thanks very much!) that you're rewriting most things in Python3 - is this Include file support getting deferred until the automation landscape settles down a bit? |
No, I'm just really busy with work atm. It's not helping that when I look at the workflow's code, I want to rewrite the whole thing… |
The
~/.ssh/config
file supports anInclude
directive to load more config files, making it easier to separate groups of hosts. For example:Hosts from these
Include
d configuration files should also be shown.The text was updated successfully, but these errors were encountered: