-
-
Notifications
You must be signed in to change notification settings - Fork 14.3k
Unix implementation for stdio set/take/replace #150668
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
base: main
Are you sure you want to change the base?
Conversation
|
rustbot has assigned @Mark-Simulacrum. Use |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me if you want to move these to tracking issue open questions
| fn replace_stdio_fd(this: BorrowedFd<'_>, other: OwnedFd) -> io::Result<()> { | ||
| cfg_select! { | ||
| not(any( | ||
| target_arch = "wasm32", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wasip1 supports fd_renumber which is like dup2 except moving the fd rather than duplicating it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But wasip1 doesn't implement try_clone_to_owned, so replace and take won't work. But ok, we can still support set with that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added it.
|
@bors r=Mark-Simulacrum |
Unix implementation for stdio set/take/replace Tracking issue: rust-lang#150667 ACP: rust-lang/libs-team#500
Unix implementation for stdio set/take/replace Tracking issue: rust-lang#150667 ACP: rust-lang/libs-team#500
Unix implementation for stdio set/take/replace Tracking issue: rust-lang#150667 ACP: rust-lang/libs-team#500
Rollup of 8 pull requests Successful merges: - #144113 (Impls and impl items inherit `dead_code` lint level of the corresponding traits and trait items) - #149880 (rustc_codegen_llvm: update alignment for double on AIX) - #150668 (Unix implementation for stdio set/take/replace) - #150670 (THIR pattern building: Move all `thir::Pat` creation into `rustc_mir_build::thir::pattern`) - #150695 (MGCA: pretty printing for struct expressions and tuple calls ) - #150698 (Improve comment clarity in candidate_may_shadow) - #150706 (Update wasm-component-ld) - #150710 (Update libc dependency of std) r? `@ghost` `@rustbot` modify labels: rollup
|
I don't think this is responsible for rollup failure in #150718 (comment), but let's double-check: @bors try jobs=dist-ohos-armv7 |
Unix implementation for stdio set/take/replace try-job: dist-ohos-armv7
This comment has been minimized.
This comment has been minimized.
Unix implementation for stdio set/take/replace Tracking issue: rust-lang#150667 ACP: rust-lang/libs-team#500
This comment has been minimized.
This comment has been minimized.
Unix implementation for stdio set/take/replace try-job: dist-various-1
|
@bors r=Mark-Simulacrum rollup=iffy |
Unix implementation for stdio set/take/replace Tracking issue: rust-lang#150667 ACP: rust-lang/libs-team#500
…uwer Rollup of 3 pull requests Successful merges: - #150094 (`c_variadic`: provide our own `va_arg` implementation for more targets) - #150569 (Ensure that static initializers are acyclic for NVPTX) - #150668 (Unix implementation for stdio set/take/replace) r? `@ghost` `@rustbot` modify labels: rollup
|
The Miri subtree was changed cc @rust-lang/miri |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
@bors try jobs=x86_64-gnu-aux |
This comment has been minimized.
This comment has been minimized.
Unix implementation for stdio set/take/replace try-job: x86_64-gnu-aux
This comment has been minimized.
This comment has been minimized.
|
Miri patch LGTM, thanks. You can change that fail-dep test into a pass-dep test. |
std supports redirecting stdio file descriptors.
|
@bors r=Mark-Simulacrum,RalfJung |
…Jung Unix implementation for stdio set/take/replace Tracking issue: rust-lang#150667 ACP: rust-lang/libs-team#500
Tracking issue: #150667
ACP: rust-lang/libs-team#500