Skip to content

Add armv7-unknown-linux-musleabihf host tools to CI - #142538

Open
alex wants to merge 1 commit into
rust-lang:mainfrom
alex:add-armv7-musl-host-tools
Open

alex wants to merge 1 commit into
rust-lang:mainfrom
alex:add-armv7-musl-host-tools

Conversation

@alex

@alex alex commented Jun 15, 2025 •

Copy link
Copy Markdown
Member

View all comments

Related to rust-lang/rustup#3398

try-job: dist-armv7-linux-musl
try-job: dist-various-1

@rustbot

rustbot commented Jun 15, 2025

Copy link
Copy Markdown
Collaborator

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
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

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Jun 15, 2025
@alex

alex commented Jun 15, 2025

Copy link
Copy Markdown
Member Author

I know there's a command to run a particular CI runner, but a) I don't know the syntax, b) not sure if I have permissions to do it?

@workingjubilee

Copy link
Copy Markdown
Member

Let's make sure you can do that:

@bors2 delegate=try

@alex Generally you edit your PR desc so it has

try-job: job

And then @ bors2 try

See

/// Tries to parse patterns of CI jobs that should be executed
/// from the commit message of the passed GitHub context
///
/// They can be specified in the form of
/// try-job: <job-pattern>
/// or
/// try-job: `<job-pattern>`
/// (to avoid GitHub rendering the glob patterns as Markdown)
fn get_try_job_patterns(&self) -> Vec<String> {

@rust-bors

rust-bors Bot commented Jun 16, 2025

Copy link
Copy Markdown
Contributor

@alex can now perform try builds on this pull request

@alex

alex commented Jun 16, 2025

Copy link
Copy Markdown
Member Author

@bors2 try

@rust-bors

rust-bors Bot commented Jun 16, 2025

Copy link
Copy Markdown
Contributor

⌛ Trying commit 0f7f1ba with merge f285d4c…

To cancel the try build, run the command @bors2 try cancel.

rust-bors Bot added a commit that referenced this pull request Jun 16, 2025
Add armv7-unknown-linux-musleabihf host tools to CI

Related to rust-lang/rustup#3398

try-job: `dist-armv7-linux-musl`
@rust-bors

rust-bors Bot commented Jun 16, 2025

Copy link
Copy Markdown
Contributor

💔 Test failed

@alex
alex force-pushed the add-armv7-musl-host-tools branch from 0f7f1ba to 9307946 Compare June 16, 2025 11:43
@alex

alex commented Jun 16, 2025

Copy link
Copy Markdown
Member Author

@bors2 try

@rust-bors

rust-bors Bot commented Jun 16, 2025

Copy link
Copy Markdown
Contributor

⌛ Trying commit 9307946 with merge cbfa86a…

To cancel the try build, run the command @bors2 try cancel.

rust-bors Bot added a commit that referenced this pull request Jun 16, 2025
Add armv7-unknown-linux-musleabihf host tools to CI

Related to rust-lang/rustup#3398

try-job: `dist-armv7-linux-musl`
@rust-bors

rust-bors Bot commented Jun 16, 2025

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: cbfa86a (cbfa86ae9d01dbbaf62d3c0f479a7fd7cfda2c14, parent: 68ac5abb067806a88464ddbfbd3c7eec877b488d)

Comment thread src/ci/docker/host-x86_64/dist-armv7-linux-musl/Dockerfile
@alex
alex force-pushed the add-armv7-musl-host-tools branch from 9307946 to b42987a Compare June 18, 2025 01:45
@Mark-Simulacrum

Copy link
Copy Markdown
Member

I think this needs to follow the process for promoting to tier 2 with host tools - https://doc.rust-lang.org/beta/rustc/target-tier-policy.html#tier-2-with-host-tools, are you preparing that elsewhere? Config here seems OK, though it should probably be removing the tier 2 build from the dist-various builder?

@rustbot author

@rustbot rustbot 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 Jun 21, 2025
@alex

alex commented Jun 22, 2025

Copy link
Copy Markdown
Member Author

Sure, I'm happy to fill out an MCP for this -- is there by chance a previous one you know of that I can use as a reference on the format/level of detail that's expected?

@alex
alex force-pushed the add-armv7-musl-host-tools branch from b42987a to af713a3 Compare June 22, 2025 15:01
@alex

alex commented Jun 22, 2025

Copy link
Copy Markdown
Member Author

@bors2 try

@rust-bors

rust-bors Bot commented Jun 22, 2025

Copy link
Copy Markdown
Contributor

⌛ Trying commit af713a3 with merge f3f4c61…

To cancel the try build, run the command @bors2 try cancel.

rust-bors Bot added a commit that referenced this pull request Jun 22, 2025
Add armv7-unknown-linux-musleabihf host tools to CI

Related to rust-lang/rustup#3398

try-job: `dist-armv7-linux-musl`
try-job: `dist-various`
@Mark-Simulacrum

Copy link
Copy Markdown
Member

You'd want to find the previous (if one exists) proposal for this target and copy/paste the target policy, filling out any item with details since the last uplift (to tier 2, if that one exists).

rust-lang/compiler-team#864 is a tier 2 uplift but is broadly the right shape.

@rust-bors

rust-bors Bot commented Jun 22, 2025

Copy link
Copy Markdown
Contributor

💔 Test failed

@alex

alex commented Jun 22, 2025

Copy link
Copy Markdown
Member Author

@bors2 try

@rust-bors

rust-bors Bot commented Jun 22, 2025

Copy link
Copy Markdown
Contributor

⌛ Trying commit af713a3 with merge 1bf1316…

To cancel the try build, run the command @bors2 try cancel.

rust-bors Bot added a commit that referenced this pull request Jun 22, 2025
Add armv7-unknown-linux-musleabihf host tools to CI

Related to rust-lang/rustup#3398

try-job: `dist-armv7-linux-musl`
try-job: `dist-various-1`
@alex

alex commented Jun 22, 2025

Copy link
Copy Markdown
Member Author

Proposed at rust-lang/compiler-team#888, thanks for the guidance.

@rust-bors

rust-bors Bot commented Jun 22, 2025

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 1bf1316 (1bf13165b06393e0b1c3b1d3beca8520afc7ba31, parent: a30f1783fe136d92545423dd30b12eb619973cdb)

@Dylan-DPC Dylan-DPC added S-waiting-on-MCP Status: PR has a compiler MCP and is waiting for the compiler MCP to complete. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 24, 2025
@rust-bors

rust-bors Bot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

☔ The latest upstream changes (presumably #154008) made this pull request unmergeable. Please resolve the merge conflicts.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-MCP Status: PR has a compiler MCP and is waiting for the compiler MCP to complete. T-infra Relevant to the infrastructure 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