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

Stablize anonymous pipe #137793

Merged
merged 2 commits into from
Mar 17, 2025
Merged

Stablize anonymous pipe #137793

merged 2 commits into from
Mar 17, 2025

Conversation

NobodyXu
Copy link
Contributor

@NobodyXu NobodyXu commented Feb 28, 2025 •
edited by tgross35
Loading

Since #135822 is staled, I create this PR to stablise anonymous pipe

Closes #127154

try-job: test-various

@rustbot

This comment was marked as resolved.

Sign in to view
@rustbot rustbot added A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Feb 28, 2025
@rustbot
Copy link
Collaborator

rustbot commented Feb 28, 2025

This PR modifies tests/run-make/. If this PR is trying to port a Makefile
run-make test to use rmake.rs, please update the
run-make port tracking issue
so we can track our progress. You can either modify the tracking issue
directly, or you can comment on the tracking issue and link this PR.

cc @jieyouxu

The Miri subtree was changed

cc @rust-lang/miri

@jieyouxu
Copy link
Member

Rolling the original libs reviewer for the other stabilization attempt.
r? @joboet

@rustbot rustbot assigned joboet and unassigned ibraheemdev Feb 28, 2025
@oli-obk
Copy link
Contributor

oli-obk commented Feb 28, 2025

Please squash the commits

@NobodyXu NobodyXu force-pushed the stablise-annoymous-pipe branch from 791a3fd to 57bdbe6 Compare February 28, 2025 14:37
@rustbot
Copy link
Collaborator

rustbot commented Feb 28, 2025

Some changes occurred in src/tools/cargo

cc @ehuss

@jieyouxu
Copy link
Member

(Some unintentional submodule changes I think)

@rust-log-analyzer

This comment has been minimized.

Sign in to view
@jieyouxu jieyouxu added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 28, 2025
@NobodyXu
Copy link
Contributor Author

NobodyXu commented Mar 1, 2025

The CI error is not related to any changes in this PR:

tidy error: could not find exception package `foldhash`

@NobodyXu NobodyXu changed the title Stablise anonymous pipe Stablize anonymous pipe Mar 1, 2025
@tgross35
Copy link
Contributor

tgross35 commented Mar 1, 2025

#135822 also did some refactoring, are you also planning to put that up? If you do it in a separate PR feel free to request a review from me and we can probably merge it pretty quick.

@NobodyXu
Copy link
Contributor Author

NobodyXu commented Mar 1, 2025

If you do it in a separate PR feel free to request a review from me and we can probably merge it pretty quick.

Yes I want to put it in separate PR, I hope this PR can merge quick and hope that debian could pick it up, so that jobserver/etc can use it to reduce their burden

@NobodyXu NobodyXu force-pushed the stablise-annoymous-pipe branch 2 times, most recently from 8acff46 to 7acb0e3 Compare March 1, 2025 13:35
@NobodyXu
Copy link
Contributor Author

NobodyXu commented Mar 1, 2025

Reverted submodules changes, and tidy passed locally, so CI should be fixed now

@jieyouxu jieyouxu mentioned this pull request Mar 1, 2025
Open
6 tasks
@NobodyXu
Copy link
Contributor Author

NobodyXu commented Mar 1, 2025

cc @joboet this pr is ready for review

@joshtriplett
Copy link
Member

LGTM

@NobodyXu
Copy link
Contributor Author

NobodyXu commented Mar 4, 2025

Can we get this PR merged please?

joboet
joboet requested changes Mar 5, 2025
View reviewed changes
Copy link
Member

@joboet joboet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still have the same request as in the original PR:

The changes here look good to me, though I'd like to see another change before stabilization:

Currently, things like the Stdio: From<PipeWriter> implementations live in sys:: which IMHO is really confusing and dangerous (because people will forget to add these implementations when porting std). Could you move all the public trait implementations to io::pipe (and os::_::pipe for the platform-specific implementations)?

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 10, 2025
@jieyouxu jieyouxu removed T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) A-tidy Area: The tidy tool labels Mar 10, 2025
@NobodyXu
Copy link
Contributor Author

Thanks, will investigate it in 1-2d

NobodyXu added 2 commits March 14, 2025 00:52
@NobodyXu
Stablize feature anonymous_pipe
340a452 
Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>
@NobodyXu
Mv os-specific trait impl of Pipe* into std::os::*
6863a99 
Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>
@NobodyXu NobodyXu force-pushed the stablise-annoymous-pipe branch from 9a91f4f to 6863a99 Compare March 13, 2025 14:05
@NobodyXu
Copy link
Contributor Author

I have fixed the error on wasm32-wasip1 by building for it locally

It would be great if someone can configure the CI to run the tests for that target to verify that

@tgross35
Copy link
Contributor

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 13, 2025
@bors
Auto merge of rust-lang#137793 - NobodyXu:stablise-annoymous-pipe, r=…
639d746 
…<try>

Stablize anonymous pipe

Since rust-lang#135822 is staled, I create this PR to stablise anonymous pipe

Closes rust-lang#127154

try-job: test-various
@bors
Copy link
Contributor

bors commented Mar 13, 2025

⌛ Trying commit 6863a99 with merge 639d746...

@bors
Copy link
Contributor

bors commented Mar 13, 2025

☀️ Try build successful - checks-actions
Build commit: 639d746 (639d746f06b58f56711e3376dfa3e63104df8fc8)

@NobodyXu
Copy link
Contributor Author

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 14, 2025
@joboet
Copy link
Member

joboet commented Mar 17, 2025

@bors r=@joshtriplett

@bors
Copy link
Contributor

bors commented Mar 17, 2025

📌 Commit 6863a99 has been approved by joshtriplett

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 17, 2025
@jhpratt jhpratt mentioned this pull request Mar 17, 2025
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 17, 2025
@bors
Auto merge of rust-lang#138595 - jhpratt:rollup-09pvfzu, r=jhpratt
9c67cec 
Rollup of 9 pull requests

Successful merges:

 - rust-lang#136355 (Add `*_value` methods to proc_macro lib)
 - rust-lang#137621 (Add std support to cygwin target)
 - rust-lang#137793 (Stablize anonymous pipe)
 - rust-lang#138341 (std: Mention clone-on-write mutation in Arc<T>)
 - rust-lang#138517 (Improve upvar analysis for deref of child capture)
 - rust-lang#138584 (Update Rust Foundation links in Readme)
 - rust-lang#138586 (Document `#![register_tool]`)
 - rust-lang#138590 (Flatten and simplify some control flow 🫓)
 - rust-lang#138592 (update change entry for rust-lang#137147)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 62c2a7b into rust-lang:master Mar 17, 2025
7 checks passed
@rustbot rustbot added this to the 1.87.0 milestone Mar 17, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Mar 17, 2025
@rust-timer
Unrolled build for rust-lang#137793
858ec72 
Rollup merge of rust-lang#137793 - NobodyXu:stablise-annoymous-pipe, r=joshtriplett

Stablize anonymous pipe

Since rust-lang#135822 is staled, I create this PR to stablise anonymous pipe

Closes rust-lang#127154

try-job: test-various
@tbu- tbu- mentioned this pull request Mar 17, 2025
Closed
@jhpratt jhpratt mentioned this pull request Mar 18, 2025
Closed
@jhpratt
Copy link
Member

jhpratt commented Mar 18, 2025

still in the queue after being merged

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jieyouxu jieyouxu jieyouxu left review comments

@joboet joboet joboet requested changes

Assignees

@joboet joboet

Labels
A-run-make Area: port run-make Makefiles to rmake.rs O-unix Operating system: Unix-like O-windows Operating system: Windows S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.87.0
Development

Successfully merging this pull request may close these issues.

Tracking Issue for anonymous pipe API