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

Delete history item > choose history item > history -r execution in wild #386

Closed
dvorka opened this issue Jan 9, 2020 · 5 comments
Closed

Comments

@dvorka
Copy link
Owner

dvorka commented Jan 9, 2020

Repro:

  • start HSTR
  • delete any history item
  • choose any history item using Enter

... and after selected item is executed also history -r. See also #348.

@natehouk
Copy link

I can verify this bug as well, but I am confused, because on my machine history -r does nothing.

Can someone please explain which version of history has the option of -r?

I see a man page that describes -r here: https://ss64.com/bash/history.html

However, I do not know which version that is or where to get that version of history?

When I run man history on my Debian machine, I receive the following man page which has NO reference of -r: http://man7.org/linux/man-pages/man3/history.3.html

It appears the version of history I have is GNU History 6.3. How do I get the version of history which has history -r? Is that installed on other distributions of Linux?

If I type which history I receive no output as well which is also strange.

This is on Debian 9/stretch.

@selurvedu
Copy link

I tried to build the dev/2.4.0 branch (commit 7d984e4), but the problem remains.

$ "$SHELL" --version
GNU bash, version 5.1.8(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
$ hstr --version
hstr version "2.4.0" (2021-06-25T08:30:00)

@dvorka dvorka closed this as completed in 7d984e4 Dec 8, 2021
@dvorka dvorka unpinned this issue Dec 10, 2021
@selurvedu
Copy link

selurvedu commented Apr 20, 2022

@dvorka I just tested this and the problem is still present in hstr 2.5.

selurvedu added a commit to selurvedu/hstr that referenced this issue Jul 23, 2022
@selurvedu
Copy link

Bump. I made #457 to fix this.

@christophschw
Copy link

Bump. I made #457 to fix this.

Thanks, just locally merged this into master and will test :)

selurvedu added a commit to selurvedu/hstr that referenced this issue Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants