Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow making RUSTC_BOOTSTRAP conditional on the crate name #77802

Merged
merged 1 commit into from
Nov 15, 2020

Commits on Nov 7, 2020

  1. Allow making RUSTC_BOOTSTRAP conditional on the crate name

    The main change is that `UnstableOptions::from_environment` now requires
    an (optional) crate name. If the crate name is unknown (`None`), then the new feature is not available and you still have to use `RUSTC_BOOTSTRAP=1`. In practice this means the feature is only available for `--crate-name`, not for `#![crate_name]`; I'm interested in supporting the second but I'm not sure how.
    
    Other major changes:
    
    - Added `Session::is_nightly_build()`, which uses the `crate_name` of
    the session
    - Added `nightly_options::match_is_nightly_build`, a convenience method
    for looking up `--crate-name` from CLI arguments.
    `Session::is_nightly_build()`should be preferred where possible, since
    it will take into account `#![crate_name]` (I think).
    - Added `unstable_features` to `rustdoc::RenderOptions`
    
      There is a user-facing change here: things like `RUSTC_BOOTSTRAP=0` no
      longer active nightly features. In practice this shouldn't be a big
      deal, since `RUSTC_BOOTSTRAP` is the opposite of stable and everyone
      uses `RUSTC_BOOTSTRAP=1` anyway.
    
    - Add tests
    
      Check against `Cheat`, not whether nightly features are allowed.
      Nightly features are always allowed on the nightly channel.
    
    - Only call `is_nightly_build()` once within a function
    
    - Use booleans consistently for rustc_incremental
    
      Sessions can't be passed through threads, so `read_file` couldn't take a
      session. To be consistent, also take a boolean in `write_file_header`.
    jyn514 committed Nov 7, 2020
    Configuration menu
    Copy the full SHA
    622c48e View commit details
    Browse the repository at this point in the history