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): add hints for openssl find_package #2707

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cmake/dependencies/common.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ add_subdirectory("${CMAKE_SOURCE_DIR}/third-party/moonlight-common-c/enet")
add_subdirectory("${CMAKE_SOURCE_DIR}/third-party/Simple-Web-Server")

# common dependencies
find_package(OpenSSL REQUIRED)
find_package(OpenSSL REQUIRED HINTS "/opt/homebrew/opt/openssl" "/usr/local/opt/openssl") # hints are for macOS
find_package(PkgConfig REQUIRED)
find_package(Threads REQUIRED)
pkg_check_modules(CURL REQUIRED libcurl)
Expand Down
17 changes: 0 additions & 17 deletions docs/source/building/macos.rst
Original file line number Diff line number Diff line change
Expand Up @@ -21,23 +21,6 @@ Install Requirements

brew install cmake doxygen graphviz icu4c miniupnpc node openssl@3 opus pkg-config [email protected]

If there are issues with an SSL header that is not found:
.. tab:: Intel

.. code-block:: bash

pushd /usr/local/include
ln -s ../opt/openssl/include/openssl .
popd

.. tab:: Apple Silicon

.. code-block:: bash

pushd /opt/homebrew/include
ln -s ../opt/openssl/include/openssl .
popd

Build
-----
.. attention:: Ensure you are in the build directory created during the clone step earlier before continuing.
Expand Down
Loading