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

Fix CMake warning, update macOS installation instructions #3830

Merged
merged 4 commits into from
Jul 28, 2020

Conversation

jngrad
Copy link
Member

@jngrad jngrad commented Jul 28, 2020

Description of changes:

  • fix GNUInstallDirs-related warning generated in CMake 3.18
  • update macOS installation instructions for command line tools
  • increase timeout on macOS runner from GitHub Workflows

Fixes CMake warning "Unable to determine default CMAKE_INSTALL_LIBDIR
directory because no target architecture is known. Please enable at
least one language before including GNUInstallDirs.".
Running `sudo xcodebuild -license accept` after `sudo xcode-select
--install` causes an error.
doc/sphinx/installation.rst Outdated Show resolved Hide resolved
doc/sphinx/installation.rst Outdated Show resolved Hide resolved
@KaiSzuttor KaiSzuttor added the automerge Merge with kodiak label Jul 28, 2020
@mkuron
Copy link
Member

mkuron commented Jul 28, 2020

Do you really think it's a good idea to leave people on their own in installing Homebrew or MacPorts? Not everyone knows how to do that and their own install instructions can be intimidating to novice users.

@jngrad
Copy link
Member Author

jngrad commented Jul 28, 2020

@mkuron I was not able to follow the MacPorts installation instructions in our user guide, the terminal threw an error. Even after you explained to me how the second command was only relevant to Xcode installed from the App Store, I still misunderstood what the command really did. The official MacPorts installation instructions explain why xcodebuild -license is only relevant to Xcode 4+. I agree it's quite verbose, but linking to it guarantees our macOS instructions are always up-to-date.

@jngrad jngrad merged commit 99a0791 into espressomd:python Jul 28, 2020
@jngrad jngrad added this to the Espresso 4.1.4 milestone Aug 18, 2020
jngrad added a commit to jngrad/espresso that referenced this pull request Aug 18, 2020
Description of changes:
- fix `GNUInstallDirs`-related warning generated in CMake 3.18
@jngrad jngrad added the CMake label Aug 18, 2020
@jngrad jngrad deleted the update-cmake-macos-instructions branch January 18, 2022 12:16
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.

3 participants