Skip to content
This repository has been archived by the owner on May 15, 2020. It is now read-only.

History search semi-broken #19

Open
pikeas opened this issue Jan 23, 2017 · 6 comments
Open

History search semi-broken #19

pikeas opened this issue Jan 23, 2017 · 6 comments

Comments

@pikeas
Copy link

pikeas commented Jan 23, 2017

Start with a simple command:

$ ls -a
...
$ ls -l
....

Now hit ctrl-r and type ls. The most recent result will populate. Hit ctrl-r again.
Expected: ls -a.
Actual: failing bck-i-search: ls.

Doing a search by typing ls and then up/down still works as expected.

Recently upgraded to your fork, what's the recommended way to debug this?

@facastagnini
Copy link

Hey @pikeas thanks for your report.
I will be afk for a few weeks, I will take a look at this as soon I am able.

@pikeas
Copy link
Author

pikeas commented Jan 23, 2017

I found zsh-users/zsh-syntax-highlighting#407, which indicates an upstream problem in zsh itself!

I've compiled zsh from their master branch and ctrl-r appears to work as expected.

Is this worth a note on the history-search module's README?

@paulmelnikow
Copy link

Good find @pikeas! Would you mind opening a PR for that?

@facastagnini
Copy link

@pikeas can you pot a reference to the commit it od the head of zsh when you compiled and worked? I want to identify the version that includes that fix in zsh.

@pikeas
Copy link
Author

pikeas commented Mar 6, 2017

@facastagnini I compiled zsh-users/zsh@64c6758, which was HEAD at the time I figured this out.

@johnpneumann
Copy link

I have 5.2 and do not have the issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants