mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-08-10 01:59:31 +00:00
Meta/vcpkg: Set default visibility for skcms symbols
Without this patch, as skia is compiled with -fvisibility=hidden, no skcms function is accessible from outside skia's code.
This commit is contained in:
parent
50b978fefb
commit
4d7b7178f9
Notes:
github-actions[bot]
2024-12-05 16:17:59 +00:00
Author: https://github.com/LucasChollet
Commit: 4d7b7178f9
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/1764
Reviewed-by: https://github.com/trflynn89
1 changed files with 11 additions and 5 deletions
|
@ -274,13 +274,22 @@ else()
|
|||
cxx=\"${VCPKG_DETECTED_CMAKE_CXX_COMPILER}\"")
|
||||
endif()
|
||||
|
||||
set(SKIA_CXX_FLAGS_DBG "${VCPKG_COMBINED_CXX_FLAGS_DEBUG}")
|
||||
set(SKIA_CXX_FLAGS_REL "${VCPKG_COMBINED_CXX_FLAGS_RELEASE}")
|
||||
|
||||
if(VCPKG_LIBRARY_LINKAGE STREQUAL "dynamic")
|
||||
foreach (str IN ITEMS SKIA_CXX_FLAGS_DBG SKIA_CXX_FLAGS_REL)
|
||||
string(APPEND ${str} " -DSKCMS_API=__attribute__((visibility(\\\\\\\"default\\\\\\\")))")
|
||||
endforeach()
|
||||
endif()
|
||||
|
||||
string_to_gn_list(SKIA_C_FLAGS_DBG "${VCPKG_COMBINED_C_FLAGS_DEBUG}")
|
||||
string_to_gn_list(SKIA_CXX_FLAGS_DBG "${VCPKG_COMBINED_CXX_FLAGS_DEBUG}")
|
||||
string_to_gn_list(SKIA_CXX_FLAGS_DBG "${SKIA_CXX_FLAGS_DBG}")
|
||||
string(APPEND OPTIONS_DBG " \
|
||||
extra_cflags_c=${SKIA_C_FLAGS_DBG} \
|
||||
extra_cflags_cc=${SKIA_CXX_FLAGS_DBG}")
|
||||
string_to_gn_list(SKIA_C_FLAGS_REL "${VCPKG_COMBINED_C_FLAGS_RELEASE}")
|
||||
string_to_gn_list(SKIA_CXX_FLAGS_REL "${VCPKG_COMBINED_CXX_FLAGS_RELEASE}")
|
||||
string_to_gn_list(SKIA_CXX_FLAGS_REL "${SKIA_CXX_FLAGS_REL}")
|
||||
string(APPEND OPTIONS_REL " \
|
||||
extra_cflags_c=${SKIA_C_FLAGS_REL} \
|
||||
extra_cflags_cc=${SKIA_CXX_FLAGS_REL}")
|
||||
|
@ -309,9 +318,6 @@ file(COPY "${SOURCE_PATH}/include"
|
|||
FILES_MATCHING PATTERN "*.h"
|
||||
)
|
||||
auto_clean("${CURRENT_PACKAGES_DIR}/include/skia")
|
||||
set(skia_dll_static "0")
|
||||
set(skia_dll_dynamic "1")
|
||||
vcpkg_replace_string("${CURRENT_PACKAGES_DIR}/include/skia/include/private/base/SkAPI.h" "defined(SKIA_DLL)" "${skia_dll_${VCPKG_LIBRARY_LINKAGE}}")
|
||||
|
||||
# vcpkg legacy layout omits "include/" component. Just duplicate.
|
||||
file(COPY "${CURRENT_PACKAGES_DIR}/include/skia/include/" DESTINATION "${CURRENT_PACKAGES_DIR}/include/skia")
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue