Skip to content

Remove usage of legacy scheme paths on RedoxOS - #138457

Merged
bors merged 2 commits into
rust-lang:masterfrom
bjorn3:redox_scheme_paths
Mar 14, 2025
Merged

Remove usage of legacy scheme paths on RedoxOS#138457
bors merged 2 commits into
rust-lang:masterfrom
bjorn3:redox_scheme_paths

Conversation

@bjorn3

Copy link
Copy Markdown
Member

The name:/path path syntax is getting phased out1 in favor of /scheme/name/path. Also using null: is no longer necessary as /dev/null is available on Redox OS too.

cc @jackpot51

Footnotes

  1. https://gitlab.redox-os.org/redox-os/rfcs/-/blob/master/text/0006-scheme-path.md

The name:/path path syntax is getting phased out in favor of
/scheme/name/path. Also using null: is no longer necessary as /dev/null
is available on Redox OS too.
@bjorn3bjorn3 added the O-redox Operating system: Redox, https://www.redox-os.org/ label Mar 13, 2025
@rustbot

Copy link
Copy Markdown
Collaborator

r? @ibraheemdev

rustbot has assigned @ibraheemdev.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbotrustbot added O-unix Operating system: Unix-like S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Mar 13, 2025
@Noratrieb

Copy link
Copy Markdown
Member

@bors r+ rollup

@bors

bors commented Mar 13, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 83ee034 has been approved by Noratrieb

It is now in the queue for this repository.

@borsbors 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 13, 2025
Redox OS is moving away from name:/path style paths to /scheme/name/path
style paths which are already handled correctly without has_redox_scheme.
@bjorn3

Copy link
Copy Markdown
MemberAuthor

Added another commit.

@bors r-
@rustbot ready

@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 13, 2025
@borsbors 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. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 13, 2025
@Noratrieb

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Mar 13, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 43499bf has been approved by Noratrieb

It is now in the queue for this repository.

@borsbors 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 13, 2025
@jhprattjhpratt mentioned this pull request Mar 14, 2025
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 14, 2025
Rollup of 16 pull requests
Successful merges:
- rust-lang#136001 (Overhaul examples for PermissionsExt)
- rust-lang#136230 (Reword incorrect documentation about SocketAddr having varying layout)
- rust-lang#136892 (Sync Fuchsia target spec with clang Fuchsia driver)
- rust-lang#136911 (Add documentation URL to selected jobs)
- rust-lang#137870 ( Improve HashMap docs for const and static initializers)
- rust-lang#138179 (Add `src/tools/x` to the main workspace)
- rust-lang#138389 (use `expect` instead of `allow`)
- rust-lang#138396 (Enable metrics and verbose tests in PR CI)
- rust-lang#138398 (atomic intrinsics: clarify which types are supported and (if applicable) what happens with provenance)
- rust-lang#138432 (fix: remove the check of lld not supporting `@response-file)`
- rust-lang#138434 (Visit `PatField` when collecting lint levels)
- rust-lang#138441 (update error message)
- rust-lang#138442 (EUV: fix place of deref pattern's interior's scrutinee)
- rust-lang#138457 (Remove usage of legacy scheme paths on RedoxOS)
- rust-lang#138461 (Remove an outdated line from a test comment)
- rust-lang#138466 (Remove myself from libs review)
Failed merges:
- rust-lang#138452 (Remove `RUN_CHECK_WITH_PARALLEL_QUERIES`)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit f6fcae0 into rust-lang:masterMar 14, 2025
@rustbotrustbot added this to the 1.87.0 milestone Mar 14, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Mar 14, 2025
Rollup merge of rust-lang#138457 - bjorn3:redox_scheme_paths, r=Noratrieb
Remove usage of legacy scheme paths on RedoxOS
The `name:/path` path syntax is getting phased out[^1] in favor of `/scheme/name/path`. Also using `null:` is no longer necessary as `/dev/null` is available on Redox OS too.
[^1]: https://gitlab.redox-os.org/redox-os/rfcs/-/blob/master/text/0006-scheme-path.md
cc `@jackpot51`
@bjorn3
bjorn3 deleted the redox_scheme_paths branch March 18, 2025 09:22
github-actionsBot pushed a commit to model-checking/verify-rust-std that referenced this pull request Mar 19, 2025
…rieb
Remove usage of legacy scheme paths on RedoxOS
The `name:/path` path syntax is getting phased out[^1] in favor of `/scheme/name/path`. Also using `null:` is no longer necessary as `/dev/null` is available on Redox OS too.
[^1]: https://gitlab.redox-os.org/redox-os/rfcs/-/blob/master/text/0006-scheme-path.md
cc `@jackpot51`
github-actionsBot pushed a commit to model-checking/verify-rust-std that referenced this pull request Mar 19, 2025
Rollup of 16 pull requests
Successful merges:
- rust-lang#136001 (Overhaul examples for PermissionsExt)
- rust-lang#136230 (Reword incorrect documentation about SocketAddr having varying layout)
- rust-lang#136892 (Sync Fuchsia target spec with clang Fuchsia driver)
- rust-lang#136911 (Add documentation URL to selected jobs)
- rust-lang#137870 ( Improve HashMap docs for const and static initializers)
- rust-lang#138179 (Add `src/tools/x` to the main workspace)
- rust-lang#138389 (use `expect` instead of `allow`)
- rust-lang#138396 (Enable metrics and verbose tests in PR CI)
- rust-lang#138398 (atomic intrinsics: clarify which types are supported and (if applicable) what happens with provenance)
- rust-lang#138432 (fix: remove the check of lld not supporting `@response-file)`
- rust-lang#138434 (Visit `PatField` when collecting lint levels)
- rust-lang#138441 (update error message)
- rust-lang#138442 (EUV: fix place of deref pattern's interior's scrutinee)
- rust-lang#138457 (Remove usage of legacy scheme paths on RedoxOS)
- rust-lang#138461 (Remove an outdated line from a test comment)
- rust-lang#138466 (Remove myself from libs review)
Failed merges:
- rust-lang#138452 (Remove `RUN_CHECK_WITH_PARALLEL_QUERIES`)
r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

O-redoxOperating system: Redox, https://www.redox-os.org/O-unixOperating system: Unix-likeS-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@bjorn3@rustbot@Noratrieb@bors@jackpot51@ibraheemdev