Skip to content

watch command doesn't work properly #8869

Closed

Description

Environment

Windows build number: 10.0.19042.0
Windows Terminal version (if applicable): 1.5.3242.0

Any other software? 
OpenSSH_for_Windows_8.1p1, LibreSSL 2.9.2

Steps to reproduce

Run watch command

Expected behavior

Should look like the powershell output in the image (1) below

Actual behavior

Confusing text wrapping as shown in the image (WT in the background)

There's also the scrollbar that appears when using watch, but ideally, I don't think the scrollbar should be available. Powershell also faces this issue ONLY when there's a previous scrollback.

1. Powershell and WT. Powershell has no scrollback now, so looks okay

image

2. Powershell and WT both have previous scrollback, so both are 'broken'

image

3. WT when scrolled up when running watch

wt

Activity

  1. ghost added
    Needs-TriageIt's a new issue that the core contributor team needs to triage at the next triage meeting
    on Jan 24, 2021
  2. ghost added
    Needs-Tag-FixDoesn't match tag requirements
    on Jan 24, 2021
  3. zadjii-msft commented on Jan 25, 2021

    @zadjii-msft
    Member

    OKay I'm not really sure what's going on here. Does this repro with WSL, without ssh? Because just running watch ls locally seems mostly fine to me.

    I think watch wants to be using the alt buffer, which conpty & WT don't yet support (#381). That would be what's causing the scrollbar to appear. I'm not really sure what else is wrong exactly here.

  4. MythreyaK commented on Jan 25, 2021

    @MythreyaK
    Author

    You're right, WSL and ssh through WSL works fine. CMD, Powershell and Powershell Core all face the issue I describe above

  5. ghost added
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    on Jan 25, 2021
  6. ghost removed
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    on Jan 25, 2021
  7. DHowett commented on Jan 25, 2021

    @DHowett
    Member

    Sorry, would you mind clarifying?

    You're using watch:

    • through WSL
    • through ssh inside WSL
    • through PowerShell
    • through ssh from PowerShell
    • through CMD
    • through ssh from CMD

    ?

  8. added
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    and removed
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    on Jan 25, 2021
  9. MythreyaK commented on Jan 25, 2021

    @MythreyaK
    Author

    It seems to be triggered sometimes by running the clear command in an active ssh session, and resizing the window sometimes (resize without watch running).

    ❌ Means there was at the least one instance where I had the problem.
    ✔ means I couldn't reproduce the issue as far as I tried (clear screen, resize window b/w steps, backtracking to previous steps and trying again.

    For example, a series of steps could be wsl → powershell core → ssh → watch date → stop watching → clear → exit ssh → clear → ssh → watch date that failed)

    ✔ wsl → watch date
    ✔ wsl → ssh → watch date
    ❌ wsl → powershell core/cmd/powershell → ssh → watch date
    ❌ Powershell Core → ssh → run clear → watch date

  10. ghost added
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    on Jan 25, 2021
  11. ghost removed
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    on Jan 25, 2021
  12. DHowett commented on Jan 25, 2021

    @DHowett
    Member

    Okay, good! Thanks! This has the same root cause as /dup PowerShell/openssh-portable#426. The fix should be available in versions of Win32-OpenSSH greater than 7.7 😄

  13. ghost closed this as completedon Jan 25, 2021
  14. ghost removed
    Needs-TriageIt's a new issue that the core contributor team needs to triage at the next triage meeting
    on Jan 25, 2021
  15. ghost added
    Resolution-ExternalFor issues that are outside this codebase
    on Jan 25, 2021
  16. ghost removed
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    on Jan 25, 2021
  17. MythreyaK commented on Jan 25, 2021

    @MythreyaK
    Author

    I'm on OpenSSH_for_Windows_8.1p1, LibreSSL 2.9.2, so not sure what's happening

  18. DHowett commented on Jan 25, 2021

    @DHowett
    Member

    It may be worth raising this issue on their tracker to figure out when it's going to be released -- I thought that 8.1 would contain recent fixes, but perhaps it does not.

  19. MythreyaK commented on Jan 25, 2021

    @MythreyaK
    Author

    Okay, thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Needs-Tag-FixDoesn't match tag requirementsResolution-ExternalFor issues that are outside this codebase

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions