-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Upgrade env_logger #47501
Upgrade env_logger #47501
Conversation
r? @estebank (rust_highfive has picked a reviewer for you, use r? to override) |
|
my hero |
Updated! |
501c848
to
16db048
Compare
Urgh. By pulling in winapi 0.3 this'll be blocked on #47280. I wonder though if that could be avoided for now? |
I think the winapi 0.3 problem won't affect this PR since cargo is not updated. The winapi dependency chain is:
If we downgrade |
@kennytm unfortunately it was a problem if winapi is anywhere in the crate dependency graph, not just as a direct dependency :( |
We could manually downgrade time to a version that pulled in winapi 0.2. It might be a bit fragile to maintain though. |
@alexcrichton Oh that's bad :( So we need to stick a |
@kennytm we could have a constraint or just use |
Updated. (If you find yourself needing to do this again for whatever reason, the magic phrase is |
@bors: r+ |
📌 Commit da00ea3 has been approved by |
@bors rollup |
@bors r- This PR brings in I believe this is because hoedown/pulldown doesn't format to the same HTML.
cc @GuillaumeGomez. |
Now blocked by either:
|
☔ The latest upstream changes (presumably #47678) made this pull request unmergeable. Please resolve the merge conflicts. |
|
@kennytm oh note that the update there was just for Cargo not for rustc, so we may still need this for rustc itself? |
Err sorry @alexcrichton you're right, missed the rest of the changes. Still this needs a rebase, and #47501 (comment). |
I'm gonna close this in favor of #48642 in the hopes that it gets through |
This restores module names to the logging output