Skip to content

Commit a0016dd

Browse files
committed
[ms-quic] Remove from ci.baseline.txt and fix UWP builds.
Also fix uwp builds so that all ci.baseline.txt entries can be removed, submitted upstream as microsoft/msquic#4373 Originally started from microsoft#39475 Co-authored by: Lily Wang <v-lilywang@microsoft.com>
1 parent c4467cb commit a0016dd

File tree

4 files changed

+43
-27
lines changed

4 files changed

+43
-27
lines changed

ports/ms-quic/fix-uwp-crt.patch

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
diff --git a/CMakeLists.txt b/CMakeLists.txt
2+
index 3f8f4d58f..df689627c 100644
3+
--- a/CMakeLists.txt
4+
+++ b/CMakeLists.txt
5+
@@ -598,9 +598,9 @@ if(WIN32)
6+
endif()
7+
8+
if (NOT QUIC_STATIC_LINK_CRT AND NOT QUIC_STATIC_LINK_PARTIAL_CRT)
9+
- # We are using dynamic linking. Ensure that only the release version of CRT is used.
10+
- message(STATUS "Configuring for release version of dynamically linked CRT")
11+
- set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreadedDLL")
12+
+ # We are using dynamic linking. Ensure that only the dynamic CRT is used.
13+
+ message(STATUS "Configuring for dynamically linked CRT")
14+
+ set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>DLL")
15+
endif()
16+
17+
else() #!WIN32

ports/ms-quic/portfile.cmake

Lines changed: 22 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@ vcpkg_from_github(
77
SHA512 5937fbc2f287567d590fc0afc947459359e5413fa25f2f193434ad6d7016f7cb0dede4e2ef5e1e4e8b21b556c5ad8ce4cb612514403bb593a49af0fb42d1cb15
88
HEAD_REF master
99
PATCHES
10-
fix-install.patch # Adjust install path of build outputs
10+
fix-install.patch # Adjust install path of build outputs
11+
fix-uwp-crt.patch # https://github.com/microsoft/msquic/pull/4373
1112
)
1213

1314
vcpkg_from_github(
@@ -17,32 +18,35 @@ vcpkg_from_github(
1718
SHA512 ff487d882c2b70ed8915a88ecf0a64724435a96187a7bb3bf401f4a2c4dc572a93f7e788040ccbd29da8bc6ac49ee11550c9d56153262c05fae173ac1d242baa
1819
HEAD_REF openssl-3.1.5+quic
1920
)
20-
file(REMOVE_RECURSE ${QUIC_SOURCE_PATH}/submodules/openssl)
21-
file(RENAME ${OPENSSL_SOURCE_PATH} ${QUIC_SOURCE_PATH}/submodules/openssl)
21+
22+
file(REMOVE_RECURSE "${QUIC_SOURCE_PATH}/submodules/openssl")
23+
file(RENAME "${OPENSSL_SOURCE_PATH}" "${QUIC_SOURCE_PATH}/submodules/openssl")
24+
2225
vcpkg_from_github(
2326
OUT_SOURCE_PATH XDP_WINDOWS
2427
REPO microsoft/xdp-for-windows
2528
REF ce228a986fd30049ed58f569d2bf20efffc250f3
2629
SHA512 4a26c5defe422ef42308d72cf8d1cab1c172ce5a10d8d830c446cb7dd93f0c41f35f3cbbfeceb687d5135272006dd1b1bc4c2089ace4866cede81d5c76206af7
27-
HEAD_REF mian
30+
HEAD_REF main
2831
)
29-
file(REMOVE_RECURSE ${QUIC_SOURCE_PATH}/submodules/xdp-for-windows)
30-
file(RENAME ${XDP_WINDOWS} ${QUIC_SOURCE_PATH}/submodules/xdp-for-windows)
32+
33+
file(REMOVE_RECURSE "${QUIC_SOURCE_PATH}/submodules/xdp-for-windows")
34+
file(RENAME "${XDP_WINDOWS}" "${QUIC_SOURCE_PATH}/submodules/xdp-for-windows")
3135

3236
vcpkg_find_acquire_program(PERL)
33-
get_filename_component(PERL_EXE_PATH ${PERL} DIRECTORY)
34-
vcpkg_add_to_path(${PERL_EXE_PATH})
37+
get_filename_component(PERL_EXE_PATH "${PERL}" DIRECTORY)
38+
vcpkg_add_to_path("${PERL_EXE_PATH}")
3539

3640
if(NOT VCPKG_HOST_IS_WINDOWS)
3741
find_program(MAKE make)
38-
get_filename_component(MAKE_EXE_PATH ${MAKE} DIRECTORY)
39-
vcpkg_add_to_path(PREPEND ${MAKE_EXE_PATH})
42+
get_filename_component(MAKE_EXE_PATH "${MAKE}" DIRECTORY)
43+
vcpkg_add_to_path(PREPEND "${MAKE_EXE_PATH}")
4044
endif()
4145

4246
if(VCPKG_TARGET_IS_WINDOWS)
4347
vcpkg_find_acquire_program(NASM)
44-
get_filename_component(NASM_EXE_PATH ${NASM} DIRECTORY)
45-
vcpkg_add_to_path(PREPEND ${NASM_EXE_PATH})
48+
get_filename_component(NASM_EXE_PATH "${NASM}" DIRECTORY)
49+
vcpkg_add_to_path(PREPEND "${NASM_EXE_PATH}")
4650
endif()
4751

4852
vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
@@ -53,30 +57,29 @@ vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
5357
string(COMPARE EQUAL "${VCPKG_CRT_LINKAGE}" "static" STATIC_CRT)
5458

5559
vcpkg_cmake_configure(
56-
SOURCE_PATH ${QUIC_SOURCE_PATH}
60+
SOURCE_PATH "${QUIC_SOURCE_PATH}"
5761
OPTIONS
5862
${FEATURE_OPTIONS}
5963
-DQUIC_SOURCE_LINK=OFF
6064
-DQUIC_TLS=openssl
61-
-DQUIC_TLS_SECRETS_SUPPORT=ON
6265
-DQUIC_USE_SYSTEM_LIBCRYPTO=OFF
6366
-DQUIC_BUILD_PERF=OFF
6467
-DQUIC_BUILD_TEST=OFF
65-
-DQUIC_STATIC_LINK_CRT=${STATIC_CRT}
66-
-DQUIC_UWP_BUILD=${VCPKG_TARGET_IS_UWP}
68+
"-DQUIC_STATIC_LINK_CRT=${STATIC_CRT}"
69+
"-DQUIC_UWP_BUILD=${VCPKG_TARGET_IS_UWP}"
6770
)
6871

6972
vcpkg_cmake_install()
7073
vcpkg_copy_pdbs()
7174

72-
if("tools" IN_LIST FEATURES)
75+
if("tools" IN_LIST "FEATURES")
7376
vcpkg_copy_tools(TOOL_NAMES quicattack quicinterop quicinteropserver quicipclient quicipserver
7477
quicpcp quicpost quicsample spinquic
7578
AUTO_CLEAN
7679
)
7780
endif()
7881

7982
vcpkg_install_copyright(FILE_LIST "${QUIC_SOURCE_PATH}/LICENSE")
80-
file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/debug/share
81-
${CURRENT_PACKAGES_DIR}/debug/include
83+
file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/share"
84+
"${CURRENT_PACKAGES_DIR}/debug/include"
8285
)

ports/ms-quic/vcpkg.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
{
22
"name": "ms-quic",
33
"version": "2.3.5",
4+
"port-version": 1,
45
"description": "Cross-platform, C implementation of the IETF QUIC protocol",
56
"homepage": "https://github.com/microsoft/msquic",
67
"license": "MIT",
@@ -17,7 +18,8 @@
1718
],
1819
"features": {
1920
"tools": {
20-
"description": "Install the tools after build"
21+
"description": "Install the tools after build",
22+
"supports": "!uwp"
2123
}
2224
}
2325
}

scripts/ci.baseline.txt

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -758,19 +758,13 @@ mozjpeg:arm-neon-android=fail
758758
mozjpeg:arm64-android=fail
759759
mozjpeg:x64-android=fail
760760
mpir:x64-android=fail
761-
# ms-gltf and ms-quic require the Microsoft GDK with Xbox Extensions which is not installed on the CI pipeline machines
761+
# ms-gltf require the Microsoft GDK with Xbox Extensions which is not installed on the CI pipeline machines
762762
ms-gdkx:x64-windows=fail
763763
ms-gdkx:x64-windows-static=fail
764764
ms-gdkx:x64-windows-static-md=fail
765765
ms-gltf:arm-neon-android=fail
766766
ms-gltf:arm64-android=fail
767767
ms-gltf:x64-android=fail
768-
ms-quic:arm64-uwp=fail
769-
ms-quic:arm64-windows=fail
770-
ms-quic:x64-uwp=fail
771-
ms-quic:x64-windows-static-md=fail
772-
ms-quic:x64-windows=fail
773-
ms-quic:x86-windows=fail
774768
monkeys-audio:arm64-windows=fail
775769
monkeys-audio:x64-windows-static=fail
776770
moos-core:x64-windows-static=fail

0 commit comments

Comments
 (0)