Skip to content

ci/circle: fix ccache cache - #12802

Merged
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/fix_circleci_caching
Nov 26, 2024
Merged

ci/circle: fix ccache cache#12802
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/fix_circleci_caching

Conversation

@benoit-pierre

@benoit-pierre benoit-pierre commented Nov 25, 2024

Copy link
Copy Markdown
Member

New ccache versions respect the XDG spec: ~/.ccache~/.cache/ccache.


This change is Reviewable

New ccache versions respect the XDG spec: `~/.ccache` → `~/.cache/ccache`.
@benoit-pierre
benoit-pierre force-pushed the pr/fix_circleci_caching branch from b5e4915 to c3a3aa6 Compare November 25, 2024 22:10
@benoit-pierre
benoit-pierre marked this pull request as draft November 25, 2024 22:13
@benoit-pierre

Copy link
Copy Markdown
Member Author

Why is it different in koreader and base? o.O

koreader:

ccache version 4.10
(default) cache_dir = /home/ko/.cache/ccache

base:

ccache version 4.10
(default) cache_dir = /home/ko/.ccache

@Frenzie

Frenzie commented Nov 25, 2024

Copy link
Copy Markdown
Member

It's different… on the same image with the same .bashrc? I don't have any hypothesis otoh.

@benoit-pierre

Copy link
Copy Markdown
Member Author
▹ docker run -ti --rm koreader/kobase:0.3.4-20.04 bash -c 'ccache --show-config | grep cache_dir'
(default) cache_dir = /home/ko/.cache/ccache
▹ docker run -ti --rm koreader/kobase:0.3.4-20.04 bash -c 'mkdir ~/.ccache && ccache --show-config | grep cach
(default) cache_dir = /home/ko/.ccache

@benoit-pierre
benoit-pierre marked this pull request as ready for review November 25, 2024 23:12
@NiLuJe

NiLuJe commented Nov 25, 2024

Copy link
Copy Markdown
Member

The joys of XDG and backwards compatibility ;D.

@Frenzie Frenzie added this to the 2025.01 milestone Nov 26, 2024
@Frenzie
Frenzie merged commit 5d68ece into koreader:master Nov 26, 2024
@benoit-pierre
benoit-pierre deleted the pr/fix_circleci_caching branch November 26, 2024 15:58
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
New ccache versions respect the XDG spec: `~/.ccache` → `~/.cache/ccache`.
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