Problem/Motivation

Following the instructions in tests/README.txt fails to correctly install phpunit, complaining that it can't find the channel "pear.symfony.com" to install Yaml.

Proposed resolution

As per this Stack Overflow response, the channel should be added and (possibly optionally) the Yaml package explicitly installed, prior to running the final --alldeps installation.

Patch to follow to add these to the README.txt

Remaining tasks

None.

User interface changes

None.

API changes

None.

None.

CommentFileSizeAuthor
#3 drush-update_readme-2019059-3.patch448 bytesergonlogic
#1 drush-phpunit-and-symfony.com-2019059-1.patch791 bytesjp.stacey

Comments

jp.stacey’s picture

jp.stacey commented
Status: Needs work » Needs review
StatusFileSize
new drush-phpunit-and-symfony.com-2019059-1.patch791 bytes

Simple documentation patch attached for review.

greg.1.anderson’s picture

greg.1.anderson commented

I believe this would be obsoleted by #2010042: Use composer to configure the test environment

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Version: 7.x-5.8 » 7.x-5.x-dev
Priority: Minor » Normal
StatusFileSize
new drush-update_readme-2019059-3.patch448 bytes

Would #2010042: Use composer to configure the test environment be backported to Drush 5? This is still a problem with the current documentation that should be fixed, no?

Also, the --alldeps is enough to install this properly, revised patch attached.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

Committed to 5.x and 6.x

Status: Fixed » Closed (fixed)

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