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.

Sign up for GitHub

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

Jump to bottom

dnsdist no longer supports ^R? #3227

Closed
zeha opened this issue Jan 13, 2016 · 7 comments
Closed

dnsdist no longer supports ^R? #3227

zeha opened this issue Jan 13, 2016 · 7 comments
Assignees
@zeha
Labels
dnsdist docs
Milestone
dnsdist-1.0.0

Comments

@zeha
Copy link
Collaborator

zeha commented Jan 13, 2016

As noted by @ahupowerdns dnsdist with libedit no longer does ^R history search.

Check/fix.
@zeha zeha self-assigned this Jan 13, 2016
@rgacogne
Copy link
Member

FWIW, it works on Arch with libedit 20150325_3.1.

@zeha
Copy link
Collaborator Author

zeha commented Jan 15, 2016

Can confirm that it works on Debian sid with libedit2 3.1-20150325-1+b1.

@ahupowerdns where did you notice ^R support gone missing?

@gryphius
Copy link
Contributor

it stopped working on centos6. I tried before updating and now after the update ctrl-r doesn't work anymore.

---> Package dnsdist.x86_64 0:0.0.598g5a3409c-1pdns.el6 will be updated
---> Package dnsdist.x86_64 0:0.0.629g439b8c5-1pdns.el6 will be an update

@rgacogne
Copy link
Member

@gryphius Do you happen to know what the libedit version on CentOS 6 is?

@gryphius
Copy link
Contributor

@rgacogne libedit-2.11-4.20080712cvs.1

@gryphius
Copy link
Contributor

adding the following line to ~/.editrc fixes ctrl-r for me:

bind "^R" em-inc-search-prev

workaround suggested by @rgacogne in IRC

@zeha zeha removed the defect label Jan 15, 2016
@zeha zeha added the docs label Feb 6, 2016
@zeha
Copy link
Collaborator Author

zeha commented Feb 6, 2016

Maybe the workaround for centos6 should just go into the documentation?

@rgacogne rgacogne added this to the dnsdist-1.0.0 milestone Apr 11, 2016
rgacogne added a commit to rgacogne/pdns that referenced this issue Apr 18, 2016
@rgacogne
dnsdist: Document libedit Ctrl-R workaround for CentOS 6
6d69b62 
Fixes PowerDNS#3227.
@rgacogne rgacogne mentioned this issue Apr 18, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zeha zeha

Labels
dnsdist docs
Projects
None yet
Milestone
dnsdist-1.0.0
Development

No branches or pull requests
3 participants
@zeha @gryphius @rgacogne