Skip to content

Comments

Fix multiprocessing freeze and check bundled ffmpeg first#52

Merged
ruzin merged 1 commit intoruzin:mainfrom
sophiamariem:fix/multiprocessing-freeze-and-bundled-ffmpeg
Feb 23, 2026
Merged

Fix multiprocessing freeze and check bundled ffmpeg first#52
ruzin merged 1 commit intoruzin:mainfrom
sophiamariem:fix/multiprocessing-freeze-and-bundled-ffmpeg

Conversation

@sophiamariem
Copy link
Contributor

@sophiamariem sophiamariem commented Feb 22, 2026

Summary

  • Add multiprocessing.freeze_support() call before CLI entry point to prevent freezing issues in PyInstaller-bundled builds
  • Check bundled ffmpeg (shipped with the app) before falling back to system paths, ensuring the app works without a system ffmpeg installation

Test plan

  • Verify PyInstaller build completes without multiprocessing-related errors
  • Verify ffmpeg setup finds bundled binary in packaged app
  • Verify fallback to system ffmpeg still works when bundled binary is absent

- Add multiprocessing.freeze_support() before CLI entry point to prevent
  PyInstaller child processes from passing Python flags (-B) to the bundled
  executable. Fixes "Error: No such option: -B" during Whisper model download
  on Intel Macs where slow Metal init causes resource_tracker respawn.

- Update setup-ffmpeg handler to check bundled ffmpeg path before system paths,
  avoiding unnecessary Homebrew installation when ffmpeg is already shipped
  in the app bundle.
@ruzin ruzin self-requested a review February 23, 2026 18:17
Copy link
Owner

@ruzin ruzin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ruzin ruzin merged commit 2cc2e9d into ruzin:main Feb 23, 2026
ruzin pushed a commit that referenced this pull request Feb 23, 2026
- Fix Intel Mac EBADEXEC error (#50)
- Improved Ollama setup diagnostics
- Check bundled ffmpeg first in setup wizard (#52)
- Add multiprocessing freeze_support (#52)
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