Page MenuHomePhabricator

Use Shell\Command in SyntaxHighlight instead of symfony/process
Closed, ResolvedPublic

Description

From my commit message:

We originally started using symfony/process because kzykhys/pygments
depended upon it. But that library was unmaintained and became broken,
so we stopped using it, and just used symfony/process directly.

Today the only reason in favor of symfony/process is that it can pass
stdin to pygments, while Shell\Command can't (T182463). But there are
downsides, like not respecting the default MediaWiki shell limits,
cannot be used with core's firejail support, and requiring an external
composer dependency.

Event Timeline

Change 396476 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[mediawiki/extensions/SyntaxHighlight_GeSHi@master] Drop symfony/process dependency, use Shell\Command instead

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

Change 396476 merged by jenkins-bot:
[mediawiki/extensions/SyntaxHighlight_GeSHi@master] Drop symfony/process dependency, use Shell\Command instead

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

Krinkle removed a project: Patch-For-Review.