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

Remove the rusti command #9818

Closed
thestinger opened this issue Oct 12, 2013 · 8 comments · Fixed by #9897
Closed

Remove the rusti command #9818

thestinger opened this issue Oct 12, 2013 · 8 comments · Fixed by #9897

Comments

@thestinger
Copy link
Contributor

It was very fragile and quirky when it did partially work, but at the moment it's completely broken. The tests are disabled, and I really doubt anyone is going to get it working for 1.0. There was a proposal to use fork to allow rolling back, but the runtime is not fork-safe and that strategy would prevent it from running on Windows which I don't think is acceptable.

@catamorphism
Copy link
Contributor

By the same logic as the rust tool, I agree. I'd be sadder about dropping rusti, but I just don't think we're going to be able to make it work.

@alexcrichton
Copy link
Member

It was decided in the weekly meeting today that we should drop the rusti command, updating the title to remove the word "consider"

@brson
Copy link
Contributor

brson commented Oct 16, 2013

Nominating.

@tautologico
Copy link
Contributor

Will the code be removed from the repo?

@alexcrichton
Copy link
Member

It'll still be in the git history, but yes, the src/librusti directory and all related makefile rules will be deleted.

@tautologico
Copy link
Contributor

I suppose it's easy to compile as an external project if someone decides to take over it? It seems to depend only on libextra, librustc and libsyntax.

@thestinger
Copy link
Contributor Author

@tautologico: it also depends on the linenoise library, and that will likely be dropped from the rust repository

bors added a commit that referenced this issue Oct 17, 2013
Closes #9818
Closes #9567
Closes #8924
Closes #8910
Closes #8392
Closes #7692
Closes #7499
Closes #7220
Closes #5038
flip1995 pushed a commit to flip1995/rust that referenced this issue Nov 21, 2022
@zzz6519003
Copy link

sad

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 a pull request may close this issue.

6 participants