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

Integrated terminal cannot be reused #6618

Closed
a1994846931931 opened this issue Nov 25, 2019 · 3 comments · Fixed by #6619 or #6668
Closed

Integrated terminal cannot be reused #6618

a1994846931931 opened this issue Nov 25, 2019 · 3 comments · Fixed by #6619 or #6668
Labels
bug bugs found in the application debug issues that related to debug functionality tasks issues related to the task system

Comments

@a1994846931931
Copy link
Contributor

a1994846931931 commented Nov 25, 2019

Description

Theia can now support integrated terminal, thanks to commit 0327a4d. But there remains a problem that the terminal cannot be reused (see the posted gif below).

Reproduction Steps

  1. Install python plugin (from VSCode marketplace)
  2. Debug with the following configuration ( console should be integratedTerminal )
    {
      "name": "Python: Current File",
      "type": "python",
      "request": "launch",
      "program": "${file}",
      "console": "integratedTerminal"
    }
  1. End the debug session but keep the integrated terminal. Then run another debug session and see how theia lanch it in a new terminal.

python-integrated

OS and Theia version:
OS: Ubuntu
Version: 145f913

Diagnostics:
In line

let terminal = this.terminalServer.all.find(t => t.title.label === options.title);

The options.title is compared with t.title.label, but t.title.label changes.

@akosyakov akosyakov added bug bugs found in the application debug issues that related to debug functionality tasks issues related to the task system labels Nov 25, 2019
@akosyakov
Copy link
Member

@elaihau Would it be possible to refactor tasks that they preserve identity as in VS Code? It breaks in many places not only for debugging. Generally running the same task multiple times should NOT be allowed and the same terminal should be used. Right now it is most annoying mismatch.

a1994846931931 added a commit to a1994846931931/theia that referenced this issue Nov 25, 2019
@elaihau
Copy link
Contributor

elaihau commented Nov 26, 2019

@a1994846931931
quick question: is the python plugin mentioned in this ticket

  • the python extension in Theia, or
  • the python extension from vsCode?

I was getting a "The debug session type "python" is not supported." error when tried to reproduce this error. Could you please advise what I should do to make debugging python code work?

Thanks !

@a1994846931931
Copy link
Contributor Author

@a1994846931931
quick question: is the python plugin mentioned in this ticket

  • the python extension in Theia, or
  • the python extension from vsCode?

I was getting a "The debug session type "python" is not supported." error when tried to reproduce this error. Could you please advise what I should do to make debugging python code work?

Thanks !

@elaihau
Hi, sorry I forgot to mention that it's VSCode's python extension.

elaihau pushed a commit that referenced this issue Nov 26, 2019
elaihau pushed a commit to elaihau/theia that referenced this issue Dec 1, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit that referenced this issue Dec 3, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves #6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Dec 3, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Dec 6, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Dec 6, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Dec 6, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Dec 6, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Dec 6, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
elaihau pushed a commit that referenced this issue Dec 8, 2019
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves #6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
akosyakov pushed a commit to akosyakov/theia that referenced this issue Feb 24, 2020
akosyakov pushed a commit to akosyakov/theia that referenced this issue Feb 24, 2020
With changes in this pull request, Theia offers users the flexibility of terminating or restarting a task, if the user tries to run a task that is actively running.
This pull request resolves eclipse-theia#6618 (comment)

Signed-off-by: Liang Huang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application debug issues that related to debug functionality tasks issues related to the task system
Projects
None yet
3 participants