Hi,

drupalCI fails on testing Textimage 8.x, basically due to 

fail: [PHP Fatal error] Line 441 of modules/textimage/src/Plugin/ImageToolkit/Operation/gd/TextimageTextToWrapper.php:
 Call to undefined function Drupal\textimage\Plugin\ImageToolkit\Operation\gd\imagettfbbox()

The tests run fine on the old infrastructure, see https://www.drupal.org/node/52349/qa

Maybe PHP is missing the FreeType library?

Comments

mondrake created an issue. See original summary.

isntall’s picture

isntall commented
Project: DrupalCI: Drupal.org Testing Infrastructure » DrupalCI: Test Runner
Component: Miscellaneous » Containers

The DCI container php does not have the FreeType library.
This will need to be added.

isntall’s picture

isntall commented
Assigned: Unassigned » isntall

I'm looking into what we need to do to get FreeType into the DCI container's PHP.
It might be a bit before this is implemented into production, though.

isntall’s picture

isntall commented
Related issues: +#2569169: php-build flags not being set
mondrake’s picture

mondrake
Location 🇮🇹
commented

Thanks for following up.

isntall’s picture

isntall commented

This is probably not going to be fixed until after DrupalCon Europe, but I'm going to try to get some dev builds of the DCI containers, so i can test if this fixes things.

isntall’s picture

isntall commented

Well that took longer to get setup than i would have hoped.

https://dispatcher.drupalci.org/job/staging_simpletest_job/141/

Looks like it is showing the same results as qa.d.o now

Note: this work still needs to be moved into production.

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Active » Reviewed & tested by the community

Thanks!

Yes the failures in the results are unrelated to this issue, but TextimageApiTest passes and that needs the FreeType library too, so I think this is good now.

Not sure if RTBC is the proper status here, but that seems to me the most appropriate :)

isntall’s picture

isntall commented
Issue tags: +Needs deployment
isntall’s picture

isntall commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs deployment

dev have been merged into production.

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Fixed » Active

Thanks, Textimage 8 is running OK now.

Sorry for being a pain, but unfortunately now the D7 version is failing on CI while (I just tested) it still passes on the old infra. Not a big deal as in any case there's not much (if nothing at all) going on on D7 versions ATM, but just wanted to raise it.

Thanks again

isntall’s picture

isntall commented
Status: Active » Fixed
Related issues: +#2606620: web-5.3 missing dockerfile copy cmd

That's a different issue, so this is being marked Fixed.

I've created an issue #2606620: web-5.3 missing dockerfile copy cmd

mondrake’s picture

mondrake
Location 🇮🇹
commented

Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.