The runas feature that was added in 93423aa2e5e4b7de6452090b0039560d2b13...

[UtahDave]

...02e4 jacks up Windows support for the minion.

Adding in these 2 try statements allows Windows to load the minion.

Is this the correct way to do this?

[terminalmage]

I've actually found that the setuid/seteuid method won't allow a user's enironment to be imported. To do this, either the user's environment file must be imported, or the command must be run with su (su - user -c 'cmd').

I'll be submitting my own pull request soon which will incorporate some of UtahDave's changes, but which will ignore the runas capability if the command is being run on Windows.

[terminalmage]

@UtahDave Before I submit this as a pull request, can you try this on Windows and make sure that it doesn't unmake the universe? :)

https://gist.github.com/1659443

[UtahDave]

archtaku's fix here https://gist.github.com/1659443 seems to work great.

Let's not use this pull request and use his to address this issue.