Skip to content

Commit 0b299f2

Browse files
ThomasDevoogdtpatrick-stephens
authored andcommitted
workflows: pr-compile-check.yaml: improve the system libraries test
- added a install system libraries step To make it clear that this is part of the test. - assert the linked system libs Check if the system lib is effectively linked as expected. Signed-off-by: Thomas Devoogdt <[email protected]>
1 parent e8f3f48 commit 0b299f2

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

.github/workflows/pr-compile-check.yaml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,13 @@ jobs:
5353
- name: Setup environment
5454
run: |
5555
sudo apt-get update
56-
sudo apt-get install -y curl gcc-9 g++-9 clang-12 libsystemd-dev gcovr libyaml-dev libluajit-5.1-dev \
57-
libnghttp2-dev libjemalloc-dev
56+
sudo apt-get install -y curl gcc-9 g++-9 clang-12 libsystemd-dev gcovr libyaml-dev
5857
sudo ln -s /usr/bin/llvm-symbolizer-12 /usr/bin/llvm-symbolizer || true
58+
59+
- name: Install system libraries for this test
60+
run: |
61+
sudo apt-get update
62+
sudo apt-get install -y libjemalloc-dev libluajit-5.1-dev libnghttp2-dev
5963
mkdir -p /tmp/libbacktrace/build && \
6064
curl -L https://github.com/ianlancetaylor/libbacktrace/archive/8602fda.tar.gz | \
6165
tar --strip-components=1 -xzC /tmp/libbacktrace/ && \
@@ -87,5 +91,10 @@ jobs:
8791

8892
- name: Display dependencies w/ ldd
8993
run: |
90-
ldd ./bin/fluent-bit
94+
export ldd_result=$(ldd ./bin/fluent-bit)
95+
echo "ldd result:"
96+
echo "$ldd_result"
97+
echo "$ldd_result" | grep libjemalloc
98+
echo "$ldd_result" | grep libluajit
99+
echo "$ldd_result" | grep libnghttp2
91100
working-directory: build

0 commit comments

Comments
 (0)