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

(GH-346) Process with an id of is not running errors (posh v2 too) #400

Closed
wants to merge 1 commit into from

Conversation

Annih
Copy link
Contributor

@Annih Annih commented Aug 27, 2015

Hello

This commit may have fixe the named error for powershell v3, but not for v2
Using a .NET style start process it's compatible with powershell v2.

Fix #346

@ferventcoder
Copy link
Member

Howdy, would you mind reading through our contributing doc and fix up your commit message?

See https://github.com/chocolatey/choco/blob/master/CONTRIBUTING.md#prepare-commits in particular. :)

The commit ea6f1ff tried to fix the following error:
> Process with an id of XXX is not running

It succeed for powershell v3, but not with v2.
Using a .NET style start process it's compatible with powershell v2.
@Annih
Copy link
Contributor Author

Annih commented Aug 27, 2015

I hope this is better now, and fits your guidelines. According to the doc all specs must pass ... but I assume it's ok if the tests were failing before :)

@ferventcoder ferventcoder changed the title Fix: Process with an id of is not running errors (posh v2 too) (GH-346) Process with an id of is not running errors (posh v2 too) Aug 27, 2015
@ferventcoder
Copy link
Member

The CLA check didn't come up, can you please review and sign the CLA at https://www.clahub.com/agreements/chocolatey/choco? Thanks!

@ferventcoder
Copy link
Member

And now it comes up. Nice.

@ferventcoder
Copy link
Member

Fixed up in 78f5a4b and merged into stable at c892f95

@Annih
Copy link
Contributor Author

Annih commented Oct 1, 2015

Thans @ferventcoder

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.

Process with an id of is not running errors on 0.9.9.8
2 participants