Skip to content

docs: document Windows CI known issues with Inkscape 1.4.x - #61

Merged
ronaldtse merged 1 commit into
mainfrom
rt-ukiryu-windows-fix
Feb 23, 2026
Merged

docs: document Windows CI known issues with Inkscape 1.4.x#61
ronaldtse merged 1 commit into
mainfrom
rt-ukiryu-windows-fix

Conversation

@ronaldtse

Copy link
Copy Markdown
Contributor

Summary

This PR documents the current state of Windows CI and the known issues with Inkscape 1.4.x.

Background

Ukiryu 0.2.1 Fix (Already Applied)

Ukiryu 0.2.1 includes a fix for PowerShell command argument handling on Windows. The fix preserves forward slashes in command arguments instead of converting them to backslashes.

The fix was in ukiryu/lib/ukiryu/shell/powershell.rb:

  • Before: arg_str = a.to_s.gsub('/', '\\') (incorrect)
  • After: arg_str = a.to_s (correct - preserves forward slashes)

Remaining Issue: Inkscape 1.4.x Bug

After the ukiryu fix, Windows CI tests still fail due to an upstream Inkscape bug:

  1. Inkscape 1.4.2 on Windows hangs or fails silently with PDF import
  2. The Ghostscript fallback path also fails because it still uses Inkscape for the final conversion

Current Workaround

Tests use skip helpers:

  • skip_inkscape_on_windows
  • skip_emf_on_windows

Test Results

Platform Status
macOS ✅ Pass
Ubuntu ✅ Pass
Windows ❌ 28 failures (Inkscape bug)

Resolution Options

  1. Pin Inkscape to 1.3.0 on Windows CI
  2. Wait for Inkscape to fix the bug
  3. Implement Ghostscript-only conversion path

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@ronaldtse
ronaldtse force-pushed the rt-ukiryu-windows-fix branch from 14403f2 to e3a8b87 Compare February 19, 2026 09:43
@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@ronaldtse
ronaldtse force-pushed the rt-ukiryu-windows-fix branch from e3a8b87 to a5cf53d Compare February 19, 2026 09:58
@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

10 similar comments
@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@github-actions

Copy link
Copy Markdown

🔗 Link Check Failed


Please fix the broken links and push a new commit.

@ronaldtse
ronaldtse force-pushed the rt-ukiryu-windows-fix branch from 9516eab to ebf23b0 Compare February 23, 2026 03:05
@ronaldtse
ronaldtse merged commit cdc42b2 into main Feb 23, 2026
11 checks passed
@ronaldtse
ronaldtse deleted the rt-ukiryu-windows-fix branch February 23, 2026 03:39
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