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

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

@jderusse jderusse changed the base branch from 5.x to 4.4 January 20, 2021 18:24
@jderusse jderusse added this to the 4.4 milestone Jan 20, 2021
@carsonbot carsonbot changed the title Allow relative path to composer cache [PhpUnitBridge] Allow relative path to composer cache Jan 20, 2021
@nicolas-grekas
Copy link
Member

Thank you @jderusse.

@nicolas-grekas nicolas-grekas merged commit 0c7eb27 into symfony:4.4 Jan 23, 2021
@jderusse jderusse deleted the phpunit-composer-cache branch January 25, 2021 09:16
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.

[PhpUnitBridge] Composer-cache is not used
3 participants