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

Clink ignores the standard readline init file (~/.inputrc) #258

Closed
pahindman opened this issue Oct 7, 2014 · 3 comments
Closed

Clink ignores the standard readline init file (~/.inputrc) #258

pahindman opened this issue Oct 7, 2014 · 3 comments

Comments

@pahindman
Copy link

Clink can only be configured by either editing clink_inputrc_base in the clink installation dir, or clink_inputrc in the clink profile dir. It would be nice if clink could use the same readline configuration that other programs use.

@mridgers
Copy link
Owner

This is a little tricky as Clink's use of Readline is not necessarily compatible with other software that also uses Readline (escape sequences for special keys for example). Hence the "clink_" prefix. Not that the current way it currently works is particularly ideal.

@rivy
Copy link

rivy commented Oct 26, 2015

Actually, IMO, it's not particularly ideal; using the default ".inputrc" file would be better and more compatible with other platforms.

As far as incompatibilities, the inputrc $if directive was specifically designed help with application variations. When using gnu readline, setting rl_readline_name sets the application name which can be used in the inputrc to customize behavior.

For example, set rl_readline_name to "CMD" (for example) and then the ".inputrc" file can be customized as needed by using:

... <common settings>
$if CMD
... <CMD settings>
$endif
... <more common settings>

Then a single inputrc file can be constructed which can be used for all shells.

@rivy
Copy link

rivy commented Dec 20, 2015

👍

Stanzilla pushed a commit to Stanzilla/clink that referenced this issue Dec 21, 2017
chrisant996 added a commit to chrisant996/clink that referenced this issue Aug 25, 2021
The fix for mridgers/clink#258 accidentally
broke that, and also broke the `clink-reload` command and made it reload
only the last one of all the inputrc files that had been loaded.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants