search

jack-theripper / yandex

PHP SDK для работы с Яндекс Диском yandex disk
MIT License
127 stars 39 forks source link

update vendors #33

Closed PheRum closed 2 years ago

PheRum commented 5 years ago

Указанные явно версии дают косяк при установке пакета на более новый проектах.

Обновите пожалуйста сразу и версию номер тега

jack-theripper commented 5 years ago

Могу взглянуть на лог о проблеме? У Вас в проекте конфликт версий или что произошло?

PheRum commented 5 years ago

@jack-theripper хм, странно сейчас уже устанавливается корректно, но все равно непонятно зачем версии вендоров указаны статично? может быть лучшим решением было бы повысить мажорную цифру в версии, если есть некоторые проблемы с версиями php или чем-то другим?

jack-theripper commented 5 years ago

Библиотека по-прежнему остаётся на версии PHP >= 5.6, в связи с этим zend-diactoros(1.8.4) крайняя где исправлена уязвимость и PHP ^5.6. Что касается зависимости от league/event, думаю, да можно обновить, хоть и changelog не большой)

PheRum commented 5 years ago

league/event просто так не получится обнови на 2.2, он будет ругаться на низкую версию zend-diactoros, которую тоже нужно будет обновить) именно это и делает мой PR)

jack-theripper commented 5 years ago

league/event самостоятельная библиотека, которая не требует каких-либо зависимостей - я с вами согласен, ее можно обновить до ^2.2. Минимальная версия PHP 5.5, ваш PR поднимет ее до 7.1 только из-за зависимостей, но в то же время сам код библиотеки для работы с яндекс диском никаких преимуществ PHP 7 не использует :-(((((

Вы поясните пожалуйста, что за проблема у вас возникла с этой библиотекой при установке на новых проектах?)

StudioMaX commented 4 years ago

Проблема тут думаю очевидная: если у проекта, к которому подключается эта библиотека, есть другие зависимости, то они войдут в конфликт с зависимостями этой библиотеки. Пример:

У этой библиотеки зависимости: "zendframework/zend-diactoros": "~1.8.4", - смотрим на https://packagist.org/packages/zendframework/zend-diactoros , на данный момент последняя версия 2.2.1, к тому же zendframework был переименован в laminas (но мы сейчас не об этом) А это значит, что при наличии в проекте любой другой библиотеки, у которой в зависимостях стоит что-то типа "zendframework/zend-diactoros": "~2.2", произойдёт конфликт между библиотеками.

"league/event": "2.1.2", - аналогично, смотрим https://packagist.org/packages/league/event - последняя версия 2.2.0, и если у какой-то другой зависимости стоит "league/event": "^2.2", то будет конфликт.

Ну и дальше по списку "php-http/curl-client": "1.4.*", - текущая версия 2.1.0 (https://packagist.org/packages/php-http/curl-client) "php-http/client-common": "^1.1" - текущая версия 2.1.0 (https://packagist.org/packages/php-http/client-common)

jack-theripper commented 4 years ago

@StudioMaX сейчас поддержки php ^7.1 нет zend-diactoros последняя 1.8.7 с поддержкой 5.6 (~1.8.4 если правильно помню должен быть эквивалент >=1.8.4 <1.9.0), а в curl-client утечка памяти и обновление не возможно - тут нужно переписывать на другую библиотеку. Сейчас pr открыт, как только так сразу приму ;-)

jack-theripper commented 3 years ago

@PheRum Здравствуйте, а вы форк обновили? // Вроде уже есть laminas если я правильно помню

PheRum commented 3 years ago

@jack-theripper да, форк обновил, так как с яндексом уже сейчас работать нужно, а в этой репе что-то движений нет( странно, что сюда что-то прилетело, вроде пулл реквест не создавал)