Repository navigation
Conversation
The Windows release jobs already build solvespace-cli.exe alongside the GUI executable, but only the GUI binary is renamed and uploaded, so the CLI is discarded at the end of every release build. No official Windows download therefore provides the command-line interface, and the GUI executable treats its argument as a file to open rather than as CLI options, so the headless export and thumbnail functionality documented in the README is unavailable to Windows users unless they build from source. Rename and upload solvespace-cli.exe next to each GUI variant it is built with, and add the four binaries to the release assets.
Member
|
Consider my comment on the issue here #1770 (comment) before deciding whether to merge this. |
github-actions Bot
added a commit
to Foadsf/solvespace
that referenced
this pull request
Sep 8, 2026
Member
|
@Foadsf and @ruevs This is a bigger change proposal, but what if we added all the command line options to the GUI app, along with a new --headless or --nogui option? As long as the GUI is dynamically linked, this should even allow it to run headless on servers that don't have a GUI installed, and it would eliminate the need to build a separate executable for CLI use. |
github-actions Bot
added a commit
to Foadsf/solvespace
that referenced
this pull request
Sep 26, 2026
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.
Fixes #1770.
The Windows release jobs already build
solvespace-cli.exe— it shows up in every CD run:but
build-windows.shrenames onlysolvespace.exe, and each job uploads only that one path, so the CLI is discarded at the end of the build. macOS ships it inside the app bundle and Linux distributions package it; Windows is the only platform where it cannot be obtained without compiling.This renames
solvespace-cli.exewith the same suffix as the GUI binary it was built alongside, uploads it in the same artifact, and adds the four files to the release assets. No build configuration changes — the binary is already produced by the existingENABLE_CLIdefault, and it is statically linked (/MT, in-tree cairo/libpng/zlib) like the GUI executable, so it needs no extra runtime files.Verified by running the modified workflow end to end on a fork. All four Windows release jobs passed, and the
windows_x86artifact contained both binaries:Running that
solvespace-cli_x86.exeon a clean Windows 11 machine, with nothing else installed:Happy to trim this to just the two OpenMP builds if eight Windows assets is more than you want on the release page.