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

First letter of command remains when iterating through history #553

Closed
louy2 opened this issue Jul 14, 2015 · 18 comments
Closed

First letter of command remains when iterating through history #553

louy2 opened this issue Jul 14, 2015 · 18 comments
Labels

Comments

@louy2
Copy link

louy2 commented Jul 14, 2015

The first letter of the first command longer than a threshold visited when iterating through history remains displayed, pushes subsequent iterations off one char, but does not affect the actual commands iterated.

Examples assume a clear prompt at the beginning.

Example:

  1. First type npm
  2. Use to iterate and see:
npm i webpack-dev-server --save-d
  1. Use to iterate and see:
npm irun build

(should be npm run build)
4. Use to iterate and see:

nwebpack

(shoulde be webpack)
5. Use to iterate and see:

nnpm run build

(should be npm run build)
6. Use to get back to clear prompt and see:

n

(should be empty)

Strangely enough, the misdisplay doesn't affect the execution of the commands. The actual executed commands ignore the added letter. Short commands in the history such as ls and npm doesn't trigger this artifact.

@doggy8088
Copy link

I met the same problem also.

@louy2
Copy link
Author

louy2 commented Jul 16, 2015

Went to test ConEmu, v150707 has no such issue. v140707 is used here, but no similar issue reports or related fix from 140707 onward discovered. Probably introduced by Cmder?

@zhaiyusci
Copy link

I've tested it more detailed and found that a command longer than 4 letters could trigger the problem. I do not know if you have exactly the same length of trigger one?

@zhfreal
Copy link

zhfreal commented Jul 23, 2015

I got the same problem and replaced the ConEmu with the latest one(150707) , but not solved

@xlijun
Copy link

xlijun commented Oct 10, 2015

This is a duplicate issue: #506
I have the same problem. When a command larger than 4 character is displayed in the command history, the first letter will remain in the line, and kept pre-pended to other commands in the history.
I guess this has something to do with the system locale? I am using a windows 7 chinese version.

@Stanzilla
Copy link
Member

@Maximus5 ConEmu issue or clink issue?

@Maximus5
Copy link
Contributor

ConEmu do not handle command line history. So that would be a clink issue, most probably.
Just get latest clink sources and build it to check if your binaries are obsolete.

Of course, you should check latest ConEmu builds.
There is no sense to report issues about OLD BUILDS!

@Stanzilla
Copy link
Member

Yup indeed, guys, go test with https://github.com/Stanzilla/cmder/releases/tag/1.2.6 please.

@Stanzilla
Copy link
Member

Just found #264, can you guys try this workaround, please?

@MartiUK
Copy link
Member

MartiUK commented Oct 13, 2015

@Stanzilla Would adding a space after the lambda solve this issue?

@Stanzilla
Copy link
Member

Not sure, that would be 3 bits then, no? And I guess the problem is that it's one symbol but two bytes and therefore clink leaves one up.

@Stanzilla
Copy link
Member

paging @mridgers for help :)

@mridgers
Copy link

It is quite possible that Clink is at fault here - I am suspicious that the lambda character is causing Readline to misbehave because it is a multi-byte character. Does the issue go away if the lambda is replaced with something ASCII-compatible like a $?

@Stanzilla
Copy link
Member

Alright so just as I thought, @louy2 @doggy8088 can you test that for us? You'd have to replace the lambda character in cmder\config\cmder.lua

@suimong
Copy link

suimong commented Oct 24, 2015

@Stanzilla The problem go away when I replace the lambda character with >, >> or other ASCII characters.

@Stanzilla
Copy link
Member

Thank you @suimong for confirming this.

@mridgers any way you can fix this or do we have to change away from the lambda?

@mridgers
Copy link

I'll look into the issue and find a fix. There'is no reason at all why you should have to change from the lambda.

@Jackbennett
Copy link
Contributor

Focus testing on #506

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

No branches or pull requests