Skip to content

Commit 24a5752

Browse files
authored
Fixed various CI failures (#328)
* Fix `Unknown argument "--skip-broken" for command "dnf5"` * Increase timeout for `render_list` & `render_old` tests * Sync FreeBSD build command with others (and build Release by default) * Fix non-default tile tests * Fixed WEBP test for `libwebp.so.4`
1 parent f652ab1 commit 24a5752

7 files changed

Lines changed: 27 additions & 19 deletions

File tree

.github/actions/cmake/build/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ runs:
1313
run: |
1414
cmake -B build -S . \
1515
-LA \
16-
-DCMAKE_BUILD_TYPE:STRING=${BUILD_TYPE:-Debug} \
16+
-DCMAKE_BUILD_TYPE:STRING=${BUILD_TYPE:-Release} \
1717
-DCMAKE_INSTALL_PREFIX:PATH=${INSTALL_PREFIX:-/usr/local} \
1818
-DENABLE_TESTS:BOOL=ON
1919
shell: bash --noprofile --norc -euxo pipefail {0}

.github/actions/dependencies/install/yum/action.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,5 +48,10 @@ runs:
4848
shell: bash --noprofile --norc -euxo pipefail {0}
4949

5050
- name: Install package(s)
51-
run: yum --assumeyes --skip-broken install ${{ inputs.packages }}
51+
run: |
52+
if command -v dnf5 &> /dev/null; then
53+
yum --assumeyes install ${{ inputs.packages }}
54+
else
55+
yum --assumeyes --skip-broken install ${{ inputs.packages }}
56+
fi
5257
shell: bash --noprofile --norc -euxo pipefail {0}

.github/actions/freebsd/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,11 @@ runs:
3030
release: 13.1
3131
run: |
3232
export CMAKE_BUILD_PARALLEL_LEVEL=$(sysctl -n hw.ncpu)
33+
export LIBRARY_PATH=/usr/local/lib
3334
cmake -B build -S . \
34-
-DCMAKE_LIBRARY_PATH:PATH=/usr/local/lib \
35+
-LA \
36+
-DCMAKE_BUILD_TYPE:STRING=${BUILD_TYPE:-Release} \
37+
-DCMAKE_INSTALL_PREFIX:PATH=${INSTALL_PREFIX:-/usr/local} \
3538
-DENABLE_TESTS:BOOL=ON
3639
cmake --build build
3740
ctest --test-dir build

.github/workflows/build-and-test.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ jobs:
5858
fail-fast: false
5959
container:
6060
env:
61-
BUILD_TYPE: Debug
6261
CC: ${{ matrix.compiler == 'LLVM' && 'clang' || 'gcc' }}
6362
CFLAGS: ${{ (matrix.build_system == 'CMake' && matrix.compiler == 'GNU') && '--coverage' || '' }}
6463
CXX: ${{ matrix.compiler == 'LLVM' && 'clang++' || 'g++' }}
@@ -86,7 +85,6 @@ jobs:
8685

8786
macOS:
8887
env:
89-
BUILD_TYPE: Release
9088
CFLAGS: -Wno-implicit-function-declaration
9189
LDFLAGS: -undefined dynamic_lookup
9290
LIBRARY_PATH: /usr/local/lib

.github/workflows/coverage.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ jobs:
99
name: Build, Test & Report Coverage
1010
runs-on: ubuntu-latest
1111
env:
12+
BUILD_TYPE: Debug
1213
CFLAGS: --coverage
1314
CXXFLAGS: --coverage
1415
steps:

tests/CMakeLists.txt

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,9 @@ set(TILE_PNG256_SHA256SUM "${TILE_DEFAULT_SHA256SUM}")
5555
set(TILE_PNG32_CMD "${CURL_EXECUTABLE} --fail --silent ${TILE_PNG32_URL}")
5656
set(TILE_PNG32_SHA256SUM "1006d92152f1e18896e0016fb43201b14bbcf7655955b74495ad3610541d325b")
5757
set(TILE_WEBP_CMD "${CURL_EXECUTABLE} --fail --silent ${TILE_WEBP_URL}")
58-
set(TILE_WEBP_SHA256SUM_6 "96fc0455b2269a7bcd4a5b3c9844529c3c77e3bb15f56e72f78a5af3bc15b6b5") # libwebp6
59-
set(TILE_WEBP_SHA256SUM_7 "a82ef9ba5dc333de88af7b645084c30ab2b01c664e17162cbf6659c287cc4df4") # libwebp7
58+
set(TILE_WEBP_SHA256SUM_4 "ef3862a57831b21ec69c15be196e1e2b4fea66246c361142631b9fa22b85decc") # libwebp.so.4
59+
set(TILE_WEBP_SHA256SUM_6 "96fc0455b2269a7bcd4a5b3c9844529c3c77e3bb15f56e72f78a5af3bc15b6b5") # libwebp.so.6
60+
set(TILE_WEBP_SHA256SUM_7 "a82ef9ba5dc333de88af7b645084c30ab2b01c664e17162cbf6659c287cc4df4") # libwebp.so.7
6061

6162
configure_file(
6263
renderd.conf.in
@@ -146,23 +147,23 @@ add_test(
146147
NAME download_tiles
147148
COMMAND ${BASH} -c "
148149
until $(${TILE_DEFAULT_CMD} --output tile.png); do
149-
echo 'Sleeping 1s';
150+
echo 'Sleeping 1s (DEFAULT)';
150151
sleep 1;
151152
done
152153
until $(${TILE_JPG_CMD} --output tile.jpg); do
153-
echo 'Sleeping 1s';
154+
echo 'Sleeping 1s (JPG)';
154155
sleep 1;
155156
done
156157
until $(${TILE_PNG256_CMD} --output tile.png256); do
157-
echo 'Sleeping 1s';
158+
echo 'Sleeping 1s (PNG256)';
158159
sleep 1;
159160
done
160161
until $(${TILE_PNG32_CMD} --output tile.png32); do
161-
echo 'Sleeping 1s';
162+
echo 'Sleeping 1s (PNG32)';
162163
sleep 1;
163164
done
164165
until $(${TILE_WEBP_CMD} --output tile.webp); do
165-
echo 'Sleeping 1s';
166+
echo 'Sleeping 1s (WEBP)';
166167
sleep 1;
167168
done
168169
"
@@ -174,7 +175,7 @@ add_test(
174175
(echo '${TILE_JPG_SHA256SUM} tile.jpg' | ${SHA256SUM_EXECUTABLE} -c) && \
175176
(echo '${TILE_PNG256_SHA256SUM} tile.png256' | ${SHA256SUM_EXECUTABLE} -c) && \
176177
(echo '${TILE_PNG32_SHA256SUM} tile.png32' | ${SHA256SUM_EXECUTABLE} -c) && \
177-
((echo '${TILE_WEBP_SHA256SUM_6} tile.webp' | ${SHA256SUM_EXECUTABLE} -c) || (echo '${TILE_WEBP_SHA256SUM_7} tile.webp' | ${SHA256SUM_EXECUTABLE} -c))
178+
((echo '${TILE_WEBP_SHA256SUM_7} tile.webp' | ${SHA256SUM_EXECUTABLE} -c) || (echo '${TILE_WEBP_SHA256SUM_6} tile.webp' | ${SHA256SUM_EXECUTABLE} -c) || (echo '${TILE_WEBP_SHA256SUM_4} tile.webp' | ${SHA256SUM_EXECUTABLE} -c))
178179
"
179180
)
180181
add_test(
@@ -255,12 +256,12 @@ set_tests_properties(render_expired PROPERTIES
255256
set_tests_properties(render_list PROPERTIES
256257
DEPENDS render_speedtest
257258
FIXTURES_REQUIRED httpd_started
258-
TIMEOUT 20
259+
TIMEOUT 60
259260
)
260261
set_tests_properties(render_old PROPERTIES
261262
DEPENDS render_speedtest
262263
FIXTURES_REQUIRED httpd_started
263-
TIMEOUT 20
264+
TIMEOUT 60
264265
)
265266
set_tests_properties(download_tiles PROPERTIES
266267
FIXTURES_REQUIRED httpd_started

tests/renderd.conf.in

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,25 +12,25 @@ XML=@PROJECT_SOURCE_DIR@/utils/example-map/mapnik.xml
1212
TILEDIR=@PROJECT_BINARY_DIR@/tests/tiles
1313
TYPE=jpg image/jpeg jpeg
1414
URI=/tiles/renderd-example-jpg
15-
XML=@PROJECT_BINARY_DIR@/tests/www/mapnik.xml
15+
XML=@PROJECT_SOURCE_DIR@/utils/example-map/mapnik.xml
1616

1717
[@MAP_NAME@_png256]
1818
TILEDIR=@PROJECT_BINARY_DIR@/tests/tiles
1919
TYPE=png image/png png256
2020
URI=/tiles/renderd-example-png256
21-
XML=@PROJECT_BINARY_DIR@/tests/www/mapnik.xml
21+
XML=@PROJECT_SOURCE_DIR@/utils/example-map/mapnik.xml
2222

2323
[@MAP_NAME@_png32]
2424
TILEDIR=@PROJECT_BINARY_DIR@/tests/tiles
2525
TYPE=png image/png png32
2626
URI=/tiles/renderd-example-png32
27-
XML=@PROJECT_BINARY_DIR@/tests/www/mapnik.xml
27+
XML=@PROJECT_SOURCE_DIR@/utils/example-map/mapnik.xml
2828

2929
[@MAP_NAME@_webp]
3030
TILEDIR=@PROJECT_BINARY_DIR@/tests/tiles
3131
TYPE=webp image/webp webp
3232
URI=/tiles/renderd-example-webp
33-
XML=@PROJECT_BINARY_DIR@/tests/www/mapnik.xml
33+
XML=@PROJECT_SOURCE_DIR@/utils/example-map/mapnik.xml
3434

3535
[renderd1]
3636
iphostname=127.0.0.1

0 commit comments

Comments
 (0)