Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PhpUnitBridge] Allow relative path to composer cache #39909

Merged
merged 1 commit into from Jan 23, 2021

Conversation

jderusse
Copy link
Member

@jderusse jderusse commented Jan 20, 2021

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets fix #37652
License MIT
Doc PR -

When users defines a relative path in the COMPOSER_CACHE_DIR env variable,
The simple-phpunit code, by changing the directory (with chdir), give to composer a different context. At the end, composer stores its cache inside the vendor/bin/.phpunit folder, and inside the vendor/bin/.phpunit/phpunit-X.Y.Z folder.

This PR convert the relative path provided by the user into Absolute path.

@nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Jan 20, 2021

Why not as a bugfix? It looks like one to me.

@jderusse jderusse force-pushed the phpunit-composer-cache branch from 2168475 to 6e464c6 Compare Jan 20, 2021
@jderusse jderusse changed the base branch from 5.x to 4.4 Jan 20, 2021
@jderusse jderusse added this to the 4.4 milestone Jan 20, 2021
@jderusse jderusse force-pushed the phpunit-composer-cache branch from 6e464c6 to aea8b98 Compare Jan 20, 2021
@carsonbot carsonbot changed the title Allow relative path to composer cache [PhpUnitBridge] Allow relative path to composer cache Jan 20, 2021
@jderusse jderusse force-pushed the phpunit-composer-cache branch from aea8b98 to cde0ffd Compare Jan 20, 2021
@nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Jan 23, 2021

Thank you @jderusse.

@nicolas-grekas nicolas-grekas merged commit 0c7eb27 into symfony:4.4 Jan 23, 2021
4 of 6 checks passed
@jderusse jderusse deleted the phpunit-composer-cache branch Jan 25, 2021
This was referenced Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants