Skip to content

Commit 9ee265d

Browse files
authored
Merge pull request ClickHouse#80478 from ClickHouse/trim-aws-contribs
Trim AWS SDK contribs
2 parents 86f6f2d + 928438f commit 9ee265d

File tree

5 files changed

+30
-46
lines changed

5 files changed

+30
-46
lines changed

.gitmodules

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -296,9 +296,6 @@
296296
[submodule "contrib/aws-c-compression"]
297297
path = contrib/aws-c-compression
298298
url = https://github.com/awslabs/aws-c-compression
299-
[submodule "contrib/aws-s2n-tls"]
300-
path = contrib/aws-s2n-tls
301-
url = https://github.com/ClickHouse/s2n-tls
302299
[submodule "contrib/crc32-vpmsum"]
303300
path = contrib/crc32-vpmsum
304301
url = https://github.com/antonblanchard/crc32-vpmsum.git

contrib/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,6 @@ add_contrib (aws-cmake
122122
aws-c-mqtt
123123
aws-c-s3
124124
aws-c-sdkutils
125-
aws-s2n-tls
126125
aws-checksums
127126
aws-crt-cpp
128127
aws-cmake

contrib/aws-cmake/CMakeLists.txt

Lines changed: 5 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@ include("${ClickHouse_SOURCE_DIR}/contrib/aws-cmake/AwsThreadName.cmake")
2727
include("${ClickHouse_SOURCE_DIR}/contrib/aws-cmake/AwsSIMD.cmake")
2828
include("${ClickHouse_SOURCE_DIR}/contrib/aws-crt-cpp/cmake/AwsGetVersion.cmake")
2929

30+
set (AWS_STUBS "${ClickHouse_SOURCE_DIR}/contrib/aws-cmake/aws_stubs.cpp")
31+
3032

3133
# Gather sources and options.
3234
set(AWS_SOURCES)
@@ -47,11 +49,6 @@ if (ENABLE_OPENSSL_ENCRYPTION)
4749
list(APPEND AWS_PRIVATE_COMPILE_DEFS "-DENABLE_OPENSSL_ENCRYPTION")
4850
endif()
4951

50-
set(USE_S2N ON)
51-
if (USE_S2N)
52-
list(APPEND AWS_PRIVATE_COMPILE_DEFS "-DUSE_S2N")
53-
endif()
54-
5552

5653
# Directories.
5754
SET(AWS_SDK_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws")
@@ -70,7 +67,6 @@ SET(AWS_EVENT_STREAM_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws-c-event-stream")
7067
SET(AWS_HTTP_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws-c-http")
7168
SET(AWS_IO_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws-c-io")
7269
SET(AWS_MQTT_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws-c-mqtt")
73-
SET(AWS_S2N_TLS_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws-s2n-tls")
7470
SET(AWS_S3_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws-c-s3")
7571
SET(AWS_SDKUTILS_DIR "${ClickHouse_SOURCE_DIR}/contrib/aws-c-sdkutils")
7672

@@ -287,39 +283,10 @@ elseif (OS_DARWIN)
287283
)
288284
endif()
289285

290-
set(AWS_IO_TLS_SRC)
291-
if (USE_S2N)
292-
file(GLOB AWS_IO_TLS_SRC
293-
"${AWS_IO_DIR}/source/s2n/*.c"
294-
)
295-
endif()
296-
297-
list(APPEND AWS_SOURCES ${AWS_IO_SRC} ${AWS_IO_OS_SRC} ${AWS_IO_TLS_SRC})
286+
list(APPEND AWS_SOURCES ${AWS_IO_SRC} ${AWS_IO_OS_SRC})
298287
list(APPEND AWS_PUBLIC_INCLUDES "${AWS_IO_DIR}/include/")
299288

300289

301-
# aws-s2n-tls
302-
if (USE_S2N)
303-
file(GLOB AWS_S2N_TLS_SRC
304-
"${AWS_S2N_TLS_DIR}/crypto/*.c"
305-
"${AWS_S2N_TLS_DIR}/error/*.c"
306-
"${AWS_S2N_TLS_DIR}/stuffer/*.c"
307-
"${AWS_S2N_TLS_DIR}/pq-crypto/*.c"
308-
"${AWS_S2N_TLS_DIR}/pq-crypto/kyber_r3/*.c"
309-
"${AWS_S2N_TLS_DIR}/tls/*.c"
310-
"${AWS_S2N_TLS_DIR}/tls/extensions/*.c"
311-
"${AWS_S2N_TLS_DIR}/utils/*.c"
312-
)
313-
314-
list(APPEND AWS_SOURCES ${AWS_S2N_TLS_SRC})
315-
316-
list(APPEND AWS_PRIVATE_INCLUDES
317-
"${AWS_S2N_TLS_DIR}/"
318-
"${AWS_S2N_TLS_DIR}/api/"
319-
)
320-
endif()
321-
322-
323290
# aws-crt-cpp
324291
file(GLOB AWS_CRT_SRC
325292
"${AWS_CRT_DIR}/source/*.cpp"
@@ -336,11 +303,6 @@ list(APPEND AWS_PUBLIC_INCLUDES "${AWS_CRT_DIR}/include/")
336303

337304

338305
# aws-c-mqtt
339-
file(GLOB AWS_MQTT_SRC
340-
"${AWS_MQTT_DIR}/source/*.c"
341-
)
342-
343-
list(APPEND AWS_SOURCES ${AWS_MQTT_SRC})
344306
list(APPEND AWS_PUBLIC_INCLUDES "${AWS_MQTT_DIR}/include/")
345307

346308

@@ -388,6 +350,8 @@ file(GLOB AWS_SDK_GLUE_SRC
388350
list(APPEND AWS_SOURCES ${AWS_SDK_GLUE_SRC})
389351
list(APPEND AWS_PUBLIC_INCLUDES "${AWS_SDK_GLUE_DIR}/include/")
390352

353+
list(APPEND AWS_SOURCES ${AWS_STUBS})
354+
391355
# Add library.
392356
add_library(_aws ${AWS_SOURCES})
393357

contrib/aws-cmake/aws_stubs.cpp

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
extern "C" {
2+
3+
/// Symbols for aws-c-mqtt and aws-s2n-tls
4+
/// which are not used anywhere except those stubs.
5+
6+
// TLS context creation (returns null)
7+
__attribute__((weak)) void *aws_tls_client_ctx_new() { return nullptr; }
8+
__attribute__((weak)) void *aws_tls_server_ctx_new() { return nullptr; }
9+
10+
// TLS static init/cleanup (do nothing)
11+
__attribute__((weak)) void aws_tls_init_static_state() {}
12+
__attribute__((weak)) void aws_tls_clean_up_static_state() {}
13+
14+
// MQTT init/cleanup (do nothing)
15+
__attribute__((weak)) void aws_mqtt_library_init() {}
16+
__attribute__((weak)) void aws_mqtt_library_clean_up() {}
17+
18+
// Darwin-specific TLS handlers (return null or 0)
19+
__attribute__((weak)) void *aws_tls_client_handler_new() { return nullptr; }
20+
__attribute__((weak)) void *aws_tls_server_handler_new() { return nullptr; }
21+
__attribute__((weak)) int aws_tls_client_handler_start_negotiation() { return 0; }
22+
__attribute__((weak)) void *aws_tls_handler_protocol() { return nullptr; }
23+
__attribute__((weak)) int aws_tls_is_alpn_available() { return 0; }
24+
25+
} // extern "C"

contrib/aws-s2n-tls

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)