Updating readme instructions #489
Conversation
|
Issue number #252 |
Readme.md
Outdated
| To check if all is working correctly, run ``./y.sh cargo build --manifest-path tests/hello-world/Cargo.toml`` in your terminal. | ||
|
|
||
|
|
||
| ### Install rustup if need it |
There was a problem hiding this comment.
I think this information should be at the top.
There was a problem hiding this comment.
But first we have to run the prepare and build commands as i understand. That is the reason that is not on top of that section.
There was a problem hiding this comment.
Euh... You need to rust to build any y.sh command.
There was a problem hiding this comment.
When you say at the top, which section do you mean?
There was a problem hiding this comment.
Euh... You need to rust to build any
y.shcommand.
Sorry, at the begining I understood something different.
There was a problem hiding this comment.
Thanks, that's a nice improvement.
Waiting to see what @GuillaumeGomez says for the ./y.sh prepare stuff.
Readme.md
Outdated
|
|
||
| ```bash | ||
| export CG_GCCJIT_DIR=[the full path to rustc_codegen_gcc] | ||
| $ ./y.sh prepare | ||
| $ ./y.sh build |
There was a problem hiding this comment.
Oh, I missed that. There was an update in my fork that I just synced and this now needs to be:
| $ ./y.sh build | |
| $ ./y.sh build --sysroot |
|
Thanks for your contribution! |
It's a pleasure :) |
Updating some sections inside of the readme to make it easier to follow.