Skip to content

Conversation

@marco-miller
Copy link
Contributor

@marco-miller marco-miller commented May 16, 2023

Start the trace server if known as stopped, and if [1] is installed, before opening traces.

[1] https://github.com/eclipse-cdt-cloud/vscode-trace-server

Contributes to fixing Issue #15.

Signed-off-by: Marco Miller [email protected]

@marco-miller marco-miller requested a review from bhufmann May 26, 2023 16:31
@marco-miller
Copy link
Contributor Author

Comment from internal, from @bhufmann:

  • "let's backlog and discuss the way forward.
  • Some downstream projects of the theia-trace-extension introduce a command to open the traces without starting the trace server automatically,
  • because they have [their] own way to start/stop the trace server
  • and want to by-pass the automatic start and stop."

Start the trace server if known as stopped, and if [1] is installed,
before opening traces.

[1] https://github.com/eclipse-cdt-cloud/vscode-trace-server

Contributes to fixing Issue #15.

Signed-off-by: Marco Miller <[email protected]>
@marco-miller marco-miller marked this pull request as ready for review June 7, 2023 15:03
Copy link
Contributor

@bhufmann bhufmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is also a question to the community if that is a general behaviour that is expected. In our use case it's a yes, but I know from the theia-trace-extension adopters, that they have other means to start the server for the end-user and there is an additional open command that doesn't try to start the server.

Copy link
Contributor

@bhufmann bhufmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besides, with the current implementation, it requires the vscode-traceserver to be installed to have the commands available. This adds a tight coupling to vscode-traceserver. What about an api in vscode-trace-extension that other extensions can register a start and stop callback. This will allow other implementations that vscode-traceserver. Just a thought.

@AlexDoanBB
Copy link
Contributor

hi, it seems that this feature has been backlogged. Is there any plan to add an api for other extensions to register start and stop callback? I'd like to build upon #149 for this feature. Please let me know your thoughts, thanks

@bhufmann
Copy link
Contributor

bhufmann commented Jul 5, 2023

This one can be closed due to #155. Thanks for the proposal.

@marco-miller marco-miller deleted the server-first branch July 11, 2023 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants