Repository navigation
Version 0.1.1: Windows installer .exe with bundled llama.cpp - #1
Merged
Merged
Conversation
- paths: data in %LOCALAPPDATA%, settings in %APPDATA% on Windows - download: free space via shutil.disk_usage, RAM via GlobalMemoryStatusEx, replace() so a finished download never fails on an existing file - localserver: llama-server.exe, no console window, and a kill-on-close job object so it stops with the program, as PR_SET_PDEATHSIG does on Linux - packaging/build_exe.sh + divide-reader.iss: PyInstaller bundle, llama.cpp b11429 cpu + vulkan (SHA-256 pinned), per-user Inno Setup installer - build-exe workflow: tests, build, silent install, start-up and llama-server checks on Windows Co-Authored-By: Claude Opus 5.5 <[email protected]>
Co-Authored-By: Claude Opus 5.5 <[email protected]>
Co-Authored-By: Claude Opus 5.5 <[email protected]>
Co-Authored-By: Claude Opus 5.5 <[email protected]>
andrejristic
changed the title
andrejristic
marked this pull request as ready for review
October 12, 2026 00:25
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a Windows installer,
DivideReader-Setup-<version>.exe, built on GitHub's Windows runner. Linux behaviour is unchanged.Code (Windows only)
paths.py: data goes in%LOCALAPPDATA%\divide-reader, settings in%APPDATA%\divide-reader.download.py: free space comes fromshutil.disk_usage, becauseos.statvfsdoes not exist on Windows. RAM comes fromGlobalMemoryStatusExinstead of/proc/meminfo.replace()is used instead ofrename().localserver.py: runsllama-server.exewith no console window. A kill-on-close job object stops it when the program exits, even when the program is killed. This matches whatPR_SET_PDEATHSIGdoes on Linux.Packaging
packaging/build_exe.sh: builds the PyInstaller bundle and adds llama.cpp b11429 cpu and vulkan builds, the same build as the .deb, with SHA-256 pinned.packaging/divide-reader.iss: per-user Inno Setup installer that needs no admin rights. It adds a Start menu entry and an optional desktop icon.packaging/divide-reader.ico: rendered from the existing SVG.CI:
build-exeruns on tags, PRs and manual dispatchllama-server.exeruns.🤖 Generated with Claude Code