Skip to content

chore: remove defunct PyInstaller build path#306

Merged
lstein merged 1 commit into
masterfrom
lstein/chore/remove-pyinstaller
Jun 7, 2026
Merged

chore: remove defunct PyInstaller build path#306
lstein merged 1 commit into
masterfrom
lstein/chore/remove-pyinstaller

Conversation

@lstein
Copy link
Copy Markdown
Owner

@lstein lstein commented Jun 7, 2026

Now that the signed uv-bootstrap launcher (.dmg / setup.exe / AppImage) is verified on all three platforms, remove the superseded PyInstaller machinery:

  • Delete .github/workflows/deploy-pyinstaller.yml (already unwired from deploy.yml).
  • Delete INSTALL/pyinstaller/ (the make_pyinstaller_image.{sh,ps1} build scripts).
  • Drop the pyinstaller dev dependency from pyproject.toml.
  • Remove the now-pointless *.spec lines from .gitignore.
  • Remove the dead sys.frozen / sys._MEIPASS branch in embeddings._clip_root() (PyInstaller-bundle-only code) and its now-unused import sys. The method keeps its signature (return None) so all callers are unaffected.
  • Tidy a stale "Replaces deploy-pyinstaller.yml" comment.

Docs already led with the signed installers (no executable-install section remains). Verified: ruff clean, embeddings imports, 42 embeddings/encoder/imagetool tests pass, YAML/TOML valid.

🤖 Generated with Claude Code

The signed uv-bootstrap launcher fully replaces the PyInstaller bundles. Remove deploy-pyinstaller.yml, INSTALL/pyinstaller/, the pyinstaller dev dependency, the now-unused *.spec gitignore lines, the stale deploy-launcher.yml comment, and the dead sys.frozen/_MEIPASS branch in embeddings._clip_root (plus its now-unused sys import).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@lstein lstein merged commit cf60669 into master Jun 7, 2026
9 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.

1 participant