mirror of
https://github.com/rust-lang/rust.git
synced 2025-06-04 19:29:07 +00:00
aadb5718dd
[breaking change] Validate crate name in `--extern` [MCP 650] Reject non-ASCII-identifier crate names passed to the CLI option `--extern` (`rustc`, `rustdoc`). Implements [MCP 650](https://github.com/rust-lang/compiler-team/issues/650) (except that we only allow ASCII identifiers not arbitrary Rust identifiers). Fixes #113035. [As mentioned on Zulip](https://rust-lang.zulipchat.com/#narrow/stream/233931-t-compiler.2Fmajor-changes/topic/Disallow.20non-identifier-valid.20--extern.20cr.E2.80.A6.20compiler-team.23650/near/376826988), doing a crater run probably doesn't make sense since it wouldn't yield anything. Most users don't interact with `rustc` directly but only ever through Cargo which always passes a valid crate name to `--extern` when it invokes `rustc` and `rustdoc`. In any case, the user wouldn't be able to use such a crate name in the source code anyway. Note that I'm not using [`rustc_session::output::validate_crate_name`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc_session/output/fn.validate_crate_name.html) (used for `--crate-name` and `#![crate_name]`) since the latter doesn't reject non-ASCII crate names and ones that start with a digit. As an aside, I've also thought about getting rid of `validate_crate_name` entirely in a separate PR (with another MCP) in favor of `is_ascii_ident` to reject more weird `--crate-name`s, `#![crate_name]`s and file names but I think that would lead to a lot of actual breakage, namely because of file names starting with a digit. In `tests/ui` 9 tests would be impacted for example. CC `@estebank` r? `@est31` |
||
|---|---|---|
| .. | ||
| rfc-0107-bind-by-move-pattern-guards | ||
| rfc-1014-stdout-existential-crisis | ||
| rfc-1445-restrict-constants-in-patterns | ||
| rfc-1623-static | ||
| rfc-1717-dllimport | ||
| rfc-1789-as-cell | ||
| rfc-1857-stabilize-drop-order | ||
| rfc-1937-termination-trait | ||
| rfc-2005-default-binding-mode | ||
| rfc-2008-non-exhaustive | ||
| rfc-2027-object-safe-for-dispatch | ||
| rfc-2091-track-caller | ||
| rfc-2093-infer-outlives | ||
| rfc-2126-crate-paths | ||
| rfc-2126-extern-absolute-paths | ||
| rfc-2151-raw-identifiers | ||
| rfc-2175-or-if-while-let | ||
| rfc-2294-if-let-guard | ||
| rfc-2302-self-struct-ctor | ||
| rfc-2306-convert-id | ||
| rfc-2361-dbg-macro | ||
| rfc-2396-target_feature-11 | ||
| rfc-2397-do-not-recommend | ||
| rfc-2421-unreserve-pure-offsetof-sizeof-alignof | ||
| rfc-2457-non-ascii-idents | ||
| rfc-2497-if-let-chains | ||
| rfc-2528-type-changing-struct-update | ||
| rfc-2565-param-attrs | ||
| rfc-2627-raw-dylib | ||
| rfc-2632-const-trait-impl | ||
| rfc-3348-c-string-literals | ||