Skip to content

fix(inkscape): disable batch_process by default for headless CI#6

Merged
ronaldtse merged 1 commit intov1from
fix-inkscape-batch-process-default
Feb 19, 2026
Merged

fix(inkscape): disable batch_process by default for headless CI#6
ronaldtse merged 1 commit intov1from
fix-inkscape-batch-process-default

Conversation

@ronaldtse
Copy link
Copy Markdown
Contributor

Summary

  • Changed batch_process flag default from true to false in Inkscape 1.0.yaml
  • Updated the description to accurately reflect that this flag forces GUI to show

Background

According to Inkscape command-line documentation:

--batch-process: Close GUI when done. Only use this when you actually need a GUI. Otherwise you usually don't need (or want) this option, as it forces the GUI to show.

The previous configuration had default: true with description "(required for headless)", which is incorrect. This flag was causing Windows CI failures in vectory because:

  1. The --batch-process flag forces the GUI to show
  2. Windows CI runs in a headless environment (no display)
  3. Inkscape would return exit code 0 but fail to create output files

Test Plan

  • After this PR is merged, trigger Windows CI in vectory
  • Verify that Inkscape export commands work correctly without --batch-process
  • Confirm all 28 previously failing tests now pass

According to Inkscape documentation, --batch-process "forces the GUI to
show" and should only be used "when you actually need a GUI". This flag
was causing Windows CI failures because it was trying to launch the GUI
in a headless environment.
@ronaldtse ronaldtse force-pushed the fix-inkscape-batch-process-default branch from db36f8b to ff48a58 Compare February 19, 2026 23:27
@ronaldtse ronaldtse merged commit f650eb4 into v1 Feb 19, 2026
5 checks passed
@ronaldtse ronaldtse deleted the fix-inkscape-batch-process-default branch February 19, 2026 23:27
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