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

Rollup of 8 pull requests #91911

Closed
wants to merge 17 commits into from

Commits on Dec 12, 2021

  1. Avoid suggesting self in visibility spec

    Co-authored-by: Esteban Kuber <estebank@users.noreply.github.com>
    bugadani and estebank committed Dec 12, 2021
    Configuration menu
    Copy the full SHA
    f3a08fd View commit details
    Browse the repository at this point in the history

Commits on Dec 13, 2021

  1. Revert "Set MACOSX_DEPLOYMENT_TARGET env var to default for linking i…

    …f not set."
    
    This reverts commit b376f56, which is
    the main part of rust-lang#90499, because it turns out that this causes a good
    amount of breakage in crates relying on the old behavior.
    
    Fixes rust-lang#91372.
    hkratz committed Dec 13, 2021
    Configuration menu
    Copy the full SHA
    3011154 View commit details
    Browse the repository at this point in the history

Commits on Dec 14, 2021

  1. Configuration menu
    Copy the full SHA
    ae21dd0 View commit details
    Browse the repository at this point in the history
  2. Remove in_band_lifetimes from rustc_trait_selection

    Sylvan Bowdler committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    1ea756b View commit details
    Browse the repository at this point in the history
  3. made compiler happy

    lameferret committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    90aa8fb View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    97e844a View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    a586e7d View commit details
    Browse the repository at this point in the history
  6. Add 2 tests

    matthiaskrgr committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    f58ef22 View commit details
    Browse the repository at this point in the history
  7. ⬆️ rust-analyzer

    lnicola committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    c771970 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#91798 - bugadani:issue-91783, r=michaelwoer…

    …ister
    
    Avoid suggest adding `self` in visibility spec
    
    Fixes rust-lang#91783
    matthiaskrgr authored Dec 14, 2021
    Configuration menu
    Copy the full SHA
    f8518d5 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#91870 - rusticstuff:macosx_min_version_reve…

    …rt, r=Mark-Simulacrum
    
    Revert setting a default for the MACOSX_DEPLOYMENT_TARGET env var for linking
    
    This reverts commit b376f56, which is the main part of rust-lang#90499, because it turns out that this causes a good amount of breakage in crates relying on the old behavior. In particular `winit`, `coreaudio` and crates that depend on them are affected. Fixes rust-lang#91372.
    
    Background:
    Before rust-lang#90499 the behavior was the following: If MACOSX_DEPLOYMENT_TARGET is not set,  we pass the minimum supported OS version to LLVM but not to the linker. The linker default depends on the Xcode version and the version of the OS it is running on. That caused one known problem in libcurl with the most recent Xcode versions. rust-lang#90499 passed the minumum supported version (10.7 for Macos x86-64) to the linker instead. This has shown to be problematic because some crates such as winit, coreaudio implicitly expect a newer minimum OS version. The libcurl issue has been fixed independently (see alexcrichton/curl-rust#417), so a revert should not really be problematic.
    
    Eventually we should probably mimic clang's behavior and fall back to the default of the currently configured Macos SDK for both the LLVM min os target version and MACOSX_DEPLOYMENT_TARGET for linking. That would entail looking at the `Version` property of the `SDKSettings.json` in the currently configured SDK.
    matthiaskrgr authored Dec 14, 2021
    Configuration menu
    Copy the full SHA
    9f931ee View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#91880 - matthiaskrgr:clippy_perf_dec, r=jyn514

    fix clippy::single_char_pattern perf findings
    matthiaskrgr authored Dec 14, 2021
    Configuration menu
    Copy the full SHA
    d9f11aa View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#91882 - Patrick-Poitras:remove-in-band-life…

    …times-from-rustc-typeck, r=jackh726
    
    Remove `in_band_lifetimes` from `rustc_typeck`
    
    Joining in on the effort to remove the `in_band_lifetimes` features, as described in issue rust-lang#91867.
    matthiaskrgr authored Dec 14, 2021
    Configuration menu
    Copy the full SHA
    00cd222 View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#91904 - SylvanB:remove_in_band_lifetimes_ru…

    …stc_trait_selection, r=petrochenkov
    
    Remove `in_band_lifetimes` from `rustc_trait_selection`
    
    Another one for rust-lang#91867
    matthiaskrgr authored Dec 14, 2021
    Configuration menu
    Copy the full SHA
    d1fe30d View commit details
    Browse the repository at this point in the history
  13. Rollup merge of rust-lang#91906 - anuvratsingh:remove_in_band_lifetim…

    …es_library_proc_macro, r=petrochenkov
    
    Removed `in_band_lifetimes` from `library\proc_macro`
    
    Issue [rust-lang#91867](rust-lang#91867)
    
    This is my first try, I followed the instructions given. Fixed all the errors that were thrown while compiling.
    Compiled with stage 0,1, and 2 all of them compiled successfully.
    matthiaskrgr authored Dec 14, 2021
    Configuration menu
    Copy the full SHA
    d24b0cc View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    063da9f View commit details
    Browse the repository at this point in the history
  15. Rollup merge of rust-lang#91909 - lnicola:rust-analyzer-2021-12-14, r…

    …=lnicola
    
    ⬆️ rust-analyzer
    
    r? `@ghost`
    matthiaskrgr authored Dec 14, 2021
    Configuration menu
    Copy the full SHA
    7f89718 View commit details
    Browse the repository at this point in the history