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

Restart in a Multitarget Debugging rotates targets in the dropdown list #17401

Closed
Spown opened this issue Dec 16, 2016 · 13 comments
Closed

Restart in a Multitarget Debugging rotates targets in the dropdown list #17401

Spown opened this issue Dec 16, 2016 · 13 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Milestone

Comments

@Spown
Copy link

Spown commented Dec 16, 2016

  • VSCode Version: 1.8
  • OS Version: Win10x64

First of all thank you for this feature! No longer need I to nest projects and open multiple windows. But there is an inconvenience problem...

Steps to Reproduce:

  1. Have a compound launch config with ['server', 'client']
  2. launch it
  3. restart 'server'

Desired behavior: 'server' is still selected in target dropdown option list.

What actually happens: 'client' gets selected

Why? Because the dropdown list removes the 'server' from the stack ->'client' becomes the only item so it gets selected -> new 'server' instance gets pushed into the list, but doesn't get reselected. If you restart once more - the same happens only with roles switched. They start to rotate.

So, if I'm used to restart via a hotkey - I normally don't look at what launch config is currently running - I just know it. If I'm now focused on the server-side development I probably want the selected target stay selected in the list because I want to restart it often. Without reselecting it manually every time.

Thank you.

@kieferrm kieferrm added the debug Debug viewlet, configurations, breakpoints, adapter issues label Dec 16, 2016
@Spown
Copy link
Author

Spown commented Dec 17, 2016

If I may suggest, the optimal behavior would be not to remove the list item when the debug session is terminated, but rather when/if the restart has failed (and maybe after a timeout). If it succeeded - the new instance would be bound to the list item.

@isidorn isidorn added this to the January 2017 milestone Dec 19, 2016
@isidorn isidorn added the bug Issue identified by VS Code Team member as probable bug label Dec 23, 2016
@isidorn
Copy link
Contributor

isidorn commented Dec 23, 2016

@Spown thanks for the suggestion, however went with the other approach of restarting refocusing the process on finish if the process was focussed before. This makes sense in general imho

Glad you like multi process debugging. Feedback welcome and writing concise issues is also appreciated :)

@jrieken
Copy link
Member

jrieken commented Jan 27, 2017

For me it doesn't focus the restarted process but the other one

jan-27-2017 15-00-27

@jrieken jrieken reopened this Jan 27, 2017
@jrieken jrieken added the verification-found Issue verification failed label Jan 27, 2017
@isidorn isidorn modified the milestones: February 2017, January 2017 Jan 27, 2017
@isidorn isidorn closed this as completed in bf4701e Feb 2, 2017
@isidorn
Copy link
Contributor

isidorn commented Feb 2, 2017

@jrieken nice find! I accidently broke this when doing some refactoring of process ids.

@Spown
Copy link
Author

Spown commented Feb 3, 2017

so, another month to wait?

@isidorn
Copy link
Contributor

isidorn commented Feb 6, 2017

@Spown no, you can use the vscode insiders to have it right now. If you want to use stable, then yes you have to wait - sorry about that.

@Spown
Copy link
Author

Spown commented Feb 9, 2017

you could've snuck it in 1.9.1... can you move it in some other milestone in case 1.9.2 comes out before 1.10?

@isidorn
Copy link
Contributor

isidorn commented Feb 10, 2017

No. In recovery builds we are carefuly about only picking critical fixes and bad regressions. Since this is neither I recommend that you use the VS Code insiders.
Sorry about that.

@chrmarti
Copy link
Contributor

  • VSCode Version: Code - Insiders 1.10.0-insider (f02f993, 2017-02-24T07:04:53.237Z)
  • OS Version: Darwin x64 15.6.0
  • Extensions: None

The launch drop-down in the debug widget switches to the second launch when the first is restarted (and vice-versa), but subsequent restarts (or any other action on the debug widget) still target the first one.

17401

@chrmarti chrmarti reopened this Feb 24, 2017
@Spown Spown changed the title Restart in a Multitarget Debugging messes the workflow Restart in a Multitarget Debugging rotates targets in the dropdown list Feb 24, 2017
@chrmarti
Copy link
Contributor

The debug widget not targeting the process shown in the drop-down is a regression from 1.9.1.

@chrmarti
Copy link
Contributor

Tricky: FocusProcessActionItem gets the newly selected item (the restarted process) before it gets the new list of items containing that item. (Suspending my investigation.)

@chrmarti
Copy link
Contributor

Verified that the drop-down and the call stack selection restore the previously selected process after that restarted, the correct process is targeted afterwards and the call stack selection follows the drop-down's selection.

The drop-down doesn't follow the call stack's selection, but I assume that is expected. Marking verified.

@chrmarti chrmarti added verified Verification succeeded and removed verification-found Issue verification failed labels Feb 27, 2017
@Spown
Copy link
Author

Spown commented Feb 27, 2017

Thank you.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
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 debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants