Skip to content

Commit 94aef50

Browse files
committed
ci: run includes-check workflow on debian:13
1 parent 98bdd30 commit 94aef50

3 files changed

Lines changed: 31 additions & 8 deletions

File tree

.github/workflows/includes-check.yml

Lines changed: 29 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,16 @@
11
name: Check includes
22
on: [push, pull_request]
33

4+
defaults:
5+
run:
6+
shell: bash
7+
48
jobs:
59
includes-check:
610
runs-on: ubuntu-latest
11+
container:
12+
image: "debian:13"
13+
options: --user 0
714
strategy:
815
matrix:
916
board: ['HACKRF_ONE', 'JAWBREAKER', 'RAD1O', 'PRALINE']
@@ -13,7 +20,23 @@ jobs:
1320
fail-fast: false
1421

1522
steps:
16-
- uses: actions/checkout@v4
23+
24+
- name: Make it work on debian
25+
run: |
26+
apt update
27+
apt install -y git python3 python3-pip python3-venv nodejs
28+
# actions/checkout insists on putting the checkout in the
29+
# working directory rather than ${{ github.workspace }}.
30+
#
31+
# This may just be because the 'runner' user does not exist
32+
# in the docker image at startup.
33+
#
34+
# also see: https://github.com/actions/runner/issues/878
35+
useradd -ms /bin/bash runner
36+
ln -s /__w /home/runner/work
37+
38+
- name: Checkout repository
39+
uses: actions/checkout@v6
1740
with:
1841
submodules: true
1942

@@ -29,11 +52,10 @@ jobs:
2952
run: |
3053
python3 -m venv environment && source environment/bin/activate
3154
python3 -m pip install PyYAML
32-
sudo apt update
33-
sudo apt install iwyu
55+
apt install -y iwyu
56+
iwyu --version
3457
3558
- name: Build libopencm3
36-
shell: bash
3759
working-directory: ${{github.workspace}}/firmware/libopencm3/
3860
run: |
3961
source ../../environment/bin/activate
@@ -43,13 +65,11 @@ jobs:
4365
run: cmake -E make_directory ${{github.workspace}}/firmware/build
4466

4567
- name: Configure CMake
46-
shell: bash
4768
working-directory: ${{github.workspace}}/firmware/build
48-
run: cmake $GITHUB_WORKSPACE/firmware/ -DCMAKE_BUILD_TYPE=Release -DBOARD=${{ matrix.board }} -DCHECK_INCLUDES=1
69+
run: cmake ${{github.workspace}}/firmware/ -DCMAKE_BUILD_TYPE=Release -DBOARD=${{ matrix.board }} -DCHECK_INCLUDES=1
4970

5071
- name: Build
5172
working-directory: ${{github.workspace}}/firmware/build
52-
shell: bash
5373
run: |
5474
source ../../environment/bin/activate
5575
output="$(cmake --build . --config Release 2>&1)"
@@ -58,8 +78,10 @@ jobs:
5878
if [[ "${line}" == "Warning: include-what-you-use"* ]]; then
5979
exit_code=1
6080
dump=1
81+
echo
6182
elif [[ "${line}" == "---" ]]; then
6283
dump=0
84+
echo
6385
fi
6486
if [[ ${dump} == "1" ]]; then
6587
echo "${line}"

firmware/common/mixer.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
#include "spi_bus.h"
2929
#elif defined(RAD1O)
3030
#include "max2871.h"
31+
#include "rffc5071.h"
3132
#endif
3233

3334
#if defined(JAWBREAKER) || defined(HACKRF_ONE) || defined(PRALINE)

firmware/common/rf_path.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
#include "hackrf_ui.h"
3232
#include "max283x.h"
3333
#include "max5864.h"
34-
#include "mixer.h"
34+
//#include "mixer.h"
3535
#include "platform_detect.h"
3636
#include "sgpio.h"
3737
#if defined(HACKRF_ONE) || defined(RAD1O) || defined(PRALINE)

0 commit comments

Comments
 (0)