This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch 9.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/9.2.x by this push:
new 004b56391c tools/check-unused-dependencies: make exceptions for tools
under clang+asan (#10093)
004b56391c is described below
commit 004b56391ce0459b43ec3583162d16b01f6f896d
Author: Brian Olsen <[email protected]>
AuthorDate: Wed Jul 26 09:01:30 2023 -0600
tools/check-unused-dependencies: make exceptions for tools under
clang+asan (#10093)
Co-authored-by: Brian Olsen <[email protected]>
(cherry picked from commit b391b25a2c3cc32042529a391da2253c085d9fec)
---
tools/check-unused-dependencies | 27 +++++++++++++++++++++++----
1 file changed, 23 insertions(+), 4 deletions(-)
diff --git a/tools/check-unused-dependencies b/tools/check-unused-dependencies
index 93dfb2c4bd..1d4da79534 100755
--- a/tools/check-unused-dependencies
+++ b/tools/check-unused-dependencies
@@ -54,25 +54,44 @@ def get_dependencies(program):
'libgcc_s.so.',
'libm.so.', # Why does Libtool call ld with -lm?
'libpthread.so.', # Because we add -lpthread to LIBS
- 'librt.so.' # clang + asan pulls this in
])):
continue
+ progbase = os.path.basename(program)
+
# Why does Libtool call ld with -lcrypto -lresolv -lssl?
- if os.path.basename(program) == 'traffic_manager':
+ if progbase == 'traffic_manager':
if any(
map(
os.path.basename(dependency).startswith, [
'libcrypto.so.',
'libresolv.so.',
- 'libssl.so.'
+ 'libssl.so.',
+ ])):
+ continue
+
+ # clang+asan pulls in dependencies for these specific tools:
+ if any(
+ map(
+ progbase.__eq__,
+ [
+ 'jtest',
+ 'http_load',
+ 'escape_mapper',
+ ])):
+ if any(
+ map(
+ os.path.basename(dependency).startswith,
+ [
+ 'librt.so',
+ 'libresolv.so',
])):
continue
+
if re.sub(r'\s+', '', dependency):
yield dependency
-
success = True
filename = 'config.status'
contents = open(filename).read()
