Skip to content

feat(build): add gcc8 ci to avoid some test failure#194

Merged
zjw1111 merged 9 commits into
alibaba:mainfrom
SGZW:zhangwei/add_ci
Mar 30, 2026
Merged

feat(build): add gcc8 ci to avoid some test failure#194
zjw1111 merged 9 commits into
alibaba:mainfrom
SGZW:zhangwei/add_ci

Conversation

@SGZW
Copy link
Copy Markdown
Contributor

@SGZW SGZW commented Mar 22, 2026

Purpose

Add a GCC 8 pipeline since new code tests frequently fail on GCC 8 like #24

Tests

API and Format

Documentation

Generative AI tooling

@SGZW SGZW force-pushed the zhangwei/add_ci branch 2 times, most recently from 6a31ade to 4c3c213 Compare March 23, 2026 06:38
@SGZW
Copy link
Copy Markdown
Contributor Author

SGZW commented Mar 23, 2026

@lucasfang @lxy-9602 can you provide some Ubuntu 20.04 runners? Thanks.

@lxy-9602
Copy link
Copy Markdown
Collaborator

@lucasfang @lxy-9602 can you provide some Ubuntu 20.04 runners? Thanks.

@SGZW
I think adding a dedicated CI job to catch compiler-specific issues like GCC 8 is a great idea.

It looks like ubuntu-20.04 might be too old for some of our current tooling — GitHub's hosted runners now recommend newer images (e.g., ubuntu-22.04). For reference:

Alternatively, do you consider setting up a GCC 8 test pipeline on the existing Ubuntu image by manually installing GCC 8? We could use install gcc-8 and g++-8 and then set the compiler explicitly in CMake.

@zjw1111
Copy link
Copy Markdown
Collaborator

zjw1111 commented Mar 23, 2026

Could you also modify the supported version to gcc8 in the document at docs/source/building.rst? Thanks!

@SGZW SGZW force-pushed the zhangwei/add_ci branch from 4c3c213 to be09b53 Compare March 23, 2026 09:13
@SGZW SGZW force-pushed the zhangwei/add_ci branch 9 times, most recently from 2c340e3 to 56d59de Compare March 24, 2026 15:41
@SGZW SGZW force-pushed the zhangwei/add_ci branch from 56d59de to ea3c275 Compare March 24, 2026 15:56
@SGZW SGZW force-pushed the zhangwei/add_ci branch 3 times, most recently from b62f3a9 to 104fad0 Compare March 29, 2026 13:19
@SGZW SGZW force-pushed the zhangwei/add_ci branch 3 times, most recently from 5a626a2 to 061a81e Compare March 29, 2026 14:57
@SGZW SGZW force-pushed the zhangwei/add_ci branch from 061a81e to 61e7a30 Compare March 29, 2026 15:03
@SGZW SGZW force-pushed the zhangwei/add_ci branch from 61e7a30 to 40ac7c3 Compare March 29, 2026 15:06
@SGZW
Copy link
Copy Markdown
Contributor Author

SGZW commented Mar 29, 2026

@lxy-9602 @zjw1111 PTAL

Copy link
Copy Markdown
Collaborator

@zjw1111 zjw1111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@zjw1111 zjw1111 merged commit 2696377 into alibaba:main Mar 30, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants