-
Notifications
You must be signed in to change notification settings - Fork 86
Switch to composer from pear for PHP dependencies #555
Comments
Why me? @michaelbausor asking you as a PHP lead :) So we've got this in our
These commands are called from here: Lines 129 to 132 in 5bcb888
Do you know the correct way of fixing it? Thanks! |
@alexander-fenster haha meant to just cc you it's a blocker for me, so i'll look into it first. I asked michael about using composer instead and that seems like a good alternative. |
Another option that may be easier that using composer is to replace the There are instructions here on using curl or wget: https://github.com/squizlabs/PHP_CodeSniffer#installation. To keep things exactly the same, we should grab the 2.9.1 release from github. That said, composer is totally fine too, whichever is easiest. |
NOPE never mind |
Fixed in #557. Closing this. |
New images of artman won't build because it requires the
pear
util, whose server has been hacked and is now down.failure log https://circleci.com/gh/googleapis/artman/8964
pear's website: http://pear.php.net/
arstechnica article: https://arstechnica.com/information-technology/2019/01/pear-php-site-breach-lets-hackers-slip-malware-into-official-download/
We can switch to
composer
instead of pear.The text was updated successfully, but these errors were encountered: