Matt Harding
Matt Harding
Okay, I've rebased the old patch. We're probably not ready to merge yet though, as we were still discussing and iterating on what we wanted to do. I'm going to...
Okay, I've spent some more time thinking about the output format, incorporating some other's ideas, and this is what I've come up with: ``` $ rustup show Default host: x86_64-pc-windows-msvc...
> I still prefer using `active because:` here. Sorry, do you mean that you want it to be on the same line as the header next to `active toolchain`, or...
Ah, gotcha. Yeah, I think you're version looks quite clean!
Okay, I'm working on this. Here's a thought I had: Should we always have the `installed toolchains` and `active toolchain -> installed targets` lists, even if they have only a...
I've uploaded the new patch, which includes the changes and a bit of a refactoring: 1. Change the output format of `rustup show`, to be in a more logical order...
Ok, I've split out as best as I can. The first one is still quite large, but I can't really split it much further without meticulously reimplementing the bugs that...
Looks like a random test failure. Can we run it again?
No worries, hope you enjoyed your break! I've addressed your comments and uploaded the new version.
Ah sorry, there's always something. Should be fixed now.