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
Improve the MacOS documentation #54
Conversation
I have tested on a MacOS with version 12.1, no XCode and no Rust installed. I followed these steps :
It worked first try without the need to pre-build manually the maplibre_apple library. |
Yes, i fixed that last week. No more need to compile it first :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverting MacOS deployment target
This file was mistakenly commited.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apply all the suggestions to fix the MacOS documentation.
Co-authored-by: Max Ammann <max@maxammann.org>
I was confused when I looked at the building documentation in the mdbooks because the first section specified Linux but the command
cargo run
can be used on many platforms with the right Rustup toolchain target. I also think that thecargo build
command should be mentioned first instead ofcargo run
as we are in the building documentation.I have extended the MacOS documentation with my experience trying to run it in XCode.
Finally I have moved the note about building in XCode that was written in the root README.md to the mdbooks. I think it helps keep the README cleaner. Also in my tests I was not required to pre-build the Rust application (as said in the note) but I will try again on a clean environment tomorrow.
This is my first contribution to the project, I am sorry in advance if I made something wrong and I would be very pleased if you can give me feedback so I don't make the same mistakes again.