Skip to content

Commit

Permalink
CMake: regroup some lines for better understanding
Browse files Browse the repository at this point in the history
  • Loading branch information
christianparpart committed Nov 19, 2023
1 parent 4e76530 commit fbf5d07
Show file tree
Hide file tree
Showing 5 changed files with 161 additions and 79 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ jobs:
id: set_vars
run: |
./scripts/ci-set-vars.sh
echo QTVER=5 >> "$GITHUB_OUTPUT"
echo QTVER=6 >> "$GITHUB_OUTPUT"
env:
REPOSITORY: ${{ github.event.repository.name }}
- name: ccache
Expand Down
1 change: 1 addition & 0 deletions autogen.sh
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ case "$OSTYPE" in
darwin*)
if test v$QTVER = v6; then
EXTRA_CMAKE_FLAGS="${EXTRA_CMAKE_FLAGS} -DQt6_DIR=$(brew --prefix qt6)/lib/cmake/Qt6"
#EXTRA_CMAKE_FLAGS="${EXTRA_CMAKE_FLAGS} -DQt6_DIR=$HOME/Qt/6.5.3/macos/lib/cmake/Qt6"
else
EXTRA_CMAKE_FLAGS="${EXTRA_CMAKE_FLAGS} -DQt5_DIR=$(brew --prefix qt5)/lib/cmake/Qt5"
fi
Expand Down
2 changes: 1 addition & 1 deletion cmake/ContourThirdParties.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -140,9 +140,9 @@ macro(ContourThirdPartiesSummary2)
message(STATUS "freetype ${THIRDPARTY_BUILTIN_freetype}")
message(STATUS "harfbuzz ${THIRDPARTY_BUILTIN_harfbuzz}")
message(STATUS "range-v3 ${THIRDPARTY_BUILTIN_range_v3}")
message(STATUS "yaml-cpp ${THIRDPARTY_BUILTIN_yaml_cpp}")
message(STATUS "termbench-pro ${THIRDPARTY_BUILTIN_termbench}")
message(STATUS "libunicode ${THIRDPARTY_BUILTIN_unicode_core}")
message(STATUS "yaml-cpp ${THIRDPARTY_BUILTIN_yaml_cpp}")
message(STATUS "boxed-cpp ${THIRDPARTY_BUILTIN_boxed_cpp}")
message(STATUS "------------------------------------------------------------------------------")
endmacro()
6 changes: 2 additions & 4 deletions scripts/install-deps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -497,9 +497,6 @@ install_deps_fedora()

install_deps_darwin()
{
# NB: yaml-cpp is available on brew, but gives linker error on Github CI.
fetch_and_unpack_yaml_cpp

fetch_and_unpack_libunicode

[ x$PREPARE_ONLY_EMBEDS = xON ] && return
Expand All @@ -515,7 +512,8 @@ install_deps_darwin()
pkg-config \
qt$QTVER \
libssh2 \
range-v3
range-v3 \
yaml-cpp
}

main()
Expand Down
Loading

0 comments on commit fbf5d07

Please sign in to comment.