nordic-dev.net/rust
nordic-dev.net/rust
2
0
Fork 0
mirror of https://github.com/rust-lang/rust.git synced 2025-05-07 07:27:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

rustc: Forbid -Z flags on stable/beta channels

Browse Source 
First deprecated in rustc 1.8.0 the intention was to never allow `-Z` flags make
their way to the stable channel (or unstable options). After a year of warnings
we've seen one of the main use cases, `-Z no-trans`, stabilized as `cargo
check`. Otherwise while other use cases remain the sentiment is that now's the
time to start forbidding `-Z` by default on stable/beta.

Closes #31847
This commit is contained in:
Alex Crichton 2017-05-04 09:34:44 -07:00
parent 222971f7d2
commit ccbcc720a6
1 changed files with 16 additions and 68 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

84
src/librustc/session/config.rs
View File

@ -1122,14 +1122,6 @@ pub fn build_target_config(opts: &Options, sp: &Handler) -> Config {
pub enum OptionStability { pub enum OptionStability {
Stable, Stable,
// FIXME: historically there were some options which were either `-Z` or
// required the `-Z unstable-options` flag, which were all intended
// to be unstable. Unfortunately we didn't actually gate usage of
// these options on the stable compiler, so we still allow them there
// today. There are some warnings printed out about this in the
// driver.
UnstableButNotReally,
Unstable, Unstable,
} }
@ -1148,17 +1140,9 @@ impl RustcOptGroup {
RustcOptGroup { opt_group: g, stability: OptionStability::Stable } RustcOptGroup { opt_group: g, stability: OptionStability::Stable }
} }
#[allow(dead_code)] // currently we have no "truly unstable" options
pub fn unstable(g: getopts::OptGroup) -> RustcOptGroup { pub fn unstable(g: getopts::OptGroup) -> RustcOptGroup {
RustcOptGroup { opt_group: g, stability: OptionStability::Unstable } RustcOptGroup { opt_group: g, stability: OptionStability::Unstable }
} }
fn unstable_bnr(g: getopts::OptGroup) -> RustcOptGroup {
RustcOptGroup {
opt_group: g,
stability: OptionStability::UnstableButNotReally,
}
}
} }
// The `opt` local module holds wrappers around the `getopts` API that // The `opt` local module holds wrappers around the `getopts` API that
@ -1180,7 +1164,6 @@ mod opt {
fn stable(g: getopts::OptGroup) -> R { RustcOptGroup::stable(g) } fn stable(g: getopts::OptGroup) -> R { RustcOptGroup::stable(g) }
fn unstable(g: getopts::OptGroup) -> R { RustcOptGroup::unstable(g) } fn unstable(g: getopts::OptGroup) -> R { RustcOptGroup::unstable(g) }
fn unstable_bnr(g: getopts::OptGroup) -> R { RustcOptGroup::unstable_bnr(g) }
pub fn opt_s(a: S, b: S, c: S, d: S) -> R { pub fn opt_s(a: S, b: S, c: S, d: S) -> R {
stable(getopts::optopt(a, b, c, d)) stable(getopts::optopt(a, b, c, d))
@ -1213,24 +1196,6 @@ mod opt {
pub fn flagmulti(a: S, b: S, c: S) -> R { pub fn flagmulti(a: S, b: S, c: S) -> R {
unstable(getopts::optflagmulti(a, b, c)) unstable(getopts::optflagmulti(a, b, c))
} }
// Do not use these functions for any new options added to the compiler, all
// new options should use the `*_u` variants above to be truly unstable.
pub fn opt_ubnr(a: S, b: S, c: S, d: S) -> R {
unstable_bnr(getopts::optopt(a, b, c, d))
}
pub fn multi_ubnr(a: S, b: S, c: S, d: S) -> R {
unstable_bnr(getopts::optmulti(a, b, c, d))
}
pub fn flag_ubnr(a: S, b: S, c: S) -> R {
unstable_bnr(getopts::optflag(a, b, c))
}
pub fn flagopt_ubnr(a: S, b: S, c: S, d: S) -> R {
unstable_bnr(getopts::optflagopt(a, b, c, d))
}
pub fn flagmulti_ubnr(a: S, b: S, c: S) -> R {
unstable_bnr(getopts::optflagmulti(a, b, c))
}
} }
/// Returns the "short" subset of the rustc command line options, /// Returns the "short" subset of the rustc command line options,
@ -1296,7 +1261,7 @@ pub fn rustc_optgroups() -> Vec<RustcOptGroup> {
opt::multi_s("", "extern", "Specify where an external rust library is located", opt::multi_s("", "extern", "Specify where an external rust library is located",
"NAME=PATH"), "NAME=PATH"),
opt::opt_s("", "sysroot", "Override the system root", "PATH"), opt::opt_s("", "sysroot", "Override the system root", "PATH"),
opt::multi_ubnr("Z", "", "Set internal debugging options", "FLAG"), opt::multi("Z", "", "Set internal debugging options", "FLAG"),
opt::opt_s("", "error-format", opt::opt_s("", "error-format",
"How errors and other messages are produced", "How errors and other messages are produced",
"human|json"), "human|json"),
@ -1305,28 +1270,20 @@ pub fn rustc_optgroups() -> Vec<RustcOptGroup> {
always = always colorize output; always = always colorize output;
never = never colorize output", "auto|always|never"), never = never colorize output", "auto|always|never"),
opt::flagopt_ubnr("", "pretty", opt::flagopt("", "pretty",
"Pretty-print the input instead of compiling; "Pretty-print the input instead of compiling;
valid types are: `normal` (un-annotated source), valid types are: `normal` (un-annotated source),
`expanded` (crates expanded), or `expanded` (crates expanded), or
`expanded,identified` (fully parenthesized, AST nodes with IDs).", `expanded,identified` (fully parenthesized, AST nodes with IDs).",
"TYPE"), "TYPE"),
opt::flagopt_ubnr("", "unpretty", opt::flagopt("", "unpretty",
"Present the input source, unstable (and less-pretty) variants; "Present the input source, unstable (and less-pretty) variants;
valid types are any of the types for `--pretty`, as well as: valid types are any of the types for `--pretty`, as well as:
`flowgraph=<nodeid>` (graphviz formatted flowgraph for node), `flowgraph=<nodeid>` (graphviz formatted flowgraph for node),
`everybody_loops` (all function bodies replaced with `loop {}`), `everybody_loops` (all function bodies replaced with `loop {}`),
`hir` (the HIR), `hir,identified`, or `hir` (the HIR), `hir,identified`, or
`hir,typed` (HIR with types for each node).", `hir,typed` (HIR with types for each node).",
"TYPE"), "TYPE"),
// new options here should **not** use the `_ubnr` functions, all new
// unstable options should use the short variants to indicate that they
// are truly unstable. All `_ubnr` flags are just that way because they
// were so historically.
//
// You may also wish to keep this comment at the bottom of this list to
// ensure that others see it.
]); ]);
opts opts
} }
@ -1704,7 +1661,7 @@ pub mod nightly_options {
use getopts; use getopts;
use syntax::feature_gate::UnstableFeatures; use syntax::feature_gate::UnstableFeatures;
use super::{ErrorOutputType, OptionStability, RustcOptGroup}; use super::{ErrorOutputType, OptionStability, RustcOptGroup};
use session::{early_error, early_warn}; use session::early_error;
pub fn is_unstable_enabled(matches: &getopts::Matches) -> bool { pub fn is_unstable_enabled(matches: &getopts::Matches) -> bool {
is_nightly_build() && matches.opt_strs("Z").iter().any(|x| *x == "unstable-options") is_nightly_build() && matches.opt_strs("Z").iter().any(|x| *x == "unstable-options")
@ -1746,15 +1703,6 @@ pub mod nightly_options {
nightly compiler", opt_name); nightly compiler", opt_name);
early_error(ErrorOutputType::default(), &msg); early_error(ErrorOutputType::default(), &msg);
} }
OptionStability::UnstableButNotReally => {
let msg = format!("the option `{}` is unstable and should \
only be used on the nightly compiler, but \
it is currently accepted for backwards \
compatibility; this will soon change, \
see issue #31847 for more details",
opt_name);
early_warn(ErrorOutputType::default(), &msg);
}
OptionStability::Stable => {} OptionStability::Stable => {}
} }
} }