Commit aec274d
committed
MDBF-1197: temporary windows msi builder hotfix regarding HeidiSQL download
windows-packages builder failing with:
```
-- Downloading https://www.heidisql.com/downloads/releases/HeidiSQL_12.14_64_Portable.zip to C:/buildbot/workers/prod/amd64-windows-packages/build/tmpdir/HeidiSQL_12.14_64_Portable/HeidiSQL_12.14_64_Portable.zip
cmake -E tar : error : ZIP decompression failed (-5) [C:\buildbot\workers\prod\amd64-windows-packages\build\win\packaging\MSI.vcxproj]
```
Because a file download 60 seconds timeout is reached
in https://github.com/MariaDB/server/blob/9fc925d77163f1d63e21a65654b0f59678abf08d/win/packaging/heidisql.cmake#L9
The file download status if not checked in cmake, it will be silently ignored
and cmake continues to decompression, resulting in the above error, due to an incomplete download.
CMAKE is looking in TEMP for third parties, setting: THIRD_PARTY_DOWNLOAD_LOCATION
https://github.com/MariaDB/server/blob/9fc925d77163f1d63e21a65654b0f59678abf08d/win/packaging/CMakeLists.txt#L139
But currently TEMP is set to a worker path build location that is removed on every run.
WVlad will permanetly fix this by making THIRD_PARTY_DOWNLOAD_LOCATION settable in 10.6
and also increasing the timeout.
Until then, I manually copied `HeidiSQL_12.14_64_Portable.zip` on the system and user TEMP location
on the windows packages host, and will remove the TEMP env variable from the configuration,
letting the system / user variable take precedence.1 parent e90e567 commit aec274d
1 file changed
+13
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
140 | 140 | | |
141 | 141 | | |
142 | 142 | | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
143 | 153 | | |
144 | 154 | | |
145 | 155 | | |
| |||
374 | 384 | | |
375 | 385 | | |
376 | 386 | | |
377 | | - | |
| 387 | + | |
378 | 388 | | |
379 | 389 | | |
380 | 390 | | |
| |||
403 | 413 | | |
404 | 414 | | |
405 | 415 | | |
406 | | - | |
| 416 | + | |
407 | 417 | | |
408 | 418 | | |
409 | 419 | | |
| |||
420 | 430 | | |
421 | 431 | | |
422 | 432 | | |
423 | | - | |
| 433 | + | |
424 | 434 | | |
425 | 435 | | |
426 | 436 | | |
| |||
0 commit comments