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

Updated install doc of the minimum required macOS #5833

Merged
merged 1 commit into from
Oct 2, 2021

Conversation

glennschler
Copy link
Contributor

@glennschler glennschler commented Oct 2, 2021

Testing GMT versions 6.1.1, 6.1.0, and 6.0.0 on macOS version 10.13.6
show only GMT 6.0.0 is able to execute even after successful installation.
The two 6.1.x versions display this error:

/Applications/GMT-6.1.0.app/Contents/MacOS/GMT-6.1.0: line 21: 89507 Abort trap: 6           "${BUNDLE_RESOURCES}/bin/gmt" "$@"
Note 1: If you want to use GMT outside of this terminal or in scripts, then follow these steps:
        a) export GMTHOME=/Applications/GMT-6.1.0.app/Contents/Resources
        b) add $GMTHOME/bin to your path
        c) export PROJ_LIB=$GMTHOME/share/proj
        d) export MAGICK_CONFIGURE_PATH=$GMTHOME/lib/GraphicsMagick/config
Note 2: GMT may use Ghostscript, GraphicsMagick, FFmpeg, and GDAL executables; see
        /Applications/GMT-6.1.0.app/Contents/Resources/share/Licenses for details.

bash-3.2$ gmt
dyld: lazy symbol binding failed: Symbol not found: ____chkstk_darwin
  Referenced from: /Applications/GMT-6.1.0.app/Contents/Resources/bin/../lib/libcurl.4.dylib
(which was built for Mac OS X 10.15)
  Expected in: /usr/lib/libSystem.B.dylib

dyld: Symbol not found: ____chkstk_darwin
  Referenced from: /Applications/GMT-6.1.0.app/Contents/Resources/bin/../lib/libcurl.4.dylib
(which was built for Mac OS X 10.15)
  Expected in: /usr/lib/libSystem.B.dylib

Abort trap: 6

These test results indicate that 6.1.0 and 6.1.1 require a minimum
macOS version of 10.15. GMT version 6.0.0 executes without error
on macOS 10.13.6.

Description of proposed changes

The install instructions for macOS application bundles chart have
been updated to show minimum macOS requirements for the
three GMT releases which were tested.

Resolve #5538

Reminders

  • Make sure that your code follows our style. Use the other functions/files as a basis.
  • Add tests for new features or tests that would have caught the bug that you're fixing.
  • Describe changes to function behavior and arguments in a comment below the function declaration.
  • If adding new functionality, add a detailed description to the documentation and/or an example.

I have tested installing GMT versions 6.1.1, 6.1.0, and 6.0.0 on my
macOS version 10.13.6.  Only GMT 6.0.0 works. The more recent versions
show this error:
```
dyld: lazy symbol binding failed: Symbol not found: ____chkstk_darwin
  Referenced from: /Applications/GMT-6.1.0.app/Contents/Resources/bin/../lib/libcurl.4.dylib (which was built for Mac OS X 10.15)
  Expected in: /usr/lib/libSystem.B.dylib

dyld: Symbol not found: ____chkstk_darwin
  Referenced from: /Applications/GMT-6.1.0.app/Contents/Resources/bin/../lib/libcurl.4.dylib (which was built for Mac OS X 10.15)
  Expected in: /usr/lib/libSystem.B.dylib

Abort trap: 6
```

The error indicates that 6.1.0 and 6.1.1 require a minimum macOS
version 10.15. GMT version 6.0.0 works fine on my macOS 10.13.6,
so I have updated the chart of minimum macOS requirements.
@welcome
Copy link

welcome bot commented Oct 2, 2021

💖 Thanks for opening this pull request! 💖

Please make sure you read our Contributing Guide and abide by our Code of Conduct.

A few things to keep in mind:

  • If you need help writing tests, take a look at the existing ones for inspiration. If you don't know where to start, let us know and we'll walk you through it.
  • All new features should be documented. It helps to write the comments below for your functions describing the it, all arguments, and return types before writing the code. This will help you think about your code design and results in better code.
  • We are grateful that you put in the effort to do this! 🎉

Copy link
Member

@PaulWessel PaulWessel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the feedback and PR!

@PaulWessel PaulWessel merged commit 3acf4a4 into GenericMappingTools:master Oct 2, 2021
@welcome
Copy link

welcome bot commented Oct 2, 2021

🎉🎉🎉 Congrats on merging your first pull request and welcome to the team! 🎉🎉🎉

We hope that this was a good experience for you. Let us know if there is any way that the contributing process could be improved.

@glennschler glennschler deleted the issue/5538 branch October 2, 2021 21:50
@maxrjones maxrjones added the documentation Improve documentation label Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improve documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants