Skip to content

Conversation

@Mark-Simulacrum
Copy link
Member

The goal of this PR is to just get the publishing working in a way that'll hopefully let clippy consume the resulting binary without too much trouble. If there's a desire to use compiletest as a library rather than a binary, I see no reason why we couldn't publish an rlib/.so/whatever, just like we do for std or the rustc libraries, that'll just take more plumbing. My hope is that the binary is a sufficient interface for now, since that works for rustc itself (bootstrap calls the binary), but if not, we can modify this PR to publish an rlib into the sysroot too, when the compiletest component is installed.

It looks like as-is clippy does use the library compiletest-rs as a library for its testing, I would need to do more digging to tell if that's an intrinsic dependency or not.

cc @Manishearth @oli-obk rust-lang/compiler-team#536

r? @jyn514

@rustbot rustbot added the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label Jul 30, 2022
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 30, 2022
@Mark-Simulacrum
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jul 30, 2022

⌛ Trying commit 6d7529751460ac93bcdf0b62122bc299e0d24060 with merge 9834b0310d8ad7a4fbca6ee673ffea9161757275...

@jyn514
Copy link
Member

jyn514 commented Jul 30, 2022

I think we should hold off on merging this until we confirm with the clippy team this is the approach they want to take. No objection to doing this, just don't want it to be wasted work.

@Mark-Simulacrum
Copy link
Member Author

Sure, that's fine with me -- in particular clarity on library vs. binary I think would be good.

Mostly wanted to open this so we can get a try build + a "fake" nightly published from it so it's easier to experiment with (for example) a clippy PR migrating to this.

@bors
Copy link
Collaborator

bors commented Jul 31, 2022

☀️ Try build successful - checks-actions
Build commit: 9834b0310d8ad7a4fbca6ee673ffea9161757275 (9834b0310d8ad7a4fbca6ee673ffea9161757275)

@Mark-Simulacrum
Copy link
Member Author

@bors try

@bors
Copy link
Collaborator

bors commented Jul 31, 2022

⌛ Trying commit 9223b0c with merge 1a32bcf2c3a5c2b33a30b93b1228eb9eb7ca6a9f...

@Manishearth
Copy link
Member

cc @rust-lang/clippy

@bors
Copy link
Collaborator

bors commented Jul 31, 2022

☀️ Try build successful - checks-actions
Build commit: 1a32bcf2c3a5c2b33a30b93b1228eb9eb7ca6a9f (1a32bcf2c3a5c2b33a30b93b1228eb9eb7ca6a9f)

@jyn514 jyn514 added S-waiting-on-team DEPRECATED: Use the team-based variants `S-waiting-on-t-lang`, `S-waiting-on-t-compiler`, ... A-clippy and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 31, 2022
@apiraino apiraino added the T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. label Aug 3, 2022
@fmease fmease added the T-clippy Relevant to the Clippy team. label Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-team DEPRECATED: Use the team-based variants `S-waiting-on-t-lang`, `S-waiting-on-t-compiler`, ... T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-clippy Relevant to the Clippy team. 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.

8 participants