-
Notifications
You must be signed in to change notification settings - Fork 511
Script output does not get written to the debug console #284
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
Comments
Yep, this is a general problem with output flushing at the moment. I've got an issue filed at the PS Editor Services repo to track the work for this: PowerShell/PowerShellEditorServices#138 |
@daviwil Thanks, this is a show-stopper for VSCode. I'm using the PowerShell Console host to run tests for now, instead. Still authoring in VSCode. |
can you make 0.6.2 available, it;s the last working verion |
Hey @bughit, there's a set of steps you can follow here to get back to 0.6.2 (just change the version from 0.6.1 to 0.6.2): Is it the script output issue that's blocking you or something else? |
yes |
Has there been any progress made on this? It's sort of killing me. |
Yep! I've made huge progress with moving everything to a new model where all console activity happens directly in the real PowerShell console running in the integrated terminal. I think I'll have a new update ready in a couple weeks. I'm really sorry this issue has been causing your trouble, soon you won't have to deal with it anymore :) |
Awesome, thanks David. |
Problem's still there with VSCode 1.7.1 (upgraded today) with version 0.7.2 of the extension... Back to ISE for now ;-) |
Hoping this gets fixed in VSCode v1.7.2 and v0.7.2 of the extension soon. Thanks! |
I hate to say, but with VSCode v1.7.2 and v0.7.2 of the extension the problem's still there... |
Yes, it is still a problem. You can workaround the issue by setting a breakpoint on the last line of script that executes. |
@daviwil any ETA on the fix for this? |
I'm working in it this week to be included with 0.8.0. Will keep you posted! |
@daviwil Nice!! I will test and tweet about it as soon as it drops! |
Thanks, @daviwil! |
This change fixes an issue where script output for very small scripts is not being written out reliably before the debug adapter terminates. The fix is to add an extra output flush before sending the TerminatedEvent back to the client. This fix is temporary until we get the REPL integration online. Resolves #138. Resolves PowerShell/vscode-powershell#284.
I've got a fix for this, it'll definitely be in 0.8.0: PowerShell/PowerShellEditorServices#306 |
Can we test somehow? |
I'll send out a preview build a little later today, would be nice to have folks try it out. |
Cool, where would I be able to find this? |
I'll post it to this issue so you'll see it. |
OK, here's a preview package. Ignore the version number here, I'm using an arbitrarily increased version just for the purpose of sending out a preview. Also, just to be clear, USE AT YOUR OWN RISK. This build was created on my dev machine as a convenience for you guys so make sure you do the proper virus scanning, etc before you use it. Download here: PowerShell-0.7.10.zip To install the preview, go into VS Code and run the This build should be stable enough to use until 0.8.0 gets released next week, but definitely file issues if you find any problems. |
@ConstantineK Do your reactions mean that it worked for you? :) |
Have to test in a bit, just something I have been waiting for :D |
Cool! Let me know how it goes :) |
I've tested with a first, very small script of mine, works fine. My plan is to try to stop using ISE for a while, and see what that gets me... I'll keep you posted. |
@sba923 thanks for letting me know! Keep me posted |
I did manage to get it to crash:
|
@rkeithhill Consistently or just once? That seems unrelated to this change but its in an area where I've seen crashes before |
Just once. Unfortunately, I haven't been able to repro it. |
Works for me, thanks David. Does this build include any other fixes? |
That's awesome actually. My intellisense is incredibly slow, but that's always been the case. Ha. |
Shouldn't be slow, are you running it on a machine with limited resources (like a VM)? Was it faster in an earlier version of the extension? |
I'm not, and I can't work out why it's so slow. I have to wait about 30 seconds to a minute to get any suggestions back. |
got the same crash as rkeithhill, twice. but now it doesn't happen again. |
Having a few situations this morning where the same behavior (output not flushing) is occuring, trying to create a reproducible scenario now. |
System Details
$PSVersionTable
:Issue Description
Sometimes I run a PowerShell script, and the Debug Console shows nothing.
Sometimes I run a PowerShell script, and the Debug Console shows output, but not Error stream output.
Sometimes I run a PowerShell script and the Debug Console shows output from the Output and Error streams, as expected.
This happens with both VSCode and VSCode Insiders (as of today's release).
Reproduction Steps
Inconsistent results.
Cheers,
Trevor Sullivan
Docker Captain
Microsoft MVP: Cloud & Data Center Management
https://trevorsullivan.net
https://twitter.com/pcgeek86
The text was updated successfully, but these errors were encountered: