-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Switch time
to jiff
for time formatting in ICE dumps
#139315
base: master
Are you sure you want to change the base?
Conversation
rustbot has assigned @Mark-Simulacrum. Use |
Some changes occurred in src/tools/clippy cc @rust-lang/clippy The Miri subtree was changed cc @rust-lang/miri rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead. cc @rust-lang/rust-analyzer These commits modify the If this was unintentional then you should revert the changes before this PR is merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's fine from the r-a side.
Maybe we should re-consider our use of this dependency if it breaks entirely unrelated code in this way. |
This comment has been minimized.
This comment has been minimized.
Currently, it's pulled in via |
bbd3069
to
15cf852
Compare
15cf852
to
c0856d4
Compare
The list of allowed third-party dependencies may have been modified! You must ensure that any new dependencies have compatible licenses before merging. |
I don't feel very confident about having our own bespoke formatting, so tried switching over to |
deranged
and fix breakagetime
to chrono
for time formatting in ICE dumps
It seems like a heavy weight dependency just for formatting rustc-ice files. Is there nothing else that can be used? |
c0856d4
to
9b8302d
Compare
time
to chrono
for time formatting in ICE dumpstime
to jiff
for time formatting in ICE dumps
Certainly seems more lightweight (and is already in use in the tree) |
FYI I am reconsidering whether to yank the release in question given the apparently larger scope of breakage than I initially assumed would happen. Keep an eye on jhpratt/deranged#21 for updates. I'll have a definitive answer in a day or two max (currently quite busy and am abroad). |
9b8302d
to
0c38dbe
Compare
0c38dbe
to
41fcdab
Compare
re. above, the relevant version was yanked, so version bumps for Rust should work as expected now |
Due to jhpratt/deranged#21, Clippy, R-A and Miri currently fail to build if we bump to 0.4.1, pulled in via
time
.Add some specific type annotations so we don't have to just pin it.I can open 3 PRs to the tool repos if preferred, but I thought it might be easier to do this than to pin the transitive dep and go back and remove it once the changes are synced back.