Skip to content

proposal: os/exec: inherit process startup info for new commands on windows. #28166

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

Closed
Brett-Kavanaugh opened this issue Oct 12, 2018 · 7 comments
Labels
FrozenDueToAge OS-Windows Proposal WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Milestone

Comments

@Brett-Kavanaugh
Copy link

My proposal is for new processes spawned with the exec library to inherit the parents startup info. This is a windows specific proposal.

Currently, if I were to try and spawn a process in a specific desktop it would not work due to the startup info property not being inherited from the parent.

Thanks!

@gopherbot gopherbot added this to the Proposal milestone Oct 12, 2018
@mattn
Copy link
Member

mattn commented Oct 12, 2018

What do you mean inherited? bInheritHandles flag?

@Brett-Kavanaugh
Copy link
Author

Yes, meaning if I start a Go process in C with special STARTUPINFO then when I call exec in the Go process it should get that special STARTUPINFO.

@ianlancetaylor
Copy link
Contributor

The place for this kind of thing is syscall.SysProcAttr. Is the existing CreationFlags field sufficient for this? Could you just set a flag there?

@Brett-Kavanaugh
Copy link
Author

Brett-Kavanaugh commented Oct 12, 2018

@ianlancetaylor

How would I set this field?

@mattn
Copy link
Member

mattn commented Oct 12, 2018

@rsc
Copy link
Contributor

rsc commented Oct 17, 2018

@Brett-Kavanaugh, given that you can control the flags in syscall.SysProcAttr, is that enough? If not, what specifically should we be looking at adding to syscall.SysProcAttr? Thanks.

@rsc rsc added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Oct 17, 2018
@andybons
Copy link
Member

Closing per lack of response. If you have more to add, please comment.

Per discussion with @golang/proposal-review

@golang golang locked and limited conversation to collaborators Oct 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge OS-Windows Proposal WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
None yet
Development

No branches or pull requests

6 participants