Skip to content

Commit d2ebc1c

Browse files
committed
Fix icu dependency
Fix macos codeql workflow Fix unsigned char convertion
1 parent d2d9d0a commit d2ebc1c

File tree

4 files changed

+17
-7
lines changed

4 files changed

+17
-7
lines changed

.github/workflows/macos-codeql.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,13 @@ jobs:
2424
]
2525
config: [Debug, Release]
2626
include:
27-
- name: macos-13-xcode-15-x86_64
27+
- name: macos-13-xcode-14-x86_64
2828
os: macos-13
2929
compiler: xcode
3030
version: "14.3"
3131
architecture: x86_64
3232

33-
- name: macos-13-xcode-15-arm64
33+
- name: macos-13-xcode-14-arm64
3434
os: macos-13
3535
compiler: xcode
3636
version: "14.3"

cmake/external/icu.cmake

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,15 +63,15 @@ else()
6363
if(CMAKE_BUILD_TYPE STREQUAL "Debug")
6464
set(ICU_I18N_LIBRARY ${ICU_SRC}/src/ICU/lib${ICU_ARCH}/icuind.lib)
6565
set(ICU_UC_LIBRARY ${ICU_SRC}/src/ICU/lib${ICU_ARCH}/icuucd.lib)
66-
set(ICU_I18N_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuin72d.dll)
67-
set(ICU_UC_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuuc72d.dll)
66+
set(ICU_I18N_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuin76d.dll)
67+
set(ICU_UC_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuuc76d.dll)
6868
else()
6969
set(ICU_I18N_LIBRARY ${ICU_SRC}/src/ICU/lib${ICU_ARCH}/icuin.lib)
7070
set(ICU_UC_LIBRARY ${ICU_SRC}/src/ICU/lib${ICU_ARCH}/icuuc.lib)
71-
set(ICU_I18N_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuin72.dll)
72-
set(ICU_UC_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuuc72.dll)
71+
set(ICU_I18N_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuin76.dll)
72+
set(ICU_UC_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icuuc76.dll)
7373
endif()
74-
set(ICU_DATA_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icudt72.dll)
74+
set(ICU_DATA_DLL ${ICU_SRC}/src/ICU/bin${ICU_ARCH}/icudt76.dll)
7575
set(ICU_INCLUDE_DIR ${ICU_SRC}/src/ICU/include)
7676
endif()
7777

source/CMakeLists.txt

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,11 @@ target_include_directories(${PROJECT_NAME}
4343
${PROJECT_SOURCE_DIR}
4444
)
4545

46+
target_compile_definitions(${PROJECT_NAME}
47+
PRIVATE
48+
$<$<BOOL:${HAVE_SPAN}>:HAVE_SPAN>
49+
)
50+
4651
if(CMAKE_CXX_COMPILER_ID MATCHES Clang AND CMAKE_CXX_COMPILER_VERSION VERSION_LESS 9.0)
4752
set(${PROJECT_NAME}_LIBS -lc++fs)
4853
endif()

source/SqliteUtils.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,12 @@ namespace vx::sqlite_utils {
255255
return { SQLITE_IOERR, SqliteErrorCategory::instance() };
256256
}
257257

258+
#ifdef HAVE_SPAN
259+
const std::span spanDump( dump.get(), static_cast<std::size_t>( serializationSize ) );
260+
std::vector<char> converted( std::begin( spanDump ), std::end( spanDump ) );
261+
#else
258262
std::vector<char> converted( dump.get(), dump.get() + serializationSize );
263+
#endif
259264
if ( converted.empty() ) {
260265

261266
SqliteErrorCategory::instance().setMessage( "Export not convertable." );

0 commit comments

Comments
 (0)