This repository has been archived by the owner. It is now read-only.

rustfmt is disabled #216

Closed
alexcrichton opened this Issue Jun 20, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@alexcrichton
Copy link
Member

alexcrichton commented Jun 20, 2016

Right now there's a failure to cargo install rustfmt in the "container" due to SSL issues, so I've disabled it for now.

@shepmaster

This comment has been minimized.

Copy link
Member

shepmaster commented Jun 20, 2016

This is, hands down, the single largest feature I have for the playground (I have to fix a lot of poorly formatted code on Stack Overflow). Is there anything I can do to help get this resurrected?

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Jun 21, 2016

Oh dear, sorry about that! I hope to merge #187 soon and then switch from playpen to docker and solve #134. Moving should allow us to move to a much more familiar environment (ubuntu + docker) instead of a foreign one we don't know how to debug or work with (arch + playpen).

I'm honestly not sure why this isn't working today in Arch but for me at least it's easier to just switch to docker and shouldn't be too hard. I hope to get around to it this week!

@shepmaster

This comment has been minimized.

Copy link
Member

shepmaster commented Jun 21, 2016

Well, maybe it will be a race! I decided to rewrite the Playground from scratch for funsies, and also decided to use docker. Although I think you might win 😸

@nrc

This comment has been minimized.

Copy link
Member

nrc commented Jun 21, 2016

Could we just pull rustfmt master and build it? Not sure if that would have the same SSL issues or not.

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Jun 21, 2016

Unfortunately that'd still require hitting the registry to download deps, and the SSL there is what's failing :(

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Jun 22, 2016

Ok, docker is now deployed along with the Rust rewrite, should be fixed now!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.