There are three commits in wsexport production that do not exist in the codebase we've been working from.
commit 1801ceafc3a0b02e3455aea93a14cb906e4af5bf Author: tools.wsexport <tools.wsexport@tools-sgebastion-07.tools.eqiad.wmflabs> Date: Tue Mar 26 17:10:45 2019 +0000 Avoids using grid for Calibre calls commit c39e0120f052fdb9882296062810c93eb90ac641 Author: tools.wsexport <tools.wsexport@tools-bastion-03.tools.eqiad.wmflabs> Date: Wed Apr 19 12:46:27 2017 +0000 Ignore calibre stdout outputs commit 35425a109242be251bb1656eccbeac836221f27b Author: tools.wsexport <tools.wsexport@tools-bastion-03.tools.eqiad.wmflabs> Date: Wed Jun 29 19:55:03 2016 +0000 Set ebook-converter config
The diff between the last common commit and master-in-production is:
tools.wsexport@tools-sgebastion-07:~/tool$ git diff 0597fc2cc HEAD diff --git a/http/book.php b/http/book.php index 2c56961..4dc1311 100644 --- a/http/book.php +++ b/http/book.php @@ -1,6 +1,9 @@ <?php $wsexportConfig = [ - 'basePath' => '..', 'tempPath' => __DIR__ . '/../temp', 'stat' => true + 'basePath' => '..', + 'tempPath' => __DIR__ . '/../temp', + 'stat' => true, + 'ebook-convert' => 'xvfb-run -a ebook-convert' ]; include_once __DIR__ . '/../book/init.php';
i.e. it changes the ebook-convert command to xvfb-run -a ebook-convert.
This customization should happen in an instillation-specific config file. We should add such a thing.