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.

Sign up for GitHub

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

Jump to bottom

Tracking issue for future-incompatibility lint soft_unstable #64266

Open
petrochenkov opened this issue Sep 7, 2019 · 3 comments
Open

Tracking issue for future-incompatibility lint soft_unstable #64266

petrochenkov opened this issue Sep 7, 2019 · 3 comments
Labels
A-lint Area: Lints (warnings about flaws in source code) such as unused_mut. C-future-incompatibility Category: Future-incompatibility lints C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-lang Relevant to the language team, which will review and decide on the PR/issue.

Comments

@petrochenkov
Copy link
Contributor

petrochenkov commented Sep 7, 2019 •
edited
Loading

What is this lint about

When some feature is unstable, but was allowed on stable due to some stability checking hole or oversight we may unstabilize it in a "soft" way to avoid breaking crates depending on the crates using the feature.

This soft unstabilization can be done using this lint.

How to fix this warning/error

Remove or cfg-out uses of the unstable feature in configurations of your code that are supposed to build on stable.

Current status
@petrochenkov petrochenkov added A-lint Area: Lints (warnings about flaws in source code) such as unused_mut. C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. labels Sep 7, 2019
@Centril Centril added C-future-incompatibility Category: Future-incompatibility lints T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-lang Relevant to the language team, which will review and decide on the PR/issue. and removed C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. labels Sep 7, 2019
@lqd lqd mentioned this issue Oct 4, 2019
Closed
@lqd
Copy link
Member

lqd commented Oct 4, 2019

#[deny(soft_unstable)] is on by default, but the error message (playground) still has a lint-like note warning mentioning it will become a hard error in the future:

= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
= note: for more information, see issue #64266 <https://github.com/rust-lang/rust/issues/64266>

do we need an issue for this ?

This was referenced Jan 5, 2020
Merged
Merged
Centril added a commit to Centril/rust that referenced this issue Jan 11, 2020
@Centril
Rollup merge of rust-lang#68018 - petrochenkov:nosoft, r=Centril
71a719c 
feature_gate: Remove `GateStrength`

The "soft feature gating" from `feature_gate/check.rs` is unused, and even if it were used, hardcoded warning is not a good solution and [deny-by-default lint](rust-lang#64266) would be a better way to do this.

cc rust-lang#67806 (comment)
r? @Centril
Centril added a commit to Centril/rust that referenced this issue Jan 11, 2020
@Centril
Rollup merge of rust-lang#68018 - petrochenkov:nosoft, r=Centril
31d7ffa 
feature_gate: Remove `GateStrength`

The "soft feature gating" from `feature_gate/check.rs` is unused, and even if it were used, hardcoded warning is not a good solution and [deny-by-default lint](rust-lang#64266) would be a better way to do this.

cc rust-lang#67806 (comment)
r? @Centril
@ehuss ehuss mentioned this issue Feb 26, 2020
Closed
apyrgio added a commit to apyrgio/tindercrypt that referenced this issue Mar 17, 2021
@apyrgio
Temporarily silence a Rust error
c11c9ed 
The generated Rust code from our proto files triggers a warning in Rust
nightly, which we treat as an error:

    error: custom inner attributes are unstable
    Error:  --> src/../proto/metadata.rs:9:4
      |
    9 | #![rustfmt::skip]
      |    ^^^^^^^^^^^^^
      |
      = note: `#[deny(soft_unstable)]` on by default
      = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
      = note: for more information, see issue #64266 <rust-lang/rust#64266>

This is already reported in the `rust-protobuf` repo [1] so until it's
fixed, we choose to silence this warning to make our builds work.

[1]: stepancheg/rust-protobuf#551
apyrgio added a commit to apyrgio/tindercrypt that referenced this issue Mar 17, 2021
@apyrgio
Temporarily silence a Rust error
3886704 
The generated Rust code from our proto files triggers a warning in Rust
nightly, which we treat as an error:

    error: custom inner attributes are unstable
    Error:  --> src/../proto/metadata.rs:9:4
      |
    9 | #![rustfmt::skip]
      |    ^^^^^^^^^^^^^
      |
      = note: `#[deny(soft_unstable)]` on by default
      = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
      = note: for more information, see issue #64266 <rust-lang/rust#64266>

This is already reported in the `rust-protobuf` repo [1] so until it's
fixed, we choose to silence this warning to make our builds work.

[1]: stepancheg/rust-protobuf#551
@Farenjihn Farenjihn mentioned this issue Mar 23, 2021
Closed
scottlamb added a commit to scottlamb/moonfire-nvr that referenced this issue Mar 24, 2021
@scottlamb
build cleanly with nightly Rust 2021-03-14+
6fd4f89 
The CI nightly builds had been broken with the following error:

```
error: custom inner attributes are unstable
  --> /home/runner/work/moonfire-nvr/moonfire-nvr/server/target/debug/build/moonfire-db-415ce696a754c614/out/schema.rs:10:4
   |
10 | #![rustfmt::skip]
   |    ^^^^^^^^^^^^^
   |
   = note: `#[deny(soft_unstable)]` on by default
   = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
   = note: for more information, see issue #64266 <rust-lang/rust#64266>
```

I'd thought this was by mistake given that #[rustfmt::skip] is still
advertised on rustfmt's github page, but maybe not. Looks like
rust-protobuf's newest version uses
`#![cfg_attr(rustfmt, rustfmt::skip)]` to avoid this error.

Also fix a warning on nightly about an extraneous semicolon.
@luiswirth luiswirth mentioned this issue Apr 18, 2021
Merged
@svanharmelen svanharmelen mentioned this issue Jun 24, 2021
Merged
@Stargateur
Copy link
Contributor

Stargateur commented Aug 31, 2021 •
edited
Loading

So this feature is used to make breaking change to avoid breaking change ? The concept of solf_unstable don't make any sense for me, specially if #[deny(soft_unstable)] is default. It's just a more complicate way that produce the same result. Or I'm missing something.

#[stability(level=2.33333333, wind-strenght=5.3)] (not from me, author probably wishes to stay in shadow)

@jianshu93 jianshu93 mentioned this issue Feb 15, 2022
Open
@yaahc
Copy link
Member

yaahc commented May 16, 2022

I think I might need to use this for #95956 as well

@winst0nsmith winst0nsmith mentioned this issue May 19, 2022
Open
@cafkafk cafkafk mentioned this issue Sep 27, 2023
Merged
cafkafk added a commit to eza-community/eza that referenced this issue Sep 27, 2023
@cafkafk
fix(rustfmt): avoid unstable inner attributes
8835021 
See: rust-lang/rust#64266
Signed-off-by: Christina Sørensen <christina@cafkafk.com>
Omnikron13 pushed a commit to Omnikron13/eza that referenced this issue Mar 17, 2024
@cafkafk @Omnikron13
fix(rustfmt): avoid unstable inner attributes
a8ffafd 
See: rust-lang/rust#64266
Signed-off-by: Christina Sørensen <christina@cafkafk.com>
@github-actions github-actions bot mentioned this issue Apr 9, 2024
Merged
@asomers asomers mentioned this issue May 26, 2024
Closed
@fmease fmease changed the title Tracking issue for soft_unstable compatibility lint Tracking issue for future-incompatibility lint soft_unstable Sep 14, 2024
@fmease fmease added the C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. label Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-lint Area: Lints (warnings about flaws in source code) such as unused_mut. C-future-incompatibility Category: Future-incompatibility lints C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-lang Relevant to the language team, which will review and decide on the PR/issue.
Projects
Lang Edition 2024
Archived in project
Milestone
No milestone
Development

No branches or pull requests
6 participants
@lqd @Centril @yaahc @petrochenkov @Stargateur @fmease