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

Proper handling of multiline copy paste #3418

Closed
testautomation opened this issue Nov 2, 2019 · 3 comments
Closed

Proper handling of multiline copy paste #3418

testautomation opened this issue Nov 2, 2019 · 3 comments
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.

Comments

@testautomation
Copy link

Description of the new feature/enhancement

As a user I want to be able to paste long commands that are spread over multiple lines where the linebreaks are escaped with \. Example below

sudo apt-get install \
    apt-transport-https \
    ca-certificates \
    curl \
    gnupg-agent \
    software-properties-common

ACTUAL

The actual behavior is that terminal tries to execute each line immediately . See example below

learn@DESKTOP-4FCRN5Q:~/projects$ sudo apt-get install \
>
Reading package lists... Done
Building dependency tree
Reading state information... Done
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
learn@DESKTOP-4FCRN5Q:~/projects$     apt-transport-https \
>
apt-transport-https: command not found
learn@DESKTOP-4FCRN5Q:~/projects$     ca-certificates \
>
ca-certificates: command not found
learn@DESKTOP-4FCRN5Q:~/projects$     curl \
>
curl: try 'curl --help' or 'curl --manual' for more information
learn@DESKTOP-4FCRN5Q:~/projects$     gnupg-agent \
>
gnupg-agent: command not found

EXPECTED

The multiline command from above example is treated as on command and after being paste waits for the user to press ENTER before it is executed.

NOTE: The Ubuntu WSL is a good example of how it should behave correctly
image

@testautomation testautomation added the Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. label Nov 2, 2019
@ghost ghost added Needs-Tag-Fix Doesn't match tag requirements Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Nov 2, 2019
@Luuk34
Copy link

Luuk34 commented Nov 2, 2019

You should not copy the final end-of-line character. If you do than the final ENTER will not be pasted, and you will have the ability to do it yourself.

@DHowett-MSFT
Copy link
Contributor

/dup #1091.

@ghost
Copy link

ghost commented Nov 3, 2019

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost ghost closed this as completed Nov 3, 2019
@ghost ghost added Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Nov 3, 2019
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.
Projects
None yet
Development

No branches or pull requests

2 participants