Description

Executing Shell::command( $wgScoreLilyPond, '--version' ) returns score-versionerr with the message:

'c:\LilyPond\usr\bin\lilypond.exe" "--version' is not recognized as an internal or external command, operable program, or batch file.

Notably, the quotation marks are quite odd in the error message. This works under Vagrant though. It only fails on Windows. Extending the issue, this probably also affects the rendering stage.

Related Objects

Mentioned In: T208841: git-changed-in-head breaks with paths with spaces in
T204475: update.php ends without messages after MCR migration when using WinCache
T199789: Extension:SyntaxHighlight fail to invoke Pygments.exe on Windows (probably shell exec is broken on Windows)

Event Timeline

Ebe123 created this task.Dec 28 2017, 11:14 PM

Shell::command( $wgScoreLilyPond . '--version' ) works (notice concatenation instead of parametration) on Windows.

Ebe123 added a subscriber: Legoktm.Dec 29 2017, 2:10 AM

MaxSem added a subscriber: MaxSem.Feb 26 2018, 9:36 PM

Daimona merged a task: T199789: Extension:SyntaxHighlight fail to invoke Pygments.exe on Windows (probably shell exec is broken on Windows).Jul 19 2018, 9:37 AM

Daimona added subscribers: Glanthor_Reviol, Daimona.

Affected as well.

Daimona mentioned this in T199789: Extension:SyntaxHighlight fail to invoke Pygments.exe on Windows (probably shell exec is broken on Windows).Aug 16 2018, 4:45 PM

Daimona mentioned this in T204475: update.php ends without messages after MCR migration when using WinCache.Sep 21 2018, 11:42 AM

osorio-juan-microsoft claimed this task.Oct 24 2018, 4:21 PM

I have an easy fix for this issue in particular (does not fix the other shell issue related to non-blocking pipes in Windows). Should I send a PR against master for mediawiki/core repo?

In T183759#4692071, @osorio-juan-microsoft wrote:

I have an easy fix for this issue in particular (does not fix the other shell issue related to non-blocking pipes in Windows). Should I send a PR against master for mediawiki/core repo?

On gerrit, yes

Change 471885 had a related patch set uploaded (by Juan Osorio (Microsoft); owner: Juan Osorio (Microsoft)):
[mediawiki/core@master] Fixes shell edge-cases in Windows

https://gerrit.wikimedia.org/r/471885

gerritbot added a project: Patch-For-Review.Nov 6 2018, 12:41 AM

Reedy mentioned this in T208841: git-changed-in-head breaks with paths with spaces in.Nov 6 2018, 11:20 AM

Krenair added a subscriber: Krenair.Nov 9 2018, 6:25 PM

Ebe123 merged a task: T221776: Score Extension can't run on MediaWiki for Windows due to lilypond.exe not recognized.Apr 24 2019, 5:14 PM

Ebe123 added a subscriber: dzeaman.

Change 471885 had a related patch set uploaded (by TheDJ; owner: Juan Osorio (Microsoft)):
[mediawiki/core@master] Fixes shell edge-cases in Windows

https://gerrit.wikimedia.org/r/471885