Commit 3327509
committed
Improve Linux build stability and model download robustness
Cross-platform build improvements:
- Add Linux-specific linker configuration in Cargo config
- Use bundled whisper-rs-sys bindings to avoid libclang/libLLVM coupling
- Update .envrc for better cross-platform compiler detection
- Add ELF interpreter patching in build.sh for Nix-built binaries
Model downloader improvements:
- Add atomic file operations with temp file cleanup
- Implement proper checksum verification with error recovery
- Add retry logic with exponential backoff for failed downloads
- Clean up stale temporary files before new downloads
- Verify models exist before listing as downloaded
These changes ensure stable builds across different Linux environments
(Nix, system, mixed) and more reliable model downloads.
Fix Windows build by making WHISPER_DONT_GENERATE_BINDINGS target-specific
The WHISPER_DONT_GENERATE_BINDINGS flag was set globally, causing
whisper-rs-sys to use bundled (Linux) bindings on Windows, which led
to compilation errors due to platform-specific struct size mismatches.
Now only Linux and macOS targets use bundled bindings, while Windows
generates bindings at build time as intended.
Fix CI builds by setting WHISPER_DONT_GENERATE_BINDINGS env var
The target-specific environment variables in .cargo/config.toml only
apply when building with explicit --target flags. CI builds without
--target flags don't pick up these settings, causing whisper-rs-sys
to try building from source and fail.
Solution: Set WHISPER_DONT_GENERATE_BINDINGS=1 as an environment
variable in the CI and release workflows for Linux and macOS.
Windows correctly generates bindings at build time and doesn't
need this flag.
Fix CI cache issue preventing WHISPER_DONT_GENERATE_BINDINGS from taking effect
- Update cache key from v1 to v2 to bust old cache
- Add explicit cargo clean -p whisper-rs-sys to force rebuild with correct env
- This ensures whisper-rs-sys uses bundled bindings instead of building from source
fix
Fix TUI default model to use ggml-base.en without .bin extension
The TUI was using 'ggml-base.en.bin' as the default model path, but
the model registry and Rust code use model IDs without file extensions
(e.g., 'ggml-base.en'). This caused the TUI to create configs with
incorrect model paths.
Now matches the Rust default in src/config.rs1 parent 9b90337 commit 3327509
File tree
10 files changed
+402
-237
lines changed- .cargo
- .github/workflows
- packaging/linux
- scripts
- src/models
- tui/src/data
10 files changed
+402
-237
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
4 | 7 | | |
5 | 8 | | |
6 | 9 | | |
| |||
13 | 16 | | |
14 | 17 | | |
15 | 18 | | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | | - | |
3 | | - | |
| 1 | + | |
| 2 | + | |
4 | 3 | | |
5 | | - | |
6 | | - | |
7 | | - | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
46 | | - | |
| 46 | + | |
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
| |||
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
61 | 65 | | |
62 | 66 | | |
63 | 67 | | |
64 | 68 | | |
65 | 69 | | |
66 | 70 | | |
67 | 71 | | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
68 | 76 | | |
69 | 77 | | |
70 | 78 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| 52 | + | |
52 | 53 | | |
53 | 54 | | |
54 | 55 | | |
| |||
100 | 101 | | |
101 | 102 | | |
102 | 103 | | |
| 104 | + | |
| 105 | + | |
103 | 106 | | |
104 | 107 | | |
105 | 108 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
91 | 99 | | |
92 | 100 | | |
93 | 101 | | |
| |||
237 | 245 | | |
238 | 246 | | |
239 | 247 | | |
240 | | - | |
| 248 | + | |
241 | 249 | | |
242 | 250 | | |
243 | 251 | | |
244 | 252 | | |
| 253 | + | |
| 254 | + | |
245 | 255 | | |
246 | 256 | | |
247 | 257 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
13 | 46 | | |
14 | 47 | | |
15 | 48 | | |
| |||
107 | 140 | | |
108 | 141 | | |
109 | 142 | | |
| 143 | + | |
| 144 | + | |
110 | 145 | | |
111 | 146 | | |
112 | 147 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
17 | 16 | | |
18 | 17 | | |
19 | 18 | | |
0 commit comments