CRM-18267 upgrade php-font-lib

[anuditverma]

Updated php-font-lib package version to latest 0.4 release.

---

• CRM-18267: upgrade php-font-lib

[civicrm-builder]

Can one of the admins verify this patch?

[eileenmcnaughton]

test this please

[eileenmcnaughton]

ok to test

[anuditverma]

Hi Eileen, I just tracked this on jenkins, there is a checkstyle error (No report files were found). I'm not sure what exactly could have caused this. Could you please help me to get over this one ? Thanks

[eileenmcnaughton]

let's re-run the test in case it was unrelated!

[eileenmcnaughton]

test this please

[anuditverma]

Hi again, Its showing the same, so any problems in my commits then ?

[colemanw]

Why is composer.json not updated in this PR? Does the lock file take care of version?

[colemanw]

The testbot is reporting:

Could not scan for classes inside "/home/jenkins/buildkit/build/core-8165-8vwvc/sites/all/modules/civicrm/vendor/phenx/php-font-lib/classes/" which does not appear to be a file nor a folder

Perhaps the composer update should be re-run from a clean copy of master.

[anuditverma]

So I need to update composer.json with the latest proposed commit ? And for the second part I guess I need to do this through a new copy of master, again.

[eileenmcnaughton]

that sounds about right - there should be a change to both composer.json & composer.lock & yeah, it's good practice to do a

git pull --rebase origin master

before reworking a patch

[anuditverma]

Hi, sorry I have been very busy with my major exams, I received Tim's email and I decided to get this one done. I did rebasing and made changes to composer.lock file but while editing composer.json I realized that php-font-lib has not been defined under "require": and I think some of the others are not there too. So do I need to add all other libraries as well that are defined in lock file ?

[anuditverma]

Submitted a new PR : #8335