Skip to content

gh-148806: Bump LLVM in CI sanitizer tests to 22 - #148805

Closed
clin1234 wants to merge 4 commits into
python:mainfrom
clin1234:patch-2
Closed

gh-148806: Bump LLVM in CI sanitizer tests to 22#148805
clin1234 wants to merge 4 commits into
python:mainfrom
clin1234:patch-2

Conversation

@clin1234

@clin1234clin1234 commented Apr 20, 2026•
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

@clin1234clin1234 changed the title ci: Bump LLVM in CI sanitizer tests to 22gh-148806: Bump LLVM in CI sanitizer tests to 22Apr 20, 2026
@bedevere-appbedevere-appBot mentioned this pull request Apr 20, 2026
Closed
StanFromIreland
StanFromIreland reviewed Apr 20, 2026
View reviewed changes
Comment threadMisc/NEWS.d/next/Tests/2026-04-20-19-27-54.gh-issue-148806.7E01Ob.rst Outdated

@StanFromIrelandStanFromIrelandApr 20, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please remove this blurb, they aren't necessary for development tools.

ashm-dev
ashm-dev reviewed Apr 20, 2026
View reviewed changes
Comment thread.github/workflows/build.yml
- name: Install dependencies
run: sudo ./.github/workflows/posix-deps-apt.sh
- name: Set up GCC-10 for ASAN
- name: Set up GCC-14 for ASAN

@ashm-devashm-devApr 20, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Any chance we could bump this to GCC 15? It's already stable.

@clin1234clin1234Apr 20, 2026

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

I don't think GCC 15 is available for Ubuntu 24.04...

@StanFromIrelandStanFromIreland added skip news infra CI, GitHub Actions, buildbots, Dependabot, etc. labels Apr 20, 2026
@ByteFlowing1337

ByteFlowing1337 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

I think we should close this PR and its related issue since #152781 has been merged?
cc @vstinner

@StanFromIreland

StanFromIreland commented Jul 3, 2026

Copy link
Copy Markdown
Member

Yes, thanks for noticing. GCC updated in #152717.

@clin1234
clin1234 deleted the patch-2 branch July 3, 2026 12:46
@vstinner

vstinner commented Jul 3, 2026

Copy link
Copy Markdown
Member

Yes, thanks for noticing. GCC updated in #152717.

I removed the step installing GCC 10 explicitly, so GCC 13 is used instead: #152782.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@StanFromIrelandStanFromIrelandStanFromIreland left review comments
@ezio-melottiezio-melottiAwaiting requested review from ezio-melottiezio-melotti is a code owner
@hugovkhugovkAwaiting requested review from hugovkhugovk is a code owner
@AA-TurnerAA-TurnerAwaiting requested review from AA-TurnerAA-Turner is a code owner
@webknjazwebknjazAwaiting requested review from webknjazwebknjaz is a code owner
+1 more reviewer
@ashm-devashm-devashm-dev left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

awaiting reviewinfraCI, GitHub Actions, buildbots, Dependabot, etc.skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@clin1234@ByteFlowing1337@StanFromIreland@vstinner@ashm-dev