Skip to content

make: tweak setup target#2289

Merged
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/tweak_setup_rule
Mar 6, 2026
Merged

make: tweak setup target#2289
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/tweak_setup_rule

Conversation

@benoit-pierre

@benoit-pierre benoit-pierre commented Mar 6, 2026

Copy link
Copy Markdown
Member

Use cmake --fresh if available (so the cache is ignored).


This change is Reviewable

Use cmake `--fresh` if available (so the cache is ignored).
@Frenzie

Frenzie commented Mar 6, 2026

Copy link
Copy Markdown
Member

Could you explain what that does, or perhaps rather when? Going through the complete long-winded CMake cache generation every time sounds rather unappealing. :-)

@benoit-pierre

Copy link
Copy Markdown
Member Author

Rational for ignoring the cache:

  • you install a new version of SDL, or a new compiler version
  • such a system change is not tracked by CMake
  • but you can run make setup to refresh the CMake config (instead of rebuilding everything)

It also make hacking on CMake files easier (you should always get the same output between to runs of make setup for the same system configuration).

It's only for the top-level cmake project, so it's pretty fast either way:

  • ~1.5s for a fresh setup
  • ~0.4s for a cached setup

Our setup rule is only triggered:

  • explicitly when using make setup
  • when one of our generated cmake files has changed (meson / cmake toolchain definitions, koreader variables)

@Frenzie Frenzie merged commit 946f513 into koreader:master Mar 6, 2026
3 checks passed
@benoit-pierre benoit-pierre deleted the pr/tweak_setup_rule branch March 6, 2026 15:33
benoit-pierre added a commit to benoit-pierre/koreader that referenced this pull request Mar 6, 2026
- zlib: minor build simplification (koreader/koreader-base#2282)
- sdl: build our version for the linux target (koreader/koreader-base#2283)
- fix: SDL3 HiDPI return value check (koreader/koreader-base#2285)
- sdl: ensure we have a recent enough version (koreader/koreader-base#2288)
- make: tweak setup target (koreader/koreader-base#2289)
- sdl: enable mouse focus click-through (koreader/koreader-base#2291)
- luajit: update to 2.1.1772619647 (koreader/koreader-base#2292)
Frenzie pushed a commit to koreader/koreader that referenced this pull request Mar 6, 2026
- zlib: minor build simplification (koreader/koreader-base#2282)
- sdl: build our version for the linux target (koreader/koreader-base#2283)
- fix: SDL3 HiDPI return value check (koreader/koreader-base#2285)
- sdl: ensure we have a recent enough version (koreader/koreader-base#2288)
- make: tweak setup target (koreader/koreader-base#2289)
- sdl: enable mouse focus click-through (koreader/koreader-base#2291)
- luajit: update to 2.1.1772619647 (koreader/koreader-base#2292)
benoit-pierre added a commit to benoit-pierre/koreader that referenced this pull request Mar 6, 2026
Use cmake `--fresh` if available (so the cache is ignored).
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
- zlib: minor build simplification (koreader/koreader-base#2282)
- sdl: build our version for the linux target (koreader/koreader-base#2283)
- fix: SDL3 HiDPI return value check (koreader/koreader-base#2285)
- sdl: ensure we have a recent enough version (koreader/koreader-base#2288)
- make: tweak setup target (koreader/koreader-base#2289)
- sdl: enable mouse focus click-through (koreader/koreader-base#2291)
- luajit: update to 2.1.1772619647 (koreader/koreader-base#2292)
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