On Sat Sep 02, 2023 at 02:09:06PM +0000, Klemens Nanni wrote: > All consumers build with it, but qt-creator and opencolorio need their > custom .cmake code for finding the library to be removed. > > net/powerdns > games/openxcom > devel/qt-creator > graphics/opencolorio
Our qt-creator is old (New depends on Qt6 and Qt6WebEngine). If all consumers are happy with it ok. Maybe in future version they will use the upstream cmake file. > > 0.8.0 release notes mention major CMake changes, but I did not dig into > what exactly messes with consumer's custom .cmake files since they're of > no use for us, anyway. > > Feedback? OK? OK rsadowski > > Index: devel/yaml-cpp/Makefile > =================================================================== > RCS file: /cvs/ports/devel/yaml-cpp/Makefile,v > retrieving revision 1.20 > diff -u -p -r1.20 Makefile > --- devel/yaml-cpp/Makefile 11 Jun 2023 07:54:27 -0000 1.20 > +++ devel/yaml-cpp/Makefile 2 Sep 2023 11:37:19 -0000 > @@ -1,16 +1,12 @@ > COMMENT = YAML parser and emitter in C++ > > -V= 0.7.0 > GH_ACCOUNT= jbeder > GH_PROJECT= yaml-cpp > -GH_TAGNAME= yaml-cpp-$V > -DISTNAME = yaml-cpp-$V > +GH_TAGNAME= 0.8.0 > > -SHARED_LIBS = yaml-cpp 4.0 # 0.5 > +SHARED_LIBS = yaml-cpp 5.0 # 0.5 > > CATEGORIES = devel > - > -HOMEPAGE = https://github.com/jbeder/yaml-cpp > > # MIT > PERMIT_PACKAGE = Yes > Index: devel/yaml-cpp/distinfo > =================================================================== > RCS file: /cvs/ports/devel/yaml-cpp/distinfo,v > retrieving revision 1.6 > diff -u -p -r1.6 distinfo > --- devel/yaml-cpp/distinfo 11 Jun 2023 07:54:27 -0000 1.6 > +++ devel/yaml-cpp/distinfo 2 Sep 2023 11:29:04 -0000 > @@ -1,2 +1,2 @@ > -SHA256 (yaml-cpp-0.7.0.tar.gz) = Q+ap/LFGrYcVFfDQhzlH5dSXocnGDFjLECqXtHIIt8M= > -SIZE (yaml-cpp-0.7.0.tar.gz) = 1033237 > +SHA256 (yaml-cpp-0.8.0.tar.gz) = ++dLvc7iHWVnFWiHBto8i+z9lG2SzURwXMYJi7I7OhY= > +SIZE (yaml-cpp-0.8.0.tar.gz) = 1017151 > Index: devel/yaml-cpp/patches/patch-CMakeLists_txt > =================================================================== > RCS file: devel/yaml-cpp/patches/patch-CMakeLists_txt > diff -N devel/yaml-cpp/patches/patch-CMakeLists_txt > --- devel/yaml-cpp/patches/patch-CMakeLists_txt 11 Jun 2023 07:54:27 > -0000 1.7 > +++ /dev/null 1 Jan 1970 00:00:00 -0000 > @@ -1,67 +0,0 @@ > -- Fix CMake export files (#1077) > - 4aad2b1666a4742743b04e765a34742512915674 > -- Install pkgconfig and cmake files into arch-dependent locations (#1055) > - 255bf1f9bccfb8cac5fc9ea70fe528423f2e58b9 > - > -Index: CMakeLists.txt > ---- CMakeLists.txt.orig > -+++ CMakeLists.txt > -@@ -127,10 +127,14 @@ set_target_properties(yaml-cpp PROPERTIES > - PROJECT_LABEL "yaml-cpp ${yaml-cpp-label-postfix}" > - DEBUG_POSTFIX "${CMAKE_DEBUG_POSTFIX}") > - > -+set(CONFIG_EXPORT_DIR "${CMAKE_INSTALL_LIBDIR}/cmake/yaml-cpp") > -+set(EXPORT_TARGETS yaml-cpp) > - configure_package_config_file( > - "${PROJECT_SOURCE_DIR}/yaml-cpp-config.cmake.in" > - "${PROJECT_BINARY_DIR}/yaml-cpp-config.cmake" > -- INSTALL_DESTINATION "${CMAKE_INSTALL_DATADIR}/cmake/yaml-cpp") > -+ INSTALL_DESTINATION "${CONFIG_EXPORT_DIR}" > -+ PATH_VARS CMAKE_INSTALL_INCLUDEDIR CONFIG_EXPORT_DIR) > -+unset(EXPORT_TARGETS) > - > - write_basic_package_version_file( > - "${PROJECT_BINARY_DIR}/yaml-cpp-config-version.cmake" > -@@ -139,30 +143,31 @@ write_basic_package_version_file( > - configure_file(yaml-cpp.pc.in yaml-cpp.pc @ONLY) > - > - if (YAML_CPP_INSTALL) > -- install(TARGETS yaml-cpp > -+ install(TARGETS yaml-cpp > - EXPORT yaml-cpp-targets > - RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR} > - LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} > - ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}) > -- install(DIRECTORY ${PROJECT_SOURCE_DIR}/include/ > -+ install(DIRECTORY ${PROJECT_SOURCE_DIR}/include/ > - DESTINATION ${CMAKE_INSTALL_INCLUDEDIR} > -- FILES_MATCHING PATTERN "*.h") > -+ FILES_MATCHING PATTERN "*.h") > - install(EXPORT yaml-cpp-targets > -- DESTINATION "${CMAKE_INSTALL_DATADIR}/cmake/yaml-cpp") > -- install(FILES > -- "${PROJECT_BINARY_DIR}/yaml-cpp-config.cmake" > -- "${PROJECT_BINARY_DIR}/yaml-cpp-config-version.cmake" > -- DESTINATION "${CMAKE_INSTALL_DATADIR}/cmake/yaml-cpp") > -+ DESTINATION "${CONFIG_EXPORT_DIR}") > -+ install(FILES > -+ "${PROJECT_BINARY_DIR}/yaml-cpp-config.cmake" > -+ "${PROJECT_BINARY_DIR}/yaml-cpp-config-version.cmake" > -+ DESTINATION "${CONFIG_EXPORT_DIR}") > - install(FILES "${PROJECT_BINARY_DIR}/yaml-cpp.pc" > -- DESTINATION ${CMAKE_INSTALL_DATADIR}/pkgconfig) > -+ DESTINATION ${CMAKE_INSTALL_LIBDIR}/pkgconfig) > - endif() > -+unset(CONFIG_EXPORT_DIR) > - > - if(YAML_CPP_BUILD_TESTS) > -- add_subdirectory(test) > -+ add_subdirectory(test) > - endif() > - > - if(YAML_CPP_BUILD_TOOLS) > -- add_subdirectory(util) > -+ add_subdirectory(util) > - endif() > - > - if (YAML_CPP_CLANG_FORMAT_EXE) > Index: devel/yaml-cpp/patches/patch-yaml-cpp-config_cmake_in > =================================================================== > RCS file: devel/yaml-cpp/patches/patch-yaml-cpp-config_cmake_in > diff -N devel/yaml-cpp/patches/patch-yaml-cpp-config_cmake_in > --- devel/yaml-cpp/patches/patch-yaml-cpp-config_cmake_in 11 Jun 2023 > 07:54:27 -0000 1.1 > +++ /dev/null 1 Jan 1970 00:00:00 -0000 > @@ -1,25 +0,0 @@ > -- Fix CMake export files (#1077) > - 4aad2b1666a4742743b04e765a34742512915674 > - > -Index: yaml-cpp-config.cmake.in > ---- yaml-cpp-config.cmake.in.orig > -+++ yaml-cpp-config.cmake.in > -@@ -3,12 +3,14 @@ > - # YAML_CPP_INCLUDE_DIR - include directory > - # YAML_CPP_LIBRARIES - libraries to link against > - > --# Compute paths > --get_filename_component(YAML_CPP_CMAKE_DIR "${CMAKE_CURRENT_LIST_FILE}" PATH) > --set(YAML_CPP_INCLUDE_DIR "@CONFIG_INCLUDE_DIRS@") > -+@PACKAGE_INIT@ > - > -+set_and_check(YAML_CPP_INCLUDE_DIR "@PACKAGE_CMAKE_INSTALL_INCLUDEDIR@") > -+ > - # Our library dependencies (contains definitions for IMPORTED targets) > --include("${YAML_CPP_CMAKE_DIR}/yaml-cpp-targets.cmake") > -+include(@PACKAGE_CONFIG_EXPORT_DIR@/yaml-cpp-targets.cmake) > - > - # These are IMPORTED targets created by yaml-cpp-targets.cmake > - set(YAML_CPP_LIBRARIES "@EXPORT_TARGETS@") > -+ > -+check_required_components(@EXPORT_TARGETS@) > Index: devel/qt-creator/Makefile > =================================================================== > RCS file: /cvs/ports/devel/qt-creator/Makefile,v > retrieving revision 1.91 > diff -u -p -r1.91 Makefile > --- devel/qt-creator/Makefile 24 Apr 2023 11:40:37 -0000 1.91 > +++ devel/qt-creator/Makefile 2 Sep 2023 14:00:38 -0000 > @@ -94,6 +94,8 @@ pre-configure: > cd ${WRKSRC}; sed -i 's,"gdb","egdb",g' ${GDB_ADJ_FILES} > # use system qbs/syntax-highlighting (qtcreator.pro) > rm -rf ${WRKSRC}/src/shared/qbs > + # custom code fails to find yaml-cpp>=0.8.0 > + rm -f ${WRKSRC}/cmake/Findyaml-cpp.cmake > > post-install: > .for _l _v in ${SHARED_LIBS} > Index: graphics/opencolorio/Makefile > =================================================================== > RCS file: /cvs/ports/graphics/opencolorio/Makefile,v > retrieving revision 1.29 > diff -u -p -r1.29 Makefile > --- graphics/opencolorio/Makefile 12 Jun 2023 21:29:25 -0000 1.29 > +++ graphics/opencolorio/Makefile 2 Sep 2023 14:04:49 -0000 > @@ -47,4 +47,8 @@ CONFIGURE_ARGS += -DOCIO_USE_SSE=ON > CONFIGURE_ARGS += -DOCIO_USE_SSE=OFF > .endif > > +pre-configure: > + # custom code fails to find yaml-cpp>=0.8.0 > + rm -f ${WRKSRC}/share/cmake/modules/Findyaml-cpp.cmake > + > .include <bsd.port.mk> >