ydb-platform/ydb-php-sdk

Add support of php 7.2

gfh31fgh2 opened this issue · 3 comments

Bug Report

YDB PHP SDK version:
latest (1.5.5)

Environment
ubuntu18 and JWT

Current behavior:

В текущем виде ydb-php-sdk не работает с php7.2 (стоит ограничение php>7.3)
Уберите пожалуйста такие ограничения, последний PR вполне годится и для старых версий.
У многих еще в ходу версия 7.2. Нет пока острой необходимости так резко повышать версию php.
У меня например старая БД работает через библиотеки php7.2, когда выбиралась ydb-php-sdk учитывалось это, тк для миграции нужно чтобы параллельно работало одновременно две БД на одних и тех же версиях.

Steps to reproduce:
Запустить установку ydb-sdk из php 7.2

rekby commented

@gfh31fgh2, подскажите какой из вариантов правильный:

  1. сначала 12 часов скрипты на 7.2 работают, а потом - проявляется какая-то ошибка несовместимости с 7.2.
  2. composer отказывается ставить зависимость, т.к. у нас указано минимальное требование 7.2, а вы используете php 7.3?
  3. Другой вариант (опишите пожалуйста)

2 вариант.
composer отказывается ставить зависимость, т.к. у вас указано минимальное требование 7.3, а я использую php 7.2

При этом я вручную накатил изменения 1.5.4 -> 1.5.5 - и все работает с версией 7.2
Но правильно, чтобы изменения прилетали в папку vendor, от composer.

все работает, спасибо!