Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Downloading PHPUnit twice ? #267

Closed
jrfnl opened this issue Jan 12, 2018 · 2 comments
Closed

Downloading PHPUnit twice ? #267

jrfnl opened this issue Jan 12, 2018 · 2 comments

Comments

@jrfnl
Copy link
Contributor

jrfnl commented Jan 12, 2018

Just wondering - I might well be reading the code incorrectly -, but by the looks of it, PHPUnit is potentially downloaded & installed twice ?

There may be an efficiency fix which can be made here or else a de-duplication.

mkdir -p $HOME/phpunit-bin/$PHPUNIT_VERSION
echo "Opting for phpunit $PHPUNIT_VERSION for PHP $TRAVIS_PHP_VERSION"
export PATH="$HOME/phpunit-bin/$PHPUNIT_VERSION:$PATH"
if [[ ! -e $HOME/phpunit-bin/$PHPUNIT_VERSION/phpunit ]]; then
wget -O $HOME/phpunit-bin/$PHPUNIT_VERSION/phpunit https://phar.phpunit.de/phpunit-$PHPUNIT_VERSION.phar
chmod +x $HOME/phpunit-bin/$PHPUNIT_VERSION/phpunit

wp-dev-lib/check-diff.sh

Lines 444 to 447 in 61a4add

PHPUNIT_VERSION=${PHPUNIT_VERSION:-5.7}
echo "Downloading PHPUnit $PHPUNIT_VERSION phar"
download https://phar.phpunit.de/phpunit-$PHPUNIT_VERSION.phar "$TEMP_TOOL_PATH/phpunit"
chmod +x "$TEMP_TOOL_PATH/phpunit"

@kasparsd
Copy link
Contributor

kasparsd commented Jul 2, 2019

Fixed in #301.

@kasparsd kasparsd closed this as completed Jul 2, 2019
@jrfnl
Copy link
Contributor Author

jrfnl commented Jul 2, 2019

🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants