Skip to content

Commit

Permalink
amend! ci: Use a container and expand versions
Browse files Browse the repository at this point in the history
ci: Use a container and expand versions

Use containers to build/test so it's not restricted to the OS versions
supported by Github runners. A few changes are needed to the workflow to
support using containers:

	1) No need for sudo, so remove it
	2) Explicitly install missing zlib
	3) Explicitly pass KDIR= while building kernel modules, since it
	   won't match `uname -r`. This assumes the container has just
	   one kernel installed and so /usr/lib/modules/*/build can be
	   used as the single symlink to the kernel headers. This should
	   be common to other distros to be added, too.

Signed-off-by: Lucas De Marchi <[email protected]>
  • Loading branch information
lucasdemarchi committed Jul 12, 2024
1 parent 78d2ebb commit ce34b6c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
- name: test
run: |
cd build
make -j$(nproc) check
make KDIR=/usr/lib/modules/*/build -j$(nproc) check
- name: install
run: |
Expand All @@ -44,4 +44,4 @@ jobs:
- name: distcheck
run: |
cd build
make distcheck
make KDIR=/usr/lib/modules/*/build distcheck

0 comments on commit ce34b6c

Please sign in to comment.