Skip to content

Windows command wrappers#212

Merged
tedwaine merged 1 commit intoAcademySoftwareFoundation:developfrom
bdeluca:pr/windows-cmd-wrappers
Feb 23, 2026
Merged

Windows command wrappers#212
tedwaine merged 1 commit intoAcademySoftwareFoundation:developfrom
bdeluca:pr/windows-cmd-wrappers

Conversation

@bdeluca
Copy link

@bdeluca bdeluca commented Feb 20, 2026

Linked issues

Fixes #211

Summarize your change.

  • Add .cmd batch wrappers for xstudio-control, xstudio-inject, and xstudiopy so the Python CLI tools work on Windows
  • Remove the non-functional xstudio.bat.in template, which contained Linux-only constructs (LD_LIBRARY_PATH, xstudio_desktop_integration)

Describe the reason for the change.

Better windows support

Describe what you have tested and on which operating system.

Tested on win/lin

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 20, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: bdeluca / name: Ben de Luca (0a75ab7)

On Windows, extensionless shell scripts aren't executable. Add .cmd
batch wrappers for xstudio-control, xstudio-inject, and xstudiopy
that set PYTHONPATH and invoke the bundled Python interpreter. The
existing Linux/macOS scripts are unchanged — CMake selects the right
variant per platform.

Remove the non-functional xstudio.bat.in template, which was a
copy of the Linux shell launcher containing Linux-only constructs
(LD_LIBRARY_PATH, xstudio_desktop_integration). On Windows,
xstudio.exe is invoked directly and needs no wrapper.

Signed-off-by: Ben De Luca <bdeluca@gmail.com>
@bdeluca bdeluca force-pushed the pr/windows-cmd-wrappers branch from 10c6d55 to 0a75ab7 Compare February 20, 2026 09:49
@bdeluca bdeluca changed the title 211: windows command wrappers Windows command wrappers Feb 20, 2026
@tedwaine tedwaine merged commit 86e3cde into AcademySoftwareFoundation:develop Feb 23, 2026
2 checks passed
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.

2 participants