From f9111ebac4d5419254754c990ab55584c7ac5bda Mon Sep 17 00:00:00 2001 From: Justin Harms Date: Thu, 12 Mar 2026 20:55:45 -0500 Subject: [PATCH] Remove xvfb-run dependency from rendering process - Eliminated the use of xvfb-run for headless rendering in the RenderProcessor, simplifying the command execution for Blender. - Updated the CheckRequiredTools function to remove the check for xvfb-run, reflecting the change in rendering requirements. --- internal/runner/runner.go | 4 ---- internal/runner/tasks/render.go | 5 +---- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/internal/runner/runner.go b/internal/runner/runner.go index 0b0ad33..66ed2e9 100644 --- a/internal/runner/runner.go +++ b/internal/runner/runner.go @@ -68,10 +68,6 @@ func (r *Runner) CheckRequiredTools() error { } log.Printf("Found zstd for compressed blend file support") - if err := exec.Command("xvfb-run", "--help").Run(); err != nil { - return fmt.Errorf("xvfb-run not found - required for headless Blender rendering. Install with: apt install xvfb") - } - log.Printf("Found xvfb-run for headless rendering without -b option") return nil } diff --git a/internal/runner/tasks/render.go b/internal/runner/tasks/render.go index 4e9460f..9de80fb 100644 --- a/internal/runner/tasks/render.go +++ b/internal/runner/tasks/render.go @@ -175,10 +175,7 @@ func (p *RenderProcessor) runBlender(ctx *Context, blenderBinary, blendFile, out args = append(args, "-f", fmt.Sprintf("%d", ctx.Frame)) } - // Wrap with xvfb-run - xvfbArgs := []string{"-a", "-s", "-screen 0 800x600x24", blenderBinary} - xvfbArgs = append(xvfbArgs, args...) - cmd := exec.Command("xvfb-run", xvfbArgs...) + cmd := exec.Command(blenderBinary, args...) cmd.Dir = ctx.WorkDir // Set up environment with custom HOME directory