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

Add a paragraph on installing ninja-build in order to build LLVM #1209

Closed
wants to merge 1 commit into from

Conversation

daira
Copy link

@daira daira commented Sep 19, 2021

refs rust-lang/rust#89091

Signed-off-by: Daira Hopwood daira@jacaranda.org

Signed-off-by: Daira Hopwood <daira@jacaranda.org>
@the8472
Copy link
Member

the8472 commented Sep 19, 2021

ninja is already listed in the prerequisites. And there it lists either gnu make or ninja. Links to install instructions should probably go there.

@daira
Copy link
Author

daira commented Sep 19, 2021

Hmm. It says "ninja, or GNU make 3.81 or later", and in rust-lang/rust#89091 I had GNU make, which was not sufficient by default. (Also, there are several things called ninja.)

@jyn514
Copy link
Member

jyn514 commented Sep 19, 2021

@daira GNU make would have worked if you had ninja = false in config.toml. We shouldn't remove those docs.

@camelid camelid added the S-waiting-on-author Status: this PR is waiting for additional action by the OP label Nov 2, 2021
@jyn514
Copy link
Member

jyn514 commented Nov 24, 2021

I'm going to close this for now - I think improving errors from x.py is better than making the guide even longer, and it's not clear to me that this would be easy to find if you're building for the first time.

@jyn514 jyn514 closed this Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: this PR is waiting for additional action by the OP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants