Hashar has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/348228 )
Change subject: Shallow stderr when running convert --version
......................................................................
Shallow stderr when running convert --version
On Wikimedia, $wgImageMagickConvertCommand points to a shell wrapper
that invokes ImageMagick convert with:
firejail --profile=/etc/firejail/mediawiki-converters.profile
firejail emits to stderr an information message:
Reading profile /etc/firejail/mediawiki-converters.profile
That ends up to HHVM stderr and is populated up to logstash as an error.
MediaWiki does check imagemagick version by running `convert -version`,
switch from wfShellExec() to wfShellExecWithStderr().
Bug: T158649
Change-Id: I78d1ef59533c605f59b42e10556bb595d6c2cc15
---
M includes/media/TransformationalImageHandler.php
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/28/348228/1
diff --git a/includes/media/TransformationalImageHandler.php
b/includes/media/TransformationalImageHandler.php
index 1ab0f36..2a74e0d 100644
--- a/includes/media/TransformationalImageHandler.php
+++ b/includes/media/TransformationalImageHandler.php
@@ -521,7 +521,7 @@
$cmd = wfEscapeShellArg(
$wgImageMagickConvertCommand ) . ' -version';
wfDebug( $method . ": Running convert
-version\n" );
$retval = '';
- $return = wfShellExec( $cmd, $retval );
+ $return = wfShellExecWithStderr( $cmd, $retval
);
$x = preg_match(
'/Version: ImageMagick
([0-9]*\.[0-9]*\.[0-9]*)/', $return, $matches
);
--
To view, visit https://gerrit.wikimedia.org/r/348228
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I78d1ef59533c605f59b42e10556bb595d6c2cc15
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Hashar <has...@free.fr>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
