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

Bugfix: Vendor-dir is defined in the config-section #2550

Merged
merged 1 commit into from Mar 9, 2016

Conversation

@ScreamingDev
Copy link
Contributor

commented Mar 9, 2016

The composer-vendor-dir can be defined in the config section of the composer.json.
Unfortunately the utils.php looks up the vendor dir at the wrong place
so the autoloader can never be reached.
This commit fixes the flaw in ::get_vendor_paths().

Bugfix: Vendor-dir is defined in the config-section
The composer-vendor-dir can be defined in the config section of the composer.json.
Unfortunately the utils.php looks up the vendor dir at another place.
This commit fixes the flaw in ::get_vendor_paths().
@danielbachhuber

This comment has been minimized.

Copy link
Member

commented Mar 9, 2016

Oops, good catch. Introduced in #1436

danielbachhuber added a commit that referenced this pull request Mar 9, 2016
Merge pull request #2550 from sourcerer-mike/vendor-dir-bug
Bugfix: Vendor-dir is defined in the config-section

@danielbachhuber danielbachhuber merged commit 1116515 into wp-cli:master Mar 9, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ScreamingDev ScreamingDev deleted the ScreamingDev:vendor-dir-bug branch Mar 9, 2016

@ScreamingDev

This comment has been minimized.

Copy link
Contributor Author

commented Mar 9, 2016

Thanks for merging so fast! :)
Looking forward for the release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.