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

Terminal: Duplicate shell startup lines #30429

Closed
cleidigh opened this issue Jul 11, 2017 · 6 comments
Closed

Terminal: Duplicate shell startup lines #30429

cleidigh opened this issue Jul 11, 2017 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code) windows VS Code on Windows issues
Milestone

Comments

@cleidigh
Copy link
Contributor

  • VSCode Version: 1.14.0
  • OS Version: Windows_NT x64 6.1.7601

Problem:

In certain codes to sizes opening a new Windows integrated terminal
will show duplicate shell startup lines.
The problem seems associated with the code window size. For instance
A half screen docked code window with the activity and sidebar visible
is more likely to show the problem, but as the left side screenshot shows
even without the sidebar it's possible to happen sometimes.
The problem seems to also happen on the first terminal launch.

Steps to reproduce:

  • open code with no terminal
  • dock code or size to half screen
  • display sidebar - increases probability of Problem
  • open new integrated terminal
  • observe duplicates startup lines (see screenshot)

terminal duplicate lines

@vscodebot vscodebot bot added new release terminal General terminal issues that don't fall under another label labels Jul 11, 2017
@Tyriar
Copy link
Member

Tyriar commented Jul 12, 2017

I can't seem to reproduce. I do see weird wrapping happening during a resize but there isn't anything we can do about that (related to upstream https://github.com/rprichard/winpty).

@Tyriar Tyriar added the info-needed Issue requires more information from poster label Jul 12, 2017
@cleidigh
Copy link
Contributor Author

@Tyriar
Hmm, so you never see duplicate lines of the window shell? I can reproduce this easily, it's quite consistent.
I have a suspicion it may be related to the PTY startup time.
I'll try to look into it more

@fabiospampinato
Copy link
Contributor

I think I can reproduce it, it happened to me after writing this code in an extension:

const term = vscode.window.createTerminal ();
term.sendText ( 'mycommand', true );

I thought it had something to do with the process not being ready to accept some text, so I added:

await term.processId;

after those lines, but it didn't work.

The only workaround I found was to wait an extra 100ms or so, but this doesn't seem an optimal solution, probably an older machine than mine, or one under heavy load might require a larger delay.

@Tyriar
Copy link
Member

Tyriar commented Jul 14, 2017

@fabiospampinato I think that's unrelated. I suspect this issue is related to winpty acting weird when the console is sized narrowly. You issue is just the text is sent before the pty is ready to pick it up so you see it at the top, this is kind of as designed. You're on a mac right?

@fabiospampinato
Copy link
Contributor

Yes, I tried to reproduce it but without success.

I'd try it in my Windows virtual machine as well, if only it didn't break after I upgraded macOS.

@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug upstream Issue identified as 'upstream' component related (exists outside of VS Code) windows VS Code on Windows issues and removed info-needed Issue requires more information from poster labels Dec 15, 2017
@Tyriar Tyriar added this to the Backlog milestone Dec 15, 2017
@Tyriar
Copy link
Member

Tyriar commented Dec 23, 2017

Duplicate #26375

@Tyriar Tyriar closed this as completed Dec 23, 2017
@Tyriar Tyriar added the *duplicate Issue identified as a duplicate of another issue(s) label Dec 23, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Feb 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code) windows VS Code on Windows issues
Projects
None yet
Development

No branches or pull requests

4 participants