jenkins-bot has submitted this change and it was merged.
Change subject: Pass additional details to logger on XeLaTeX failures.
......................................................................
Pass additional details to logger on XeLaTeX failures.
Requires Icafbc480be9173b33f1168d1d8adbfc1a3433c91
Change-Id: I468171dcd1b2a2a6087f94ef54110a4e5db6d590
---
M bin/mw-ocg-latexer
M lib/index.js
2 files changed, 7 insertions(+), 0 deletions(-)
Approvals:
Cscott: Looks good to me, approved
jenkins-bot: Verified
diff --git a/bin/mw-ocg-latexer b/bin/mw-ocg-latexer
index ac65465..bfa6e36 100755
--- a/bin/mw-ocg-latexer
+++ b/bin/mw-ocg-latexer
@@ -98,6 +98,9 @@
if ( err instanceof Error ) {
msg.message = err.message;
msg.stack = err.stack;
+ if (err.stdout || err.stderr) {
+ msg.details = { log: err.stdout, err: err.stderr };
+ }
} else {
msg.message = '' + err;
}
diff --git a/lib/index.js b/lib/index.js
index ab17b40..e63cae6 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -1769,6 +1769,10 @@
then(function(readStream) {
readStream.pipe(writeStream);
return deferred.promise;
+ }, function(err) {
+ // we use 'exitCode' to communicate exit status
+ err.exitCode = err.code;
+ throw err;
});
}
};
--
To view, visit https://gerrit.wikimedia.org/r/159401
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I468171dcd1b2a2a6087f94ef54110a4e5db6d590
Gerrit-PatchSet: 1
Gerrit-Project:
mediawiki/extensions/Collection/OfflineContentGenerator/latex_renderer
Gerrit-Branch: master
Gerrit-Owner: Cscott <[email protected]>
Gerrit-Reviewer: Cscott <[email protected]>
Gerrit-Reviewer: GWicke <[email protected]>
Gerrit-Reviewer: MaxSem <[email protected]>
Gerrit-Reviewer: Mwalker <[email protected]>
Gerrit-Reviewer: Subramanya Sastry <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
