Skip to content

Commit f1135a4

Browse files
authored
Add '-Wno-error=deprecated-declarations' to mandatory Rolling CI jobs. (#3706)
Add new optional CI Rolling job that does fail on deprecations
1 parent 848c062 commit f1135a4

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,17 @@ jobs:
2020
- IMAGE: rolling-ci
2121
CCOV: true
2222
ROS_DISTRO: rolling
23+
# Ignore deprecation errors from upstream packages on rolling
24+
SUPPRESS_DEPRECATION_ERROR: true
2325
- IMAGE: rolling-ci
2426
ROS_DISTRO: rolling
2527
IKFAST_TEST: true
2628
CLANG_TIDY: pedantic
29+
SUPPRESS_DEPRECATION_ERROR: true
30+
- IMAGE: rolling-ci
31+
ROS_DISTRO: rolling
32+
# Non-blocking job to surface deprecation warnings
33+
DEPRECATION_CHECK: true
2734
- IMAGE: humble-ci
2835
ROS_DISTRO: humble
2936
- IMAGE: jazzy-ci
@@ -51,7 +58,7 @@ jobs:
5158
-DCMAKE_SHARED_LINKER_FLAGS=-fuse-ld=lld
5259
-DCMAKE_MODULE_LINKER_FLAGS=-fuse-ld=lld
5360
-DCMAKE_BUILD_TYPE=${{ matrix.env.CCOV && 'Debug' || 'Release'}}
54-
-DCMAKE_CXX_FLAGS="-Werror $CXXFLAGS${{ matrix.env.CCOV && ' --coverage -O2 -fno-omit-frame-pointer' || ''}}"
61+
-DCMAKE_CXX_FLAGS="-Werror ${{ matrix.env.SUPPRESS_DEPRECATION_ERROR && '-Wno-error=deprecated-declarations ' || ''}}$CXXFLAGS${{ matrix.env.CCOV && ' --coverage -O2 -fno-omit-frame-pointer' || ''}}"
5562
UPSTREAM_CMAKE_ARGS: "-DCMAKE_CXX_FLAGS=''"
5663
DOWNSTREAM_CMAKE_ARGS: -DCMAKE_CXX_FLAGS="-Wall -Wextra"
5764
CCACHE_DIR: ${{ github.workspace }}/.ccache
@@ -67,8 +74,9 @@ jobs:
6774
CXX: ${{ matrix.env.CLANG_TIDY && 'clang++' }}
6875
ADDITIONAL_DEBS: lld
6976

70-
name: ${{ matrix.env.IMAGE }}${{ matrix.env.CCOV && ' + ccov' || ''}}${{ matrix.env.IKFAST_TEST && ' + ikfast' || ''}}${{ matrix.env.CLANG_TIDY && (github.event_name != 'workflow_dispatch' && ' + clang-tidy (delta)' || ' + clang-tidy (all)') || '' }}
77+
name: ${{ matrix.env.IMAGE }}${{ matrix.env.CCOV && ' + ccov' || ''}}${{ matrix.env.IKFAST_TEST && ' + ikfast' || ''}}${{ matrix.env.CLANG_TIDY && (github.event_name != 'workflow_dispatch' && ' + clang-tidy (delta)' || ' + clang-tidy (all)') || '' }}${{ matrix.env.DEPRECATION_CHECK && ' + deprecation check' || '' }}
7178
runs-on: ubuntu-latest
79+
continue-on-error: ${{ matrix.env.DEPRECATION_CHECK || false }}
7280
steps:
7381
- name: "Free up disk space"
7482
if: matrix.env.CCOV

0 commit comments

Comments
 (0)