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

chore: fix warnings on nightly #558

Merged
merged 3 commits into from
Feb 2, 2020

Conversation

davidbarsky
Copy link
Member

This PR fixes two types of warnings:

  1. Usage of deprecated methods on std::error::Error.
  2. An unused warning on extern crate proc_macro in tracing-attributes. This does not occur on the stable release of 1.41, but it does occur on 1.42.0-beta.1 and 1.42.0-nightly (cd1ef390e 2020-01-31).

Prior to merging this PR, I'd like to determine whether the second warning is a deliberate change or a regression.

@davidbarsky davidbarsky requested a review from a team February 2, 2020 18:26
@davidbarsky
Copy link
Member Author

It appears to be deliberate; I'm now able to build tracing-attributes without extern crate proc_macro on beta and nightly.

@davidbarsky
Copy link
Member Author

@mystor informed me that this is a deliberate change in Cargo (rust-lang/cargo#7700). Thanks!

Copy link
Member

@hawkw hawkw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved modulo some nits

tracing-subscriber/src/filter/env/mod.rs Outdated Show resolved Hide resolved
tracing-subscriber/src/reload.rs Outdated Show resolved Hide resolved
@davidbarsky davidbarsky merged commit 3c80984 into master Feb 2, 2020
@davidbarsky davidbarsky deleted the david/fix-error-deprecation-warnings branch February 2, 2020 22:42
hawkw added a commit that referenced this pull request Feb 3, 2020
Fixed

- **fmt**: Fixed empty `{}` printed after spans with no fields (f079f2d)
- **fmt**: Fixed inconsistent formatting when ANSI colors are disabled (506a482)
- **fmt**: Fixed mis-aligned levels when ANSI colors are disabled (eba1adb)
- Fixed warnings on nightly Rust compilers (#558)

Signed-off-by: Eliza Weisman <eliza@buoyant.io>
hawkw added a commit that referenced this pull request Feb 3, 2020
Fixed

- **fmt**: Fixed empty `{}` printed after spans with no fields (f079f2d)
- **fmt**: Fixed inconsistent formatting when ANSI colors are disabled (506a482)
- **fmt**: Fixed mis-aligned levels when ANSI colors are disabled (eba1adb)
- Fixed warnings on nightly Rust compilers (#558)

Signed-off-by: Eliza Weisman <eliza@buoyant.io>
hawkw added a commit that referenced this pull request Feb 3, 2020
Fixed

- **fmt**: Fixed empty `{}` printed after spans with no fields (f079f2d)
- **fmt**: Fixed inconsistent formatting when ANSI colors are disabled (506a482)
- **fmt**: Fixed mis-aligned levels when ANSI colors are disabled (eba1adb)
- Fixed warnings on nightly Rust compilers (#558)

Signed-off-by: Eliza Weisman <eliza@buoyant.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants