Skip to content
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

languages: Add ssh_config.d/*.conf as a glob for sshclientconfig #11947

Conversation

edmonds
Copy link
Contributor

@edmonds edmonds commented Oct 27, 2024

The ssh_config format supports a config file fragment inclusion mechanism by using the Include directive. Some Linux distros such as Debian, Ubuntu, and distros derived from Debian/Ubuntu specify Include /etc/ssh/ssh_config.d/*.conf in their default /etc/ssh/ssh_config config file [0]. Similarly, ssh users can also use this directive in their ~/.ssh/config file, for instance specifying Include ssh_config.d/*.conf to include all files matching the glob ~/.ssh/ssh_config.d/*.conf into their ssh client configuration.

This commit adds the glob ssh_config.d/*.conf to the sshclientconfig language so that these config file fragments will be detected.

[0] https://salsa.debian.org/ssh-team/openssh/-/blob/e9f52d3c18a4bb5e2055bcec0cf08abfb137ea2c/debian/patches/debian-config.patch#L126

The ssh_config format supports a config file fragment inclusion
mechanism by using the `Include` directive. Some Linux distros such
as Debian, Ubuntu, and distros derived from Debian/Ubuntu specify
`Include /etc/ssh/ssh_config.d/*.conf` in their default
/etc/ssh/ssh_config config file [0]. Similarly, ssh users can also use
this directive in their ~/.ssh/config file, for instance specifying
`Include ssh_config.d/*.conf` to include all files matching the glob
`~/.ssh/ssh_config.d/*.conf` into their ssh client configuration.

This commit adds the glob `ssh_config.d/*.conf` to the sshclientconfig
language so that these config file fragments will be detected.

[0] https://salsa.debian.org/ssh-team/openssh/-/blob/e9f52d3c18a4bb5e2055bcec0cf08abfb137ea2c/debian/patches/debian-config.patch#L126
@the-mikedavis the-mikedavis added the A-language-support Area: Support for programming/text languages label Oct 30, 2024
@the-mikedavis the-mikedavis merged commit 83f1b98 into helix-editor:master Nov 20, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-language-support Area: Support for programming/text languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants