Skip to content

Commit 04c29bd

Browse files
pks-tgitster
authored andcommitted
ci: merge linux-gcc-default into linux-gcc
The "linux-gcc-default" job is mostly doing the same as the "linux-gcc" job, except for a couple of minor differences: - We use an explicit GCC version instead of the default version provided by the distribution. We have other jobs that test with "gcc-8", making this distinction pointless. - We don't set up the Python version explicitly, and instead use the default Python version. Python 2 has been end-of-life for quite a while now though, making this distinction less interesting. - We set up the default branch name to be "main" in "linux-gcc". We have other testcases that don't and also some that explicitly use "master". - We use "ubuntu:20.04" in one job and "ubuntu:latest" in another. We already have a couple other jobs testing these respectively. So overall, the job does not add much to our test coverage. Drop the "linux-gcc-default" job and adapt "linux-gcc" to start using the default GCC compiler, effectively merging those two jobs into one. Signed-off-by: Patrick Steinhardt <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent c5bc9a7 commit 04c29bd

File tree

3 files changed

+0
-13
lines changed

3 files changed

+0
-13
lines changed

.github/workflows/main.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,6 @@ jobs:
271271
pool: ubuntu-latest
272272
- jobname: linux-gcc
273273
cc: gcc
274-
cc_package: gcc-8
275274
pool: ubuntu-20.04
276275
- jobname: linux-TEST-vars
277276
cc: gcc
@@ -286,9 +285,6 @@ jobs:
286285
- jobname: osx-gcc
287286
cc: gcc-13
288287
pool: macos-13
289-
- jobname: linux-gcc-default
290-
cc: gcc
291-
pool: ubuntu-latest
292288
- jobname: linux-leaks
293289
cc: gcc
294290
pool: ubuntu-latest

.gitlab-ci.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,14 +46,10 @@ test:linux:
4646
- jobname: linux-gcc
4747
image: ubuntu:20.04
4848
CC: gcc
49-
CC_PACKAGE: gcc-8
5049
- jobname: linux-TEST-vars
5150
image: ubuntu:20.04
5251
CC: gcc
5352
CC_PACKAGE: gcc-8
54-
- jobname: linux-gcc-default
55-
image: ubuntu:latest
56-
CC: gcc
5753
- jobname: linux-leaks
5854
image: ubuntu:latest
5955
CC: gcc

ci/lib.sh

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -328,11 +328,6 @@ export SKIP_DASHED_BUILT_INS=YesPlease
328328

329329
case "$distro" in
330330
ubuntu-*)
331-
if test "$jobname" = "linux-gcc-default"
332-
then
333-
break
334-
fi
335-
336331
# Python 2 is end of life, and Ubuntu 23.04 and newer don't actually
337332
# have it anymore. We thus only test with Python 2 on older LTS
338333
# releases.

0 commit comments

Comments
 (0)