Compile UI tests as libraries by default #842
Labels
major-change
A proposal to make a major change to rustc
T-compiler
Add this label so rfcbot knows to poll the compiler team
Proposal
I propose to compile UI tests as libraries by default. I.e. compiletest will add
--crate-type=lib
by default, unless it's arun-pass
/run-fail
test (or other test type that requires a binary).We could also add
//@ crate-type [...]
annotation to override this if need be.The upside of compiling as libraries by default is that we will no longer need to workaround "missing main function" by adding
fn main() {}
in every test.While it's not much, it's still useless noise that we also have to write every time.
Mentors or Reviewers
@jieyouxu would you like to review the compiletest changes?
For ui tests themselves literally anyone can review it, since it will just be
fn main() {}
deletion.Process
The main points of the Major Change Process are as follows:
@rustbot second
.-C flag
, then full team check-off is required.@rfcbot fcp merge
on either the MCP or the PR.You can read more about Major Change Proposals on forge.
The text was updated successfully, but these errors were encountered: