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

Small changes to Mac install instructions. #582

Merged
merged 2 commits into from
Sep 11, 2023
Merged

Small changes to Mac install instructions. #582

merged 2 commits into from
Sep 11, 2023

Conversation

billetd
Copy link

@billetd billetd commented Jul 18, 2023

Added instructions for finding the X11 path, as well as clarifying what the directory should look like (i.e., not ending in the include sub directory.

Mentioned in issue #581

Added instructions for finding the X11 path, as well as clarifying what the directory should look like.
@billetd billetd self-assigned this Jul 18, 2023
@billetd billetd changed the title Small changes to Max install instructions. Small changes to Mac install instructions. Jul 18, 2023
Added instructions for users with ARM64 architecture Macs. This is required so the shared executable are compiled correctly.
@billetd
Copy link
Author

billetd commented Jul 24, 2023

Added some additional instructions for users on newer Macs, as they have a different CPU architecture (ARM64 on M1 Macs and newer, versus x86_64 for Intel based Macs). Rosetta, the Apple back-compatibility translation layer, is required in order to compile some shared executables (such as those for using AACGM, file reading) to work correctly.

@ecbland
Copy link

ecbland commented Sep 11, 2023

This looks good! Not sure why the mkdocs check is failing (not just for this PR) -- it works just fine when I build it locally. Merging now.

@ecbland ecbland merged commit e777243 into develop Sep 11, 2023
@ecbland ecbland deleted the mac_doc_fix branch September 11, 2023 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants