Skip to content

Commit f698e24

Browse files
committed
Disable the consistently failing c-extension linter on latest python3 as it
looks like an issue with the underlying system headers not matching the python installation when bootstrapped with setup-python there. We instead simply rely on the consistent header/python version in latest Ubuntu LTS linter for now, but might want to revisit that at a later point.
1 parent bb6d8e7 commit f698e24

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

.github/workflows/python-c-ext-sanity-check.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,8 @@ permissions:
6767
jobs:
6868
lint-c-ext-python3-latest:
6969
name: Sanity check c-extension module code in latest stable python3
70+
# TODO: enable once we have figured out how to install matching headers here
71+
if: ${{ false }} # Disabled until we figure out how to fix system headers
7072
runs-on: ubuntu-latest
7173
steps:
7274
- name: Set up latest stable python 3.x
@@ -95,7 +97,7 @@ jobs:
9597
[ ! -e splint.log ] || ! grep -q ' Cannot continue' splint.log
9698
9799
lint-c-ext-python3-ubuntu-lts:
98-
# Mainly fallback when the above job doesn't work
100+
# For a consistent python and header environment when the above job doesn't work
99101
name: Sanity check c-extension module code in latest Ubuntu LTS python3
100102
runs-on: ubuntu-24.04
101103
steps:
@@ -127,7 +129,7 @@ jobs:
127129
# - ancient upstream static binary doesn't work ('no such file')
128130
# - installing fedora rpm fails with glibc incompatibility
129131
# - building from git clone fails (autoconf and configure failures)
130-
# TODO: enable once we have figured out how to instal splint on rocky
132+
# TODO: enable once we have figured out how to install splint on rocky
131133
if: ${{ false }} # Disabled until we figure out how to run splint on rocky
132134
runs-on: ubuntu-latest
133135
container:

0 commit comments

Comments
 (0)