Skip to content

Commit

Permalink
Change CONAN_PKG::xxx casing following upstream breaking changes. Ope…
Browse files Browse the repository at this point in the history
…nSSL is an exception

Read the issue, and the specific OpenSSL comment here: conan-io/conan#6269 (comment)
  • Loading branch information
jmarrec committed Jan 9, 2020
1 parent 95c8d39 commit 90cfe51
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 8 deletions.
2 changes: 1 addition & 1 deletion ProjectMacros.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ macro(CREATE_TEST_TARGETS BASE_NAME SRC DEPENDENCIES)
endif()

target_link_libraries(${BASE_NAME}_tests
CONAN_PKG::GTest
CONAN_PKG::gtest
${ALL_DEPENDENCIES}
)

Expand Down
2 changes: 1 addition & 1 deletion embedded/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ add_executable(CreateEmbeddedSource
CreateEmbeddedSource.cpp
)

target_link_libraries(CreateEmbeddedSource CONAN_PKG::ZLIB)
target_link_libraries(CreateEmbeddedSource CONAN_PKG::zlib)

# Add a specific manifest for CreateEmbeddedSource that will include the LongPathAware attribute, which,
# in conjunction with the regkey LongPathsEnabled=1 will make it work with paths that are longer than MAX_PATH (win10 only)
Expand Down
2 changes: 1 addition & 1 deletion src/cli/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ target_link_libraries(openstudio
openstudio_radiance
)

# Change to CONAN_PKG::OpenSSL vs OPENSSL_CRYPTO_LIBRARY if this is needed.
# Change to CONAN_PKG::openssl vs OPENSSL_CRYPTO_LIBRARY if this is needed.
# link CLI with ruby static library, ruby bindings (openstudio.so) are linked to dynamic ruby
#if (APPLE)
# target_link_libraries(openstudio
Expand Down
2 changes: 1 addition & 1 deletion src/generateiddfactory/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ add_executable(${target_name}
set_target_properties(${target_name} PROPERTIES COMPILE_DEFINITIONS OPENSTUDIO_DIRECT_INCLUDE)

target_link_libraries(${target_name}
CONAN_PKG::Boost
CONAN_PKG::boost
#CONAN_PKG::boost_filesystem
#CONAN_PKG::boost_regex
#CONAN_PKG::boost_program_options
Expand Down
2 changes: 1 addition & 1 deletion src/install_utility/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
add_executable(install_utility main.cpp)

target_link_libraries(install_utility
CONAN_PKG::Boost
CONAN_PKG::boost
# CONAN_PKG::boost_filesystem
)

Expand Down
8 changes: 5 additions & 3 deletions src/utilities/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -402,11 +402,12 @@ add_subdirectory(pch)

set(${target_name}_depends
CONAN_PKG::sqlite3
CONAN_PKG::JsonCpp
CONAN_PKG::ZLIB
CONAN_PKG::jsoncpp
CONAN_PKG::zlib
CONAN_PKG::fmt
CONAN_PKG::pugixml
CONAN_PKG::Boost
CONAN_PKG::boost
#If we use the boost modular package:
#CONAN_PKG::boost_filesystem
#CONAN_PKG::boost_crc
#CONAN_PKG::boost_regex
Expand All @@ -418,6 +419,7 @@ set(${target_name}_depends
#CONAN_PKG::boost_functional
#CONAN_PKG::boost_geometry
CONAN_PKG::cpprestsdk
# TODO: this may change to lowercase
CONAN_PKG::OpenSSL
CONAN_PKG::geographiclib
)
Expand Down

0 comments on commit 90cfe51

Please sign in to comment.