Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

arm64: make sure the cache size stays in check #76

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lazka
Copy link
Member

@lazka lazka commented May 29, 2023

since we disable caching now we also no longer prune the cache

lazka referenced this pull request May 29, 2023
it's not really helping in case of self-hosted runners,
so just disable it there.
@jeremyd2019
Copy link
Member

What does the mtime represent? Some tools set the mtime to the last modified header when downloading, so I don't think it would be very meaningful in this context.

@lazka
Copy link
Member Author

lazka commented May 29, 2023

right, I tested and pacman keeps the original mtime. So we should use atime instead.

since we disable caching now we also no longer prune the cache
@jeremyd2019
Copy link
Member

I'm not sure atime works either ☹️ . The expression paccache uses is find "$PWD" -name '*.pkg.tar*.sig' -prune -o \( -name '*.pkg.tar*' -printf '%A@ %T@ %p\n' \) and when I run that I find a different atime (first column) every run, so I think atime gets updated by the check 🤦

@jeremyd2019
Copy link
Member

jeremyd2019 commented May 29, 2023

I think I'd either just do what setup-msys2 does (-rfuk0, -rfk1), or just ignore it and see what happens.

@lazka lazka marked this pull request as draft May 30, 2023 07:34
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.

2 participants