Fix vendor location #2
Conversation
Fixing issue with autoloader when executing from outside the source directory: php-parallel-lint#1
|
@grogy Any chances this fix will be merged soon? Our CI checks have started to fail recently. Not sure what changed, but it cannot find This change fixes it. |
|
Thank you. This section can be replaced with using PSR-4 and autoloading in composer. I do it later or anyone can send PR :-) |
|
@grogy I would, but there's still a few too many open PRs waiting.... |
|
You have right. I will resolve it during next 7 days |
|
@grogy Happy to hear that and I look forward to the next release. |
I'm afraid this is not precisely correct. As this section is solving a different problem. Changing classmap autoloading to PSR-4 will not change anything in regards to this exact problem. |
|
I tried the same situation. Do you have right @e1himself PSR-4 does not solve this case. Example for sumulation: # show folder structure
$ ls /app
composer.json vendor ...
# clone this tool
$ git clone [email protected]:php-parallel-lint/PHP-Parallel-Lint.git /app/folder1/folder2/folder3/folder4/folder5/php-parallel-lint
# run with error
$ cd /
$ php /app/folder1/folder2/folder3/folder4/folder5/php-parallel-lint/parallel-lint
You must set up the project dependencies, run the following commands:
curl -s http://getcomposer.org/installer | php
php composer.phar installSolution is run create-project like and 1 small fix in code. composer create-project php-parallel-lint/php-parallel-lint:dev-master /app/folder-v2-1/folder2/folder3/folder4/folder5/php-parallel-lintPlease try it in your project. |
|
@jrfnl ou :( it's the same solution. Credit for a change belong to you |
|
@grogy No worries. Just glad it's fixed in the repo now. Might have saved you some time though if you'd looked at the open PRs first. |
Fixing issue with autoloader when executing from outside the source directory: #1