Skip to content

Сборка для macOS Intel на Intel-машине - #2

Merged
bboyJohnn merged 1 commit into
mainfrom
claude/festive-ptolemy-2hrfx1
Oct 19, 2026
Merged

bboyJohnn merged 1 commit into
mainfrom
claude/festive-ptolemy-2hrfx1

Conversation

@bboyJohnn

Copy link
Copy Markdown
Owner

Проблема

В релизе v3.0 нет CineConvert-macos-intel.dmg.

Задание «macOS Intel» шло на ARM-машине macos-14. Python из setup-python с architecture: x64 там всё равно работал как arm64. В логе это видно по строкам Platform: macOS-14.8.9-arm64 и EXE target arch: arm64. В итоге получилась вторая сборка для Apple Silicon с тем же именем файла, и при слиянии артефактов она заменила первую.

Что изменено

  • Runner: macOS Intel собирается на macos-15-intel — это настоящая Intel-машина.
  • Имя .dmg берётся из архитектуры собранной программы (lipo -archs), а не из машины сборки. Файл теперь не может получить чужое имя.
  • Проверка в задании release: до публикации проверяется, что на месте все 5 файлов. Иначе релиз не выходит.

Проверка

  • pyflakes build_release.py проходит.
  • Проверка файлов локально падает, когда нет Intel-.dmg, и проходит, когда есть все пять.
  • Сборка для 4 систем будет проверена CI этого PR.

Generated by Claude Code

On the ARM macos-14 runner the x64 Python ran natively as arm64, so the
Intel job built a second Apple Silicon app with the same name and the
release ended up without the Intel dmg.

- macOS Intel builds on macos-15-intel
- the dmg is named from the architecture of the built executable (lipo)
- the release job checks that all five files exist before publishing

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BAXRzRGWDemk9sYyF2v18g
@bboyJohnn
bboyJohnn marked this pull request as ready for review October 19, 2026 17:19
@bboyJohnn
bboyJohnn merged commit d1df033 into main Oct 19, 2026
5 checks passed
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