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

Readme contains link to the dead fork of rust #29

Open
Shchvova opened this issue Aug 24, 2018 · 8 comments
Open

Readme contains link to the dead fork of rust #29

Shchvova opened this issue Aug 24, 2018 · 8 comments

Comments

@Shchvova
Copy link

Shchvova commented Aug 24, 2018

link https://github.com/riscv-rust/rust is broken

Problem is this is first search result which comes up when searching for "Risc-V rust" please, fix it, because it seems like rust on Risc-V is dead and this thing is obsolete.

# OBSOLETE

This is no longer the recommended way to get a Rust toolchain for RISCV.
Instead, you can clone the [riscv-rust/rust](https://github.com/riscv-rust/rust)
fork of Rust and build it as normal:

    ./x.py build
@danc86
Copy link
Member

danc86 commented Aug 24, 2018

Oops yeah, we better fix that. It's even better than that nowadays, you can just rustup a nightly and you get working RISCV support. :-)

@Shchvova
Copy link
Author

Shchvova commented Aug 24, 2018

Please do. I tried to find how to do it, but couldn’t. RISC-V not even on https://forge.rust-lang.org/platform-support.html

@Shchvova
Copy link
Author

There’s rust issue rust-lang/rust#34333 about risc rupport, but basically it stalled into “not working”.
I suggest updating documentation and links because from what I could find I concluded that rust doesn’t support risc.

@dvc94ch
Copy link
Member

dvc94ch commented Aug 24, 2018

Please rustup target add riscv32imac-unknown-none-elf.

@danc86 Good thing I deleted the riscv-rust repo otherwise we'd be getting weekly reports from people trying to use it. Have you submitted a PR for advancing the rust lld fork? I think we can delete this repo too after to avoid confusion...

@danc86
Copy link
Member

danc86 commented Aug 24, 2018

Yep lld in rust is updated on master, although I think it hasn't made it into any nightly build yet...

@laysakura
Copy link

laysakura commented Jan 28, 2020

@dvc94ch @danc86 Are there still any concerns about deleting this repo?

This repo is pinned to https://github.com/riscv-rust and it even seems like this repo is one of the most important for The RISC-V team.

@danc86
Copy link
Member

danc86 commented Jan 29, 2020

I don't know if deleting the repo is a good idea, Github will let us mark the repo as "archived" which might be better. Then we won't break any existing historical links to the repo or old commits from it which might be referenced somewhere.

I think the README pretty clearly points people to the right place already, if they do happen to land here from a Google search or wherever.

Do you think we need to make the README more obvious somehow, before we mark the repo as archived?

@laysakura
Copy link

laysakura commented Jan 29, 2020

It's comfortable for me just if this repo is unpinned from https://github.com/riscv-rust .

I assume pinned repositories are kind of most importantly maintained ones.

If I were a maintainer of this repo, I would unpin and archive it.

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

No branches or pull requests

4 participants