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.

Sign up for GitHub

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

Jump to bottom

std::run loops forever if exec fails #9081

Closed
thestinger opened this issue Sep 9, 2013 · 2 comments
Closed

std::run loops forever if exec fails #9081

thestinger opened this issue Sep 9, 2013 · 2 comments
Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added.

Comments

@thestinger
Copy link
Contributor

We should just be using posix_spawn here, both for performance and to avoid these edge cases.
This was referenced Sep 9, 2013
Closed
Closed
Closed
@alexcrichton
Copy link
Member

This works both in the native and the libuv cases now (not sure what ended up fixing this).

Flagging as needstest, although all of the process tests need to be run on 1:1 and M:N. I'm making a framework for this soon after #10965 lands, so I'll take care of this soon after that.

@alexcrichton
Copy link
Member

A test is now checked in (https://github.com/mozilla/rust/blob/b7f673a627c674b289721ee339fd2980919c6afd/src/libstd/io/process.rs#L200-L213), closing.

flip1995 pushed a commit to flip1995/rust that referenced this issue Jul 18, 2022
@bors
Auto merge of rust-lang#9085 - xFrednet:9081-update-format-push-versi…
f30b68d 
…on, r=Manishearth,flip1995

Correct lint version for `format_push_string`

Closes rust-lang#9081

changelog: none

IDK what else to say. Look I can draw an ascii penguin =D:

```
 (^v^)
<(   )>
  w w
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alexcrichton @thestinger