Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

GH-139596: Cease caching config.cache & ccache in GH Actions#139623

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
gpshead merged 3 commits intopython:mainfromAA-Turner:del-config.cache-cache
Nov 11, 2025

Conversation

@AA-Turner
Copy link
Member

@AA-TurnerAA-Turner commentedOct 5, 2025
edited by bedevere-appbot
Loading

This removes theconfig.cache andccache caches from GitHub Actions. Note that we still install and useccache viaposix-deps-apt.sh, so we will get the benefit from that still (if any), but we will no longer save it to the GHA cache.

A

namurphy reacted with thumbs up emoji
@AA-TurnerAA-Turner changed the titleGH-139596: Cease caching config.cache in GH ActionsGH-139596: Cease caching config.cache & ccache in GH ActionsOct 5, 2025
@mhsmith
Copy link
Member

For future reference:

vstinner reacted with thumbs up emoji

with:
path:config.cache
# Include env.pythonLocation in key to avoid changes in environment when setup-python updates Python
key:${{ github.job }}-${{ env.IMAGE_OS_VERSION }}-${{ needs.build-context.outputs.config-hash }}-${{ env.pythonLocation }}
Copy link
Member

Choose a reason for hiding this comment

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

Can we also removeconfig-hash from reusable-context.yml?

@gpsheadgpshead added needs backport to 3.13bugs and security fixes needs backport to 3.14bugs and security fixes labelsNov 11, 2025
@gpsheadgpshead merged commitc744ccb intopython:mainNov 11, 2025
52 checks passed
@miss-islington-app
Copy link

Thanks@AA-Turner for the PR, and@gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington-app
Copy link

Sorry,@AA-Turner and@gpshead, I could not cleanly backport this to3.14 due to a conflict.
Please backport usingcherry_picker on command line.

cherry_picker c744ccb2c92746bc7be6316ab478dbc13e176e97 3.14

@miss-islington-app
Copy link

Sorry,@AA-Turner and@gpshead, I could not cleanly backport this to3.13 due to a conflict.
Please backport usingcherry_picker on command line.

cherry_picker c744ccb2c92746bc7be6316ab478dbc13e176e97 3.13

@gpshead
Copy link
Member

the backport labels were opportunistic checks - actions configs have been rearranged since

gpshead added a commit to gpshead/cpython that referenced this pull requestNov 11, 2025
path:${{ env.CROSS_BUILD_WASI }}/config.cache
# Should be kept in sync with the other config.cache step above.
key:${{ github.job }}-${{ env.IMAGE_OS_VERSION }}-${{ env.WASI_SDK_VERSION }}-${{ env.WASMTIME_VERSION }}-${{ inputs.config_hash }}-${{ hashFiles('Tools/wasm/wasi/__main__.py') }}-${{ env.pythonLocation }}
run:python3 Tools/wasm/wasi.py make-build-python
Copy link
Member

Choose a reason for hiding this comment

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

Looks like a merge misfire:

Suggested change
run:python3 Tools/wasm/wasi.py make-build-python
run:python3 Tools/wasm/wasi make-build-python

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

Reviewers

@mhsmithmhsmithmhsmith left review comments

@zwarezwarezware left review comments

@gpsheadgpsheadgpshead approved these changes

@ezio-melottiezio-melottiAwaiting requested review from ezio-melottiezio-melotti is a code owner

@hugovkhugovkAwaiting requested review from hugovkhugovk is a code owner

Assignees

@gpsheadgpshead

Labels

needs backport to 3.13bugs and security fixesneeds backport to 3.14bugs and security fixesskip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@AA-Turner@mhsmith@gpshead@zware

[8]ページ先頭

©2009-2025 Movatter.jp