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

PowerShell internal process - The handle is invalid. #526

Closed
ferventcoder opened this issue Jan 1, 2016 · 5 comments
Closed

PowerShell internal process - The handle is invalid. #526

ferventcoder opened this issue Jan 1, 2016 · 5 comments

Comments

@ferventcoder
Copy link
Member

It appears that with some installs, there are errors with running the process and all that comes back is an invalid handle. Need to investigate what causes this.

==> default: Elevating Permissions and running C:\Users\ADMINI~1\AppData\Local\Temp\chocolatey\arduino\1.6.7\arduinoInstall.EXE /S . This may take a while, depending on the statements.
==> default: ERROR: The handle is invalid.

@ferventcoder
Copy link
Member Author

This seems to happen only in the verifier in vagrant. :/

@ferventcoder
Copy link
Member Author

I found the issue! It's trying to write out an error stream to a file in Start-ChocolateyProcessAsAdmin. Rewriting that now.

@ferventcoder
Copy link
Member Author

This may also fix #434.

ferventcoder added a commit that referenced this issue Jan 11, 2016
Start-ChocolateyProcessAsAdmin was previously using an outdated way to
read error messages from a stream that causes errors when running
PowerShell with C#. Instead we should ensure that the process uses an
async redirection of output and not need to read the output at the end
from a filestream.
ferventcoder added a commit that referenced this issue Jan 11, 2016
* stable:
  (GH-526) Fix: The handle is invalid
  (GH-532) Fix - Temp uses 8.3 Path
  (maint) update binroot deprecation
@ferventcoder
Copy link
Member Author

It's still an issue related to redirecting stdout/stderr to a file stream, followed up with #572.

ferventcoder added a commit that referenced this issue Feb 11, 2016
* pr617:
  (maint) comment cleanup
  (GH-526) set messaging to output as error
  (GH-520) Debug/Verbose should be global scope
  (maint) formatting
  (GH-616) Deterministicly capture unzipped/extracted files
ferventcoder added a commit that referenced this issue Feb 11, 2016
* stable:
  (maint) comment cleanup
  (GH-526) set messaging to output as error
  (GH-520) Debug/Verbose should be global scope
  (maint) formatting
  (GH-616) Deterministicly capture unzipped/extracted files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants