running make pdfexport in terminal produces error

Bug #1914298 reported by Lisa Seeto
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Robert Lyon
20.04
Fix Released
Medium
Unassigned
20.10
Fix Released
Medium
Unassigned

Bug Description

In order to enable PDF exports on the site we must enable the pdf plugin which requires "chrome-php" via the terminal command: "make pdfexport". While running this command an error is produced and the process fails with the following:

make pdfexport
Composer already installed...
Updating external dependencies with Composer...
Loading composer repositories with package information
Warning from https://repo.packagist.org: You are using an outdated version of Composer. Composer 2.0 is now available and you should upgrade. See https://getcomposer.org/2
Updating dependencies (including require-dev)
Nothing to install or update
Package phpunit/php-token-stream is abandoned, you should avoid using it. No replacement was suggested.
Package phpunit/phpunit-mock-objects is abandoned, you should avoid using it. No replacement was suggested.
Generating autoload files
Pulling Headless-chromium-php from download ...
Archive: pdf_tmp.zip
  End-of-central-directory signature not found. Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive. In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip: cannot find zipfile directory in one of pdf_tmp.zip or
        pdf_tmp.zip.zip, and cannot find pdf_tmp.zip.ZIP, period.
Makefile:108: recipe for target 'pdfexport' failed
make: *** [pdfexport] Error 9

It appears that there has been a change in the availability of the master.zip file upstream- this will require us to fetch the correct files in our Makefile (and any other places it's pulled) in a different way.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/11537
Committed: https://git.mahara.org/mahara/mahara/commit/cb6e4b1c556e5054599c778aa898b14c9a543246
Submitter: Lisa Seeto (<email address hidden>)
Branch: master

commit cb6e4b1c556e5054599c778aa898b14c9a543246
Author: Robert Lyon <email address hidden>
Date: Tue Feb 2 11:18:30 2021 +1300

Bug 1914298: pdfexport fetch fails due to path changes

Change-Id: Id400b17fed284a3c5833d163717074b75189b178
Signed-off-by: Robert Lyon <email address hidden>

Lisa Seeto (lisaseeto)
Changed in mahara:
status: In Progress → Fix Committed
milestone: none → 21.04.0
Changed in mahara:
importance: Undecided → Medium
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "20.10_STABLE" branch: https://reviews.mahara.org/11749

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "20.04_STABLE" branch: https://reviews.mahara.org/11750

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/11750
Committed: https://git.mahara.org/mahara/mahara/commit/66e9887e23ca7d996f2ae3a3d23e9b135fb49df5
Submitter: Robert Lyon (<email address hidden>)
Branch: 20.04_STABLE

commit 66e9887e23ca7d996f2ae3a3d23e9b135fb49df5
Author: Robert Lyon <email address hidden>
Date: Tue Feb 2 11:18:30 2021 +1300

Bug 1914298: pdfexport fetch fails due to path changes

Change-Id: Id400b17fed284a3c5833d163717074b75189b178
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit cb6e4b1c556e5054599c778aa898b14c9a543246)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/11749
Committed: https://git.mahara.org/mahara/mahara/commit/48057aedd8e586757d9fc19f28c2723db0aa489d
Submitter: Robert Lyon (<email address hidden>)
Branch: 20.10_STABLE

commit 48057aedd8e586757d9fc19f28c2723db0aa489d
Author: Robert Lyon <email address hidden>
Date: Tue Feb 2 11:18:30 2021 +1300

Bug 1914298: pdfexport fetch fails due to path changes

Change-Id: Id400b17fed284a3c5833d163717074b75189b178
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit cb6e4b1c556e5054599c778aa898b14c9a543246)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.