Garrett Campbell

Results 456 comments of Garrett Campbell

Closing due to not getting necessary info.

Tracking this, looking forward to a fix, thanks!

@MikeWeller Thank you for notifying us. Marking this as `stale-exempt` to keep it alive. @xisui-MSFT FYI.

@tvatter Thanks for following up and reminding us of this frustration. I've upgraded this bug from the backlog to On Deck, which is where we pull bugs from to solve...

@dgazzoni @kh-huawei @tvatter @MikeWeller There actually is already support for this, assuming that you have the `cmake.ctest.testExplorerIntegrationEnabled` setting enabled. If you do, then you can navigate to the Testing Pane...

@dgazzoni Thanks for following up, I also just tested this and I see the behavior you're seeing. I'm moments away from opening a PR that fixes this behavior for you,...

@dgazzoni @MikeWeller @tvatter @kh-huawei I've created a PR here #4141 and dropped a vsix in the description for you to test. You'll need to download the zip file, and NOT...

@dgazzoni Thanks for testing that. I thought that I tested and ensured that it still killed the process. I'll try to test again, if you have any specific test repro...

I'm looking at my test repro wit that setting enabled, and I don't see any issue, the ctest.exe invocation seems to be properly killed. Is there something I'm misunderstanding?

In my repro it works, but of course my repro is much simpler, so thank you for the repro. I'll test with your repro, but I do wonder if it...